Use dicts to configure the tests

Add some polarion IDs

Author: elfiesmelfie

ci/logging_tests_computes.yml

@@ -7,10 +7,10 @@
     KUBECONFIG: "{{ cifmw_openshift_kubeconfig }}"
     PATH: "{{ cifmw_path }}"
   vars:
-    journal_test_id: "RHOSO-12681"
-    journal_list:
-      - ceilometer_agent_compute
-      - nova_compute
+    #telemetry_logging_journal_test_id: "RHOSO-12681"
+    telemetry_logging_journal_list:
+      - { name: ceilometer_agent_compute, test_id: "RHOSO-12681" }
+      - { name: nova_compute, test_id: "RHOSO-12681" }
   tasks:
     - name: "Run Telemetry Logging tests"
       ansible.builtin.include_role:

ci/logging_tests_controller.yml

@@ -8,43 +8,44 @@
     PATH: "{{ cifmw_path }}"
   vars:
     common_pod_test_id: "RHOS0-12672"
-    common_pod_status_str: "Running"
-    common_pod_nspace: openstack-operators
+    # common_pod_status_str: "Running"
+    # common_pod_nspace: openstack-operators
     common_pod_list:
-      - telemetry-operator-controller-manager
-      - openstack-operator-controller-manager
+      - { name: telemetry-operator-controller-manager, nspace: openstack-operators, status_str: "Running", test_id: "{{ common_pod_test_id }}" }
+      - { name: openstack-operator-controller-manager, nspace: openstack-operators, status_str: "Running", test_id: "{{ common_pod_test_id }}" }
     common_subscription_test_id: "RHOSO-12678"
-    common_subscription_nspace: openshift-operators-redhat
+    # common_subscription_nspace: openshift-operators-redhat
     common_subscription_list:
-      - loki-operator
+      - { name: loki-operator, nspace: openshift-operators-redhat, test_id: "{{ common_subscription_test_id }}"}
     common_project_test_id: "RHOSO-12668"
     common_project_list:
-      - openshift-openstack-infra
-      - openshift
-      - openstack-operators
-      - openshift-logging
+      - { name: openshift-openstack-infra, test_id: "{{ common_project_test_id }}" }
+      - { name: openshift, test_id: "{{ common_project_test_id }}"}
+      - { name: openstack-operators, test_id: "{{ common_project_test_id }}" }
+      - { name: openshift-logging, test_id: "{{ common_project_test_id }}"}
     common_service_test_id: "RHOSO-12675"
-    common_service_nspace: openshift-logging
+    # common_service_nspace: openshift-logging
     common_service_list:
-      - cluster-logging-operator-metrics
-      - logging-loki-compactor-grpc
-      - logging-loki-compactor-http
-      - logging-loki-distributor-grpc
-      - logging-loki-distributor-http
-      - logging-loki-gateway-http
-      - logging-loki-gossip-ring
-      - logging-loki-index-gateway-grpc
-      - logging-loki-index-gateway-http
-      - logging-loki-ingester-grpc
-      - logging-loki-ingester-http
-      - logging-loki-querier-grpc
-      - logging-loki-querier-http
-      - logging-loki-query-frontend-grpc
-      - logging-loki-query-frontend-http
-      - openstack-logging
+      - { name: cluster-logging-operator-metrics, nspace: openshift-logging,test_id: "RHOSO-13744"}
+      - { name: logging-loki-compactor-grpc, nspace: openshift-logging, test_id: "RHOSO-13746"}
+      - { name: logging-loki-compactor-http, nspace: openshift-logging, test_id: "RHOSO-13747"}
+      - { name: logging-loki-distributor-grpc, nspace: openshift-logging, test_id: "RHOSO-13748"}
+      - { name: logging-loki-distributor-http, nspace: openshift-logging, test_id: "RHOSO-13749"}
+      - { name: logging-loki-gateway-http, nspace: openshift-logging, test_id: "RHOSO-13750"}
+      - { name: logging-loki-gossip-ring, nspace: openshift-logging, test_id: "RHOSO-13751"}
+      - { name: logging-loki-index-gateway-grpc, nspace: openshift-logging, test_id: "RHOSO-13752"}
+      - { name: logging-loki-index-gateway-http, nspace: openshift-logging, test_id: "RHOSO-13753"}
+      - { name: logging-loki-ingester-grpc, nspace: openshift-logging, test_id: "RHOSO-13754"}
+      - { name: logging-loki-ingester-http, nspace: openshift-logging, test_id: "RHOSO-13755"}
+      - { name: logging-loki-querier-grpc, nspace: openshift-logging, test_id: "RHOSO-13756"}
+      - { name: logging-loki-querier-http, nspace: openshift-logging, test_id: "RHOSO-13757"}
+      - { name: logging-loki-query-frontend-grpc, nspace: openshift-logging, test_id: "RHOSO-13758"}
+      - { name: logging-loki-query-frontend-http, nspace: openshift-logging, test_id: "RHOSO-13759"}
+      - { name: openstack-logging, nspace: openshift-logging, test_id: "RHOSO-13760"}
+
     common_file_test_id: "RHOSO-12754"
     common_file_list:
