Skip to content

Commit

Permalink
[ci/logging] Use include_role instead of import_role for using the co…
Browse files Browse the repository at this point in the history
…mmon role

The values for the loop items don't seem to render otherwise.
  • Loading branch information
elfiesmelfie committed Dec 11, 2024
1 parent 79b4f04 commit 903fd72
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion ci/logging_tests_computes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@
- nova_compute
tasks:
- name: "Run Telemetry Logging tests"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: telemetry_logging
14 changes: 7 additions & 7 deletions ci/logging_tests_controller.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
- openshift-logging
tasks:
- name: "Verify logging infrastructure components"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common

- name: "Verify logging pods are running in openstack"
Expand All @@ -43,7 +43,7 @@
- openstackclient
tasks:
- name: "Verify Running Pods"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common


Expand All @@ -62,7 +62,7 @@
- loki-operator-controller-manager
tasks:
- name: "Verify Pods running"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common

- name: "Verify logging pods are running in openshift-logging"
Expand Down Expand Up @@ -91,7 +91,7 @@
### see JIRA LOG-5431 if pods not running
tasks:
- name: "Verify Pods running"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common

- name: "Verify logging pods are running in minio-dev"
Expand All @@ -109,7 +109,7 @@
- minio
tasks:
- name: "Run pod running tests"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common


Expand All @@ -128,7 +128,7 @@
- logging-edpm-deployment-openstack-edpm-ipam
tasks:
- name: "Run pods completed tests"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common

- name: "Verify the crds exist"
Expand All @@ -147,5 +147,5 @@
- rulerconfigs.loki.grafana.com
tasks:
- name: "Run the crds tests"
ansible.builtin.import_role:
ansible.builtin.include_role:
name: common

0 comments on commit 903fd72

Please sign in to comment.