playbooks: ansible-lint fix key-order

Fix key-order ansible-lint rule violations

Fixed key-order rule violations across 3 file(s).

Affected files:
- playbooks/roles/bootlinux/tasks/main.yml
- playbooks/roles/devconfig/tasks/check-apt-mirrors.yml
- playbooks/roles/mmtests_compare/tasks/main.yml

Applied using the ansible-lint-fix-rules.py script with --fix=key-order.

Generated-by: Ansible Lint (ansible-lint-fix-rules.py)
Signed-off-by: Daniel Gomez <[email protected]>

Author: dkruces

playbooks/roles/bootlinux/tasks/main.yml

@@ -126,6 +126,11 @@
 
 - name: Debug kernel ref settings for 9P builds
   delegate_to: localhost
+  when:
+    - bootlinux_debug_ref|bool
+    - bootlinux_9p|bool
+  run_once: true
+
   block:
     - name: Print kernel ref settings for 9P debug (localhost context)
       ansible.builtin.debug:
@@ -144,12 +149,11 @@
 
     - name: End play gracefully for kernel ref debug
       ansible.builtin.meta: end_play
+- name: Debug kernel ref settings for non-9P builds
   when:
     - bootlinux_debug_ref|bool
-    - bootlinux_9p|bool
-  run_once: true
+    - not bootlinux_9p|bool
 
-- name: Debug kernel ref settings for non-9P builds
   block:
     - name: Print kernel ref settings for non-9P debug (per-node context)
       ansible.builtin.debug:
@@ -169,10 +173,6 @@
 
     - name: End play gracefully for kernel ref debug
       ansible.builtin.meta: end_play
-  when:
-    - bootlinux_debug_ref|bool
-    - not bootlinux_9p|bool
-
 - name: Create data partition
   ansible.builtin.include_role:
     name: create_data_partition

playbooks/roles/devconfig/tasks/check-apt-mirrors.yml

@@ -107,6 +107,12 @@
         - hop1_mirror_host is defined
 
     - name: Use hop1 mirror with DEB822 format for debian testing
+      when:
+        - has_hop1_mirror is defined
+        - has_hop1_mirror | bool
+        - hop1_mirror_connectivity is not failed
+        - hop1_mirror_host is defined
+
       block:
         - name: Backup current sources
           ansible.builtin.copy:
@@ -141,20 +147,14 @@
             cache_valid_time: 0
           become: true
 
-        - name: Inform user about hop1 mirror usage
-          ansible.builtin.debug:
+        - ansible.builtin.debug:
             msg: |
               Using local hop1 mirror with DEB822 format:
               Mirror: {{ hop1_mirror_host }}{{ hop1_mirror_path }}
 
               This provides faster package downloads from your local mirror.
               Sources have been converted to modern DEB822 format.
-      when:
-        - has_hop1_mirror is defined
-        - has_hop1_mirror | bool
-        - hop1_mirror_connectivity is not failed
-        - hop1_mirror_host is defined
-
+          name: Inform user about hop1 mirror usage
 - name: Fall back to official Debian mirrors if current mirror fails
   block:
     - name: Backup current sources (DEB822 format)

playbooks/roles/mmtests_compare/tasks/main.yml

@@ -138,6 +138,7 @@
   register: patch_results
 
 - name: Get kernel versions from nodes
+  tags: ["compare"]
   block:
     - name: Get baseline kernel version
       ansible.builtin.command: uname -r
@@ -153,7 +154,6 @@
       when:
         - groups['dev'] is defined
         - groups['dev'] | length > 0
-  tags: ["compare"]
 
 - name: Set node information facts
   ansible.builtin.set_fact:
@@ -274,6 +274,9 @@
 
 - name: Generate performance graphs using gnuplot
   delegate_to: localhost
+  run_once: true
+  when: kdevops_baseline_and_dev|bool
+  tags: ["compare", "graphs"]
   block:
     - name: Check for available iterations data
       ansible.builtin.find:
@@ -305,9 +308,6 @@
       failed_when: false
       environment:
         MMTESTS_AUTO_PACKAGE_INSTALL: never
-  run_once: true
-  when: kdevops_baseline_and_dev|bool
-  tags: ["compare", "graphs"]
 
 - name: Find generated graph files
   delegate_to: localhost