Merge "Suppressing ignore-errors rule in irrelevant cases"

roles/environment/teardown/tasks/main.yml

@@ -33,7 +33,7 @@
         virsh net-destroy "{{ item.item.name }}"
       when: libvirt_check is success and item is success
       with_items: "{{ network_check.results }}"
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
       become: true
 
     - name: Undefine libvirt networks
@@ -41,7 +41,7 @@
         virsh net-undefine "{{ item.item.name }}"
       when: libvirt_check is success and item is success
       with_items: "{{ network_check.results }}"
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
       become: true
 
     - name: Remove bridge whitelisting from qemu bridge helper

roles/libvirt/teardown/nodes/tasks/main.yml

@@ -94,7 +94,7 @@
       command: >
         virsh destroy "{{ undercloud_node.name }}"
       when: undercloud_check is success
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
 
     - name: Undefine undercloud vm
       command: >
@@ -107,7 +107,7 @@
           '{{ undercloud_node.name }}'.qcow2 2>&1 > /dev/null
         virsh vol-delete --pool '{{ libvirt_volume_pool }}' \
           '{{ undercloud_node.name }}'.qcow2
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
 
     - name: Destroy intermediate disk image
       file:
@@ -133,7 +133,7 @@
       command: >
         virsh pool-destroy "{{ libvirt_volume_pool }}"
       when: pool_check is success
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
 
     - name: Undefine volume pool
       command: >
@@ -160,4 +160,4 @@
       shell: |
          nmcli c down brovc;
          nmcli c down bridge-slave-{{ virthost_provisioning_interface }}
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors

roles/libvirt/teardown/user/tasks/main.yml

@@ -9,4 +9,4 @@
     manage_dir: true
   with_file:
     - "{{ virt_power_key }}.pub"
-  ignore_errors: true
+  ignore_errors: true  # noqa ignore-errors

roles/provision/teardown/tasks/main.yml

@@ -22,7 +22,7 @@
     - name: Kill (SIGTERM) all processes owned by non-root user
       command: >
         pkill -u {{ non_root_user }}
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
       become: true
       when: proc_exist is success
 
@@ -41,7 +41,7 @@
       command: >
         pkill -9 -u {{ non_root_user }}
       when: proc_exist is success
-      ignore_errors: true
+      ignore_errors: true  # noqa ignore-errors
       become: true
 
     # Now that we have taken care of any processes owned by this user