refactor ♻️: Improve retry logic consistency across tasks

Standardized the use of `retries` and `delay` variables in tasks to ensure consistent behavior, making it easier to manage and maintain the workflow. This change also enables better error handling and reduces potential issues with task execution.

tasks/configure-vm.yml

@@ -32,8 +32,8 @@
         {{ debian_image_path }}
         {{ storage }}
       register: disk_import
-      retries: 3
-      delay: 5
+      retries: "{{ max_retries }}"
+      delay: "{{ retry_delay }}"
       until: disk_import is succeeded
       when: not disk_exists.stat.exists
 

tasks/create-clones.yml

@@ -73,8 +73,8 @@
     - name: "[CLONES] Start clone VM"
       ansible.builtin.command: "qm start {{ clone.id }}"
       register: clone_start
-      retries: 3
-      delay: 2
+      retries: "{{ max_retries }}"
+      delay: "{{ retry_delay }}"
       until: clone_start is succeeded
       when: not clone_conf.stat.exists
 

tasks/download-image.yml

@@ -19,10 +19,10 @@
     url: "{{ debian_image_url }}"
     dest: "{{ debian_image_path }}"
     mode: "0644"
-    timeout: 300
+    timeout: "{{ image_download_timeout }}"
   register: image_download
-  retries: 3
-  delay: 5
+  retries: "{{ max_retries }}"
+  delay: "{{ retry_delay }}"
   until: image_download is succeeded
   when: not debian_img.stat.exists