-      - /etc/rsyslog.d/10-telemetry.conf
+      - { name: "/etc/rsyslog.d/10-telemetry.conf", test_id: "{{ common_file_test_id }}"}
   tasks:
     - name: "Verify logging infrastructure components"
       ansible.builtin.include_role:
@@ -59,10 +60,10 @@
     PATH: "{{ cifmw_path }}"
   vars:
     common_pod_test_id: "RHOSO-12752"
-    common_pod_status_str: "Running"
-    common_pod_nspace: openstack
+    # common_pod_status_str: "Running"
+    # common_pod_nspace: openstack
     common_pod_list:
-      - openstackclient
+      - { name: openstackclient, status_str: "Running", nspace: "openstack", test_id: "{{ common_pod_test_id }}"} 
   tasks:
     - name: "Verify Running Pods"
       ansible.builtin.include_role:
@@ -78,10 +79,10 @@
     PATH: "{{ cifmw_path }}"
   vars:
     common_pod_test_id: "RHOSO-12673"
-    common_pod_status_str: "Running"
-    common_pod_nspace: openshift-operators-redhat
+    #common_pod_status_str: "Running"
+    #common_pod_nspace: openshift-operators-redhat
     common_pod_list:
-      - loki-operator-controller-manager
+      - { name: loki-operator-controller-manager, nspace: openshift-operators-redhat, status_str: "Running", test_id: "{{ common_pod_test_id }}"}
   tasks:
     - name: "Verify Pods running"
       ansible.builtin.include_role:
@@ -95,19 +96,20 @@
     KUBECONFIG: "{{ cifmw_openshift_kubeconfig }}"
     PATH: "{{ cifmw_path }}"
   vars:
-    common_pod_test_id: "RHOSO-12676"
-    common_pod_status_str: "Running"
-    common_pod_nspace: openshift-logging
+    # common_pod_test_id: "RHOSO-12676"
+    # common_pod_status_str: "Running"
+    # common_pod_nspace: openshift-logging
     common_pod_list:
-      - cluster-logging-operator
+       - cluster-logging-operator
       - logging-loki-compactor
       - logging-loki-distributor
       #- logging-loki-gateway
       - logging-loki-index-gateway
       - logging-loki-ingester
       - logging-loki-querier
       - logging-loki-query-frontend
-      - collector
+      - { name: collector, nspace: openshift-logging, status_str: "Running", test_id: "RHOSO-13745"}
+
 
   ### see JIRA LOG-5431 if pods not running
   tasks:
@@ -124,10 +126,10 @@
     PATH: "{{ cifmw_path }}"
   vars:
     common_pod_test_id: "RHOSO-12674"
-    common_pod_status_str: "Running"
-    common_pod_nspace: minio-dev
+    # common_pod_status_str: "Running"
+    # common_pod_nspace: minio-dev
     common_pod_list:
-      - minio
+      - { name: minio, nspace: minio-dev, status_str: "Running", test_id: "{{ common_pod_test_id }}" }
   tasks:
     - name: "Run pod running tests"
       ansible.builtin.include_role:
@@ -143,10 +145,10 @@
     PATH: "{{ cifmw_path }}"
   vars:
     common_pod_test_id: "RHOSO-12679"
-    common_pod_nspace: openstack
-    common_pod_status_str: "Completed"
+    #common_pod_nspace: openstack
+    #common_pod_status_str: "Completed"
     common_pod_list:
-      - logging-edpm-deployment-openstack-edpm-ipam
+      - { name: logging-edpm-deployment-openstack-edpm-ipam, nspace: openstack, status_str: "Completed", test_id: "{{ common_pod_test_id }}"}
   tasks:
     - name: "Run pods completed tests"
       ansible.builtin.include_role:
