Merge pull request 'Rename async job variables for consistency' (#5) from fix/basic_tag_violation into main

Reviewed-on: #5

roles/ufw/tasks/main.yml

@@ -18,11 +18,11 @@
 - name: Add UFW to default runlevel (Alpine)
   ansible.builtin.command:
     cmd: rc-update add ufw default
-  register: rc_update_result
-  changed_when: >- # ">- : transform multiple lines into one line"
-    'already' not in rc_update_result.stdout
-    and 'already' not in rc_update_result.stderr
-  failed_when: rc_update_result.rc != 0
+  register: ufw_rc_update_result
+  changed_when: >- # " >- : transform multiple lines into one line"
+    'already' not in ufw_rc_update_result.stdout
+    and 'already' not in ufw_rc_update_result.stderr
+  failed_when: ufw_rc_update_result.rc != 0
   when: ansible_facts['os_family'] == "Alpine"
 
 - name: Set default incoming policy

roles/upgrade/tasks/main.yml

@@ -10,7 +10,7 @@
     update_cache: true
   async: 3600
   poll: 0
-  register: alpine_upgrade
+  register: upgrade_alpine_job
   # By default async job return change: true
   # Force adapting the change condition for idempotency
   changed_when: false
@@ -18,18 +18,18 @@
 
 - name: Wait for Alpine upgrade
   ansible.builtin.async_status:
-    jid: "{{ alpine_upgrade.ansible_job_id }}"
-  register: alpine_result
-  until: alpine_result.finished
+    jid: "{{ upgrade_alpine_job.ansible_job_id }}"
+  register: upgrade_alpine_result
+  until: upgrade_alpine_result.finished
   retries: 60
   delay: 10
   # By default async job return change: true
   # Force adapting the change condition for idempotency
-  changed_when: alpine_result.changed | default(false)
+  changed_when: upgrade_alpine_result.changed | default(false)
   ignore_unreachable: true
   when:
     - ansible_facts['os_family'] == "Alpine"
-    - alpine_upgrade.ansible_job_id is defined
+    - upgrade_alpine_job.ansible_job_id is defined
 
 - name: Upgrade all packages (Debian)
   ansible.builtin.apt:
@@ -37,7 +37,7 @@
     update_cache: true
   async: 3600
   poll: 0
-  register: debian_upgrade
+  register: upgrade_debian_job
   # By default async job return change: true
   # Force adapting the change condition for idempotency
   changed_when: false
@@ -45,15 +45,15 @@
 
 - name: Wait for Debian upgrade
   ansible.builtin.async_status:
-    jid: "{{ debian_upgrade.ansible_job_id }}"
-  register: debian_result
-  until: debian_result.finished
+    jid: "{{ upgrade_debian_job.ansible_job_id }}"
+  register: upgrade_debian_result
+  until: upgrade_debian_result.finished
   retries: 60
   delay: 10
   # By default async job return change: true
   # Force adapting the change condition for idempotency
-  changed_when: debian_result.changed | default(false)
+  changed_when: upgrade_debian_result.changed | default(false)
   ignore_unreachable: true
   when:
     - ansible_facts['os_family'] == "Debian"
-    - debian_upgrade.ansible_job_id is defined
+    - upgrade_debian_job.ansible_job_id is defined