playbooks: ansible-lint fix jinja

Fix jinja ansible-lint rule violations

Fixed jinja rule violations across 34 file(s).

Affected files include:
- playbooks/roles/blktests/tasks/main.yml
- playbooks/roles/bootlinux/tasks/install/packages.yml
- playbooks/roles/bootlinux/tasks/main.yml
- playbooks/roles/build_qemu/tasks/install-deps/main.yml
- playbooks/roles/cxl/tasks/main.yml
... and 29 more files

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

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

Author: dkruces

playbooks/roles/blktests/tasks/main.yml

@@ -178,7 +178,7 @@
 
 - name: Store last kernel variable
   ansible.builtin.set_fact:
-    last_kernel: "{{ uname_cmd.stdout_lines | regex_replace('\\]') | regex_replace('\\[') | replace(\"'\",'') }}"
+    last_kernel: "{{ uname_cmd.stdout_lines | regex_replace('\\]') | regex_replace('\\[') | replace(\"'\", '') }}"
   tags: ["blktests", "copy_results", "augment_expunge_list"]
   run_once: true
 

playbooks/roles/bootlinux/tasks/install/packages.yml

@@ -23,7 +23,7 @@
 
     - name: Add the core kernel package
       ansible.builtin.set_fact:
-        kernel_packages: "{{ kernel_packages + ['/tmp/' + item.path | basename ] }}"
+        kernel_packages: "{{ kernel_packages + ['/tmp/' + item.path | basename] }}"
       when:
         - "'devel' not in item.path"
         - "'headers' not in item.path"

playbooks/roles/bootlinux/tasks/main.yml

@@ -111,11 +111,11 @@
 - name: Determine active kernel parameters for A/B testing with 9P
   ansible.builtin.set_fact:
     target_linux_git: "{{ bootlinux_dev_tree if bootlinux_dev_tree != '' else target_linux_git }}"
-    active_linux_ref: "{{ target_linux_dev_ref if targeting_dev_nodes|default(false)|bool else target_linux_ref }}"
-    active_linux_kernelrelease: "{{ target_linux_dev_kernelrelease if (targeting_dev_nodes|default(false)|bool and bootlinux_tree_custom_kernelrelease|bool) else
-      target_linux_kernelrelease }}"
-    active_linux_localversion: "{{ target_linux_dev_localversion if (targeting_dev_nodes|default(false)|bool and bootlinux_tree_custom_localversion|bool) else target_linux_localversion
-      }}"
+    active_linux_ref: "{{ target_linux_dev_ref if targeting_dev_nodes | default(false) | bool else target_linux_ref }}"
+    active_linux_kernelrelease: "{{ target_linux_dev_kernelrelease if (targeting_dev_nodes | default(false) | bool and bootlinux_tree_custom_kernelrelease | bool)
+      else target_linux_kernelrelease }}"
+    active_linux_localversion: "{{ target_linux_dev_localversion if (targeting_dev_nodes | default(false) | bool and bootlinux_tree_custom_localversion | bool) else
+      target_linux_localversion }}"
     target_linux_config: "config-{{ target_linux_dev_ref }}"
   when:
     - kdevops_baseline_and_dev|bool

playbooks/roles/build_qemu/tasks/install-deps/main.yml

@@ -4,7 +4,7 @@
   ignore_errors: true
   with_first_found:
     - files:
-        - "{{ ansible_facts['os_family']|lower }}.yml"
+        - "{{ ansible_facts['os_family'] | lower }}.yml"
       skip: true
   tags: vars
 

playbooks/roles/cxl/tasks/main.yml

@@ -84,7 +84,7 @@
 
 - name: Store last kernel variable
   ansible.builtin.set_fact:
-    last_kernel: "{{ uname_cmd.stdout_lines | regex_replace('\\]') | regex_replace('\\[') | replace(\"'\",'') }}"
+    last_kernel: "{{ uname_cmd.stdout_lines | regex_replace('\\]') | regex_replace('\\[') | replace(\"'\", '') }}"
   tags: ["ndctl", "cxl-test-prep", "copy_results"]
   run_once: true
 

playbooks/roles/devconfig/tasks/install-deps/main.yml

@@ -20,7 +20,7 @@
   ignore_errors: true
   with_first_found:
     - files:
-        - "{{ ansible_facts['os_family']|lower }}.yml"
+        - "{{ ansible_facts['os_family'] | lower }}.yml"
       skip: true
   tags: vars
 

playbooks/roles/devconfig/tasks/install-deps/redhat/main.yml

@@ -109,21 +109,21 @@
 
 - name: Add btrfs-progs to install package list
   ansible.builtin.set_fact:
-    packages: "{{ packages + [ 'btrfs-progs' ] }}"
+    packages: "{{ packages + ['btrfs-progs'] }}"
   when:
     - devconfig_try_install_kdevtools|bool
     - ansible_distribution == 'Fedora'
 
 - name: Add GNU screen to install package list
   ansible.builtin.set_fact:
-    packages: "{{ packages + [ 'screen' ] }}"
+    packages: "{{ packages + ['screen'] }}"
   when:
     - devconfig_try_install_kdevtools|bool
     - ansible_facts['os_family']|lower != 'redhat' or ansible_facts['distribution_major_version'] | int < 8
 
 - name: Add Tmux to install package list
   ansible.builtin.set_fact:
-    packages: "{{ packages + [ 'tmux' ] }}"
+    packages: "{{ packages + ['tmux'] }}"
   when:
     - devconfig_try_install_kdevtools|bool
     - ansible_facts['os_family']|lower == 'redhat' or ansible_facts['distribution_major_version'] | int >= 8

playbooks/roles/devconfig/tasks/kotd-rev-kernel/main.yml

@@ -13,7 +13,7 @@
   ignore_errors: true
   with_first_found:
     - files:
-        - "{{ ansible_facts['os_family']|lower }}.yml"
+        - "{{ ansible_facts['os_family'] | lower }}.yml"
       skip: true
   tags: vars
 

playbooks/roles/devconfig/tasks/main.yml

@@ -627,7 +627,7 @@
 - name: "Group up facts we'll give to the host"
   tags: ["journal_ln"]
   ansible.builtin.set_fact:
-    node_host_ip: "{{ ansible_ssh_host}} {{ ansible_default_ipv4.address }}"
+    node_host_ip: "{{ ansible_ssh_host }} {{ ansible_default_ipv4.address }}"
   delegate_facts: true
   when: devconfig_enable_systemd_journal_remote|bool
 

playbooks/roles/fstests/tasks/main.yml

@@ -611,8 +611,8 @@
     number: "{{ item }}"
     state: present
     part_type: "logical"
-    part_start: "{{ ((item - 5) * test_partsize ) + 1 }}GiB"
-    part_end: "{{ ((item - 4) * test_partsize ) }}GiB"
+    part_start: "{{ ((item - 5) * test_partsize) + 1 }}GiB"
+    part_end: "{{ ((item - 4) * test_partsize) }}GiB"
   loop: "{{ range(5, 17) | list }}"
   when:
     - fstests_testdev_nvme_partition_euis|bool
@@ -730,7 +730,7 @@
 - name: Set the export volname prefix
   tags: vars
   ansible.builtin.set_fact:
-    volname_prefix: "{{ ansible_host|regex_replace(kdevops_host_prefix + '-') }}-fs"
+    volname_prefix: "{{ ansible_host | regex_replace(kdevops_host_prefix + '-') }}-fs"
   when:
     - fstests_fstyp == "nfs" or fstests_fstyp == "cifs"
 
@@ -1100,7 +1100,7 @@
 
 - name: Store last kernel variable
   ansible.builtin.set_fact:
-    last_kernel: "{{ uname_cmd.stdout_lines | regex_replace('\\]') | regex_replace('\\[') | replace(\"'\",'') }}"
+    last_kernel: "{{ uname_cmd.stdout_lines | regex_replace('\\]') | regex_replace('\\[') | replace(\"'\", '') }}"
   tags: ["oscheck", "fstests", "copy_results", "print_results", "augment_expunge_list"]
   run_once: true
 
@@ -1194,7 +1194,7 @@
     chdir: "{{ fstests_data_target }}"
   environment:
     FSTYP: "{{ fstests_fstyp }}"
-    FSTESTS_LINUX_LOCALVERSION: "{{ target_linux_localversion  | default ('') }}"
+    FSTESTS_LINUX_LOCALVERSION: "{{ target_linux_localversion | default('') }}"
     KEEP_DMESG: "yes"
   register: failed_tests_limit
   when:
@@ -1273,11 +1273,11 @@
     FSTYP: "{{ fstests_fstyp }}"
     FSTESTS_SPARSE_FILE_PATH: "{{ sparsefiles_path }}"
     FSTESTS_SETUP_SYSTEM: "{{ fstests_setup_system }}"
-    FSTESTS_LINUX_LOCALVERSION: "{{ target_linux_localversion  | default ('') }}"
+    FSTESTS_LINUX_LOCALVERSION: "{{ target_linux_localversion | default('') }}"
     FSTESTS_TESTDEV_SPARSEFILE_GENERATION: "{{ sparsefiles_generation }}"
     FSTESTS_SPARSE_FILE_SIZE: "{{ sparsefiles_size }}"
     FSTESTS_SPARSE_FILENAME_PREFIX: "{{ sparsefiles_filename_prefix }}"
-    FSTESTS_RUN_LARGE_DISK_TESTS: "{{ run_large_disk_tests | default ('') }}"
+    FSTESTS_RUN_LARGE_DISK_TESTS: "{{ run_large_disk_tests | default('') }}"
     FSTESTS_RUN_AUTO_GROUP_TESTS: "{{ run_auto_group_tests }}"
     FSTESTS_RUN_CUSTOM_GROUP_TESTS: "{{ run_custom_group_tests }}"
     FSTESTS_EXCLUDE_TEST_GROUPS: "{{ exclude_test_groups }}"