@@ -160,12 +162,12 @@
     KUBECONFIG: "{{ cifmw_openshift_kubeconfig }}"
     PATH: "{{ cifmw_path }}"
   vars:
-    common_crd_test_id: "RHOSO-12670"
+    #common_crd_test_id: "RHOSO-12670"
     common_crd_list:
-      - alertingrules.loki.grafana.com
-      - lokistacks.loki.grafana.com
-      - recordingrules.loki.grafana.com
-      - rulerconfigs.loki.grafana.com
+      - { name: alertingrules.loki.grafana.com, test_id: "RHOSO-12670"}
+      - { name: lokistacks.loki.grafana.com, test_id: "RHOSO-12670"}
+      - { name: recordingrules.loki.grafana.com, test_id: "RHOSO-12670"}
+      - { name: rulerconfigs.loki.grafana.com, test_id: "RHOSO-12670"}
   tasks:
     - name: "Run the crds tests"
       ansible.builtin.include_role:

roles/common/tasks/container_test.yml

@@ -2,15 +2,15 @@
 - name: Get container status
   ansible.builtin.shell:
     cmd: |
-      podman ps -a --format "{{ '{{.Names}} {{.Status}}' }}" | grep {{ container_name }}
+      podman ps -a --format "{{ '{{.Names}} {{.Status}}' }}" | grep {{ container.name }}
   changed_when: false
   register: container_status
 
 - name: |
-    Verify {{ container_name }} container status
-    {{ common_container_test_id }}
+    Verify {{ container.name }} container status
+    {{ container.test_id if container.test_id is defined }}
   ansible.builtin.assert:
     that:
       - "'unhealthy' not in container_status.stdout"
-    success_msg: "Container '{{ container_name }}' is in 'healthy' status."
-    fail_msg: "Container '{{ container_name }}' is not in 'healthy' status. Current status: {{ container_status.stdout }}"
+    success_msg: "Container '{{ container.name }}' is in 'healthy' status."
+    fail_msg: "Container '{{ container.name }}' is not in 'healthy' status. Current status: {{ container_status.stdout }}"

roles/common/tasks/cr_tests.yml

@@ -1,25 +1,25 @@
 ---
 - name: |
-    TEST Verify that the {{ item.kind }} {{ item.name }} CR exists
+    TEST Verify that the {{ cr.kind }} {{ cr.name }} CR exists
     {{ common_cr_test_id }}
   ansible.builtin.command:
     cmd: |
-      oc get {{ item.kind }} {{ item.name }}
+      oc get {{ cr.kind }} {{ cr.name }}
   register: result
   changed_when: false
   failed_when:
     - result.rc != 0
 
 - name: |
-    TEST Verify that the {{ item.kind }} {{ item.name }} CR is {{ item.condition_type }}
+    TEST Verify that the {{ cr.kind }} {{ cr.name }} CR is {{ cr.condition_type }}
     {{ common_cr_ready_test_id }}
   ansible.builtin.command:
     cmd: |
-      oc get {{ item.kind }} {{ item.name }} -o jsonpath='{.status.conditions[?(@.type=="{{ item.condition_type }}")].status}{"\n"}'
+      oc get {{ cr.kind }} {{ cr.name }} -o jsonpath='{.status.conditions[?(@.type=="{{ cr.condition_type }}")].status}{"\n"}'
   register: result
   changed_when: false
   failed_when:
     - result.stdout != "True"
   when:
     - common_cr_ready_test_id is defined
-    - item.condition_type is defined
+    - cr.condition_type is defined

roles/common/tasks/crd_tests.yml

@@ -1,16 +1,16 @@
 ---
-- name: Verify CRD "{{ item }}"
+- name: Verify CRD "{{ crd.name }}"
   ansible.builtin.shell:
     cmd: |
-      oc get crd "{{ item }}"
+      oc get crd "{{ crd.name }}"
   changed_when: false
   register: output
 
 - name: |
-    TEST Verify CRD {{ item }} is found
-    {{ common_crd_test_id }}
+    TEST Verify CRD {{ crd.name }} is found
+    {{ crd.test_id if crd.test_id is defined }}
   ansible.builtin.assert:
     that:
       - "'NotFound' not in output.stderr"
-    success_msg: "CRD {{ item }} is found."
-    fail_msg: "CRD {{ item }} not found. Error: {{ output.stderr }}"
+    success_msg: "CRD {{ crd.name }} is found."
+    fail_msg: "CRD {{ crd.name }} not found. Error: {{ output.stderr }}"

roles/common/tasks/endpoint_tests.yml

