Merge pull request #9 from mbaldessari/ansible

Fix Ansible linting job

Author: mbaldessari

.ansible-lint

@@ -19,3 +19,4 @@ warn_list:
 
 skip_list:
   - var-naming[no-role-prefix]
+  - no-changed-when

ansible/odf_fix_dataimportcrons.yml

@@ -20,7 +20,7 @@
 
     - name: Exit if normal version check is not right
       ansible.builtin.meta: end_play
-      when: cluster_version not in [ '4.16', 'UNSET' ]
+      when: cluster_version not in ['4.16', 'UNSET']
 
     - name: Find default storageclass
       ansible.builtin.shell: |
@@ -70,7 +70,7 @@
 
         - name: Extract dic names
           ansible.builtin.set_fact:
-            dic_names: "{{ dic_names | default([]) + [ item.metadata.name ] }}"
+            dic_names: "{{ dic_names | default([]) + [item.metadata.name] }}"
           loop: "{{ dic_list.resources }}"
 
         - name: Show names
@@ -91,8 +91,8 @@
 
         - name: Record datasources that need cleanup
           ansible.builtin.set_fact:
-            cron_cleanups: "{{ cron_cleanups + [ item.metadata.labels['cdi.kubevirt.io/dataImportCron'] ] }}"
-            ds_cleanups: "{{ ds_cleanups + [ item.metadata.name ] }}"
+            cron_cleanups: "{{ cron_cleanups + [item.metadata.labels['cdi.kubevirt.io/dataImportCron']] }}"
+            ds_cleanups: "{{ ds_cleanups + [item.metadata.name] }}"
           loop: "{{ ds_cleanup_list.resources }}"
           when:
             - item['metadata']['labels']['cdi.kubevirt.io/dataImportCron'] is defined