Fix a bunch of failed quality rules (#6646)

tests/cloud_playbooks/upload-logs-gcs.yml

@@ -55,6 +55,7 @@
 
     - name: Apply the lifecycle rules  # noqa 301
       command: "{{ dir }}/google-cloud-sdk/bin/gsutil lifecycle set {{ dir }}/gcs_life.json gs://{{ test_name }}"
+      changed_when: false
       environment:
         BOTO_CONFIG: "{{ dir }}/.boto"
       no_log: True

tests/testcases/015_check-nodes-ready.yml

@@ -15,8 +15,8 @@
   - import_role:
       name: cluster-dump
 
-  - name: Check kubectl output  # noqa 305
+  - name: Check kubectl output
-    shell: "{{ bin_dir }}/kubectl get nodes"
+    command: "{{ bin_dir }}/kubectl get nodes"
     changed_when: false
     register: get_nodes
     no_log: true
@@ -24,8 +24,8 @@
   - debug:
       msg: "{{ get_nodes.stdout.split('\n') }}"
 
-  - name: Check that all nodes are running and ready  # noqa 305
+  - name: Check that all nodes are running and ready
-    shell: "{{ bin_dir }}/kubectl get nodes --no-headers -o yaml"
+    command: "{{ bin_dir }}/kubectl get nodes --no-headers -o yaml"
     changed_when: false
     register: get_nodes_yaml
     until:

tests/testcases/020_check-pods-running.yml

@@ -15,8 +15,8 @@
   - import_role:
       name: cluster-dump
 
-  - name: Check kubectl output  # noqa 305
+  - name: Check kubectl output
-    shell: "{{ bin_dir }}/kubectl get pods --all-namespaces -owide"
+    command: "{{ bin_dir }}/kubectl get pods --all-namespaces -owide"
     changed_when: false
     register: get_pods
     no_log: true
@@ -24,8 +24,8 @@
   - debug:
       msg: "{{ get_pods.stdout.split('\n') }}"
 
-  - name: Check that all pods are running and ready  # noqa 305
+  - name: Check that all pods are running and ready
-    shell: "{{ bin_dir }}/kubectl get pods --all-namespaces --no-headers -o yaml"
+    command: "{{ bin_dir }}/kubectl get pods --all-namespaces --no-headers -o yaml"
     changed_when: false
     register: run_pods_log
     until:
@@ -38,8 +38,8 @@
     failed_when: false
     no_log: true
 
-  - name: Check kubectl output  # noqa 305
+  - name: Check kubectl output
-    shell: "{{ bin_dir }}/kubectl get pods --all-namespaces -owide"
+    command: "{{ bin_dir }}/kubectl get pods --all-namespaces -owide"
     changed_when: false
     register: get_pods
     no_log: true

tests/testcases/030_check-network.yml

@@ -39,11 +39,12 @@
 
     when: kubelet_rotate_server_certificates | default(false)
 
-  - name: Create test namespace  # noqa 301 305
+  - name: Create test namespace
-    shell: "{{ bin_dir }}/kubectl create namespace test"
+    command: "{{ bin_dir }}/kubectl create namespace test"
+    changed_when: false
 
-  - name: Run 2 busybox pods in test ns  # noqa 305
+  - name: Run 2 busybox pods in test ns
-    shell: "{{ bin_dir }}/kubectl run {{ item }} --image={{ test_image_repo }}:{{ test_image_tag }} --namespace test --command -- tail -f /dev/null"
+    command: "{{ bin_dir }}/kubectl run {{ item }} --image={{ test_image_repo }}:{{ test_image_tag }} --namespace test --command -- tail -f /dev/null"
     changed_when: false
     loop:
     - busybox1
@@ -52,8 +53,8 @@
   - import_role:
       name: cluster-dump
 
-  - name: Check that all pods are running and ready  # noqa 305
+  - name: Check that all pods are running and ready
-    shell: "{{ bin_dir }}/kubectl get pods --namespace test --no-headers -o yaml"
+    command: "{{ bin_dir }}/kubectl get pods --namespace test --no-headers -o yaml"
     changed_when: false
     register: run_pods_log
     until:
@@ -66,8 +67,8 @@
     failed_when: false
     no_log: true
 
-  - name: Get pod names  # noqa 305
+  - name: Get pod names
-    shell: "{{ bin_dir }}/kubectl get pods -n test -o json"
+    command: "{{ bin_dir }}/kubectl get pods -n test -o json"
     changed_when: false
     register: pods
     no_log: true
@@ -90,8 +91,8 @@
     register: running_pods
     no_log: true
 
-  - name: Check kubectl output  # noqa 305
+  - name: Check kubectl output
-    shell: "{{ bin_dir }}/kubectl get pods --all-namespaces -owide"
+    command: "{{ bin_dir }}/kubectl get pods --all-namespaces -owide"
     changed_when: False
     register: get_pods
     no_log: true