@@ -1,16 +1,16 @@
 ---
-- name: Verify endpoint "{{ item }}"
+- name: Verify endpoint "{{ endpoint }}"
   ansible.builtin.shell:
     cmd: |
-      {{ openstack_cmd }} endpoint list --service {{ item[0] }} --interface {{ item[2] }}
+      {{ openstack_cmd }} endpoint list --service {{ endpoint[0] }} --interface {{ endpoint[2] }}
   changed_when: false
   register: output
 
 - name: |
-    TEST Check that {{ item[0] }} endpoint exists
+    TEST Check that {{ endpoint[0] }} endpoint exists
     {{ common_endpoint_test_id }}
   ansible.builtin.assert:
     that:
       - output.stdout != ""
-    success_msg: "The endpoint {{ item[0] }} is found."
-    fail_msg: "The endpoint for service {{ item[0] }} "
+    success_msg: "The endpoint {{ endpoint[0] }} is found."
+    fail_msg: "The endpoint for service {{ endpoint[0] }} "

roles/common/tasks/file_tests.yml

@@ -1,10 +1,10 @@
 ---
-- name: Get stats for file - "{{ item }}"
+- name: Get stats for file - "{{ file.name }}"
   ansible.builtin.stat:
-    path: "{{ item }}"
+    path: "{{ ifile.name }}"
   register: fstats
   failed_when: 
     - fstats.stat.pw_name != "root"
     - fstats.stat.size | int < 300
     - not fstats.stat.exists
-    - not fstats.stat.isreg
+    - not fstats.stat.isreg

roles/common/tasks/main.yml

@@ -1,74 +1,80 @@
 ---
 - name: "Verify pod - {{ common_pod_test_id }}"
   when:
-    - common_pod_test_id is defined
     - common_pod_list is defined
-    - common_pod_nspace is defined
-    - common_pod_status_str is defined
   ansible.builtin.include_tasks: "pod_tests.yml"
   loop: "{{ common_pod_list }}"
+  loop_control:
+    loop_var: pod
 
 - name: "Run subscription tests"
   when:
-    - common_subscription_nspace is defined
     - common_subscription_list is defined
   ansible.builtin.include_tasks: "subscription_tests.yml"
   loop: "{{ common_subscription_list }}"
+  loop_control:
+    loop_var: subscription
 
 - name: "Run endpoint tests"
   when:
     - common_endpoint_list is defined
   ansible.builtin.include_tasks: "endpoint_tests.yml"
   loop: "{{ common_endpoint_list }}"
+  loop_control:
+    loop_var: endpoint
 
 - name: "Run project tests"
   when:
-    - common_project_test_id is defined
     - common_project_list is defined
   ansible.builtin.include_tasks: "project_tests.yml"
   loop: "{{ common_project_list }}"
+  loop_control:
+    loop_var: project
 
 - name: "Run service tests"
   when:
-    - common_service_test_id is defined
-    - common_service_nspace is defined
     - common_service_list is defined
   ansible.builtin.include_tasks: "service_tests.yml"
   loop: "{{ common_service_list }}"
+  loop_control:
+    loop_var: service
 
 - name: "Run file tests"
   when:
-    - common_file_test_id is defined
     - common_file_list is defined
   ansible.builtin.include_tasks: "file_tests.yml"
   loop: "{{ common_file_list }}"
-  
+  loop_control:
+    loop_var: file
+
 - name: "Run manifest tests"
   when:
-    - common_manifest_test_id is defined
     - common_manifest_list is defined
   ansible.builtin.include_tasks: "manifest_tests.yml"
   loop: "{{ common_manifest_list }}"
+  loop_control:
+    loop_var: manifest
 
 - name: "Run crd tests"
   when:
-    - common_crd_test_id is defined
     - common_crd_list is defined
   ansible.builtin.include_tasks: "crd_tests.yml"
   loop: "{{ common_crd_list }}"
+  loop_control:
+    loop_var: crd
 
 - name: "Run CR tests"
   when:
-    - common_cr_test_id is defined
     - common_cr_list is defined
   ansible.builtin.include_tasks: "cr_tests.yml"
   loop: "{{ common_cr_list }}"
+  loop_control:
+    loop_var: cr
 
 - name: "Verify container tests"
   when:
-    - common_container_list is defined
     - common_container_test_id is defined
   ansible.builtin.include_tasks: "container_test.yml"
   loop: "{{ common_container_list }}"
   loop_control:
-    loop_var: container_name
+    loop_var: container