Skip to content

Commit 3e94faf

Browse files
committed
Use ignore_errors in plays for test_logging
The logging plugins we use need all the tests to run in order to report properly. Adding ignore_errors to the plays that run the tests allows all the tests to run even when one fails. Also added verbosity to the ansible config to get more info about the failing tests
1 parent a7796db commit 3e94faf

File tree

3 files changed

+15
-17
lines changed

3 files changed

+15
-17
lines changed

ci/ansible.cfg

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,6 @@ callbacks_enabled = custom_logger
33
callback_plugins = ../callback_plugins
44
# additional paths to search for roles
55
roles_path = ../roles:/usr/share/ansible/roles:/etc/ansible/roles:~/.ansible/roles:../../ci-framework/roles
6+
7+
# temp update to get more information from the playbooks running functional tests
8+
verbosity = 5

ci/logging_tests_computes.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
---
12
- name: Prepare Logging Tests
23
hosts: computes
34
gather_facts: false
@@ -9,6 +10,7 @@
910
- name: "Verify ctlplane logging containers and files"
1011
hosts: computes
1112
gather_facts: no
13+
ignore_errors: true
1214
vars:
1315
container_test_id: "RHOSO-12753"
1416
container_list:
@@ -19,7 +21,6 @@
1921
file_test_id: "RHOSO-12754"
2022
file_list:
2123
- /etc/rsyslog.d/10-telemetry.conf
22-
2324
tasks:
2425
- name: "Run file and container tests"
2526
ansible.builtin.import_role:
@@ -29,12 +30,12 @@
2930
- name: "Verify logging journalctl identifiers"
3031
hosts: computes
3132
gather_facts: no
33+
ignore_errors: true
3234
vars:
3335
identifiers_test_id: "RHOSO-12681"
3436
identifiers_list:
3537
- ceilometer_agent_compute
3638
- nova_compute
37-
3839
tasks:
3940
- name: "Verify journalctl logging identifiers"
4041
ansible.builtin.import_role:

ci/logging_tests_local.yml

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
1+
---
12
- name: Prepare Logging Tests
23
hosts: controller
34
gather_facts: false
5+
ignore_errors: true
46
vars:
57
id_rsa_path: "{{ playbook_dir }}/id_rsa.ctlplane"
68
environment:
@@ -32,15 +34,14 @@
3234
- name: "Verify logging projects, endpoints, credentials, nodes, pods, services, manifests and subscriptions"
3335
hosts: controller
3436
gather_facts: no
37+
ignore_errors: true
3538
vars:
3639
proj_test_id: "RHOSO-12668"
3740
proj_list:
3841
- openshift-openstack-infra
3942
- openshift
4043
- openstack-operators
4144
- openshift-logging
42-
43-
4445
endpoint_test_id: "RHOSO-12682"
4546
endpoint_list:
4647
- [nova,compute,public]
@@ -60,30 +61,26 @@
6061
- [neutron,network,public]
6162
- [neutron,network,internal]
6263

63-
6464
cred_test_id: "RHOSO-12670"
6565
cred_list:
6666
- alertingrules.loki.grafana.com
6767
- lokistacks.loki.grafana.com
6868
- recordingrules.loki.grafana.com
6969
- rulerconfigs.loki.grafana.com
7070

71-
7271
node_test_id: "RHOSO-12671"
7372
node_list:
7473
- edpm-compute-0
7574
- edpm-compute-1
7675
- crc
7776

78-
7977
pod_test_id: "RHOS0-12672"
8078
pod_status_str: "Running"
8179
pod_nspace: openstack-operators
8280
pod_list:
8381
- telemetry-operator-controller-manager
8482
- dataplane-operator-controller-manager
8583

86-
8784
service_test_id: "RHOSO-12675"
8885
service_nspace: openshift-logging
8986
service_list:
@@ -104,36 +101,30 @@
104101
- logging-loki-query-frontend-http
105102
- logging-view-plugin
106103
- openstack-logging
107-
108-
109104
manifest_test_id: "RHOSO-12677"
110105
manifest_list:
111106
- "loki-operator 2"
112107
- "loki-helm-operator 1"
113-
114-
115108
subscription_test_id: "RHOSO-12678"
116109
subscription_nspace: openshift-operators-redhat
117110
subscription_list:
118111
- loki-operator
119-
120112
tasks:
121113
- name: "Run logging project, endpoint, credential, node, pod, service, manifest and subscription tests"
122114
ansible.builtin.import_role:
123115
name: common
124116

125117

126-
127118
- name: "Verify logging pods are running in openstack"
128119
hosts: controller
129120
gather_facts: no
121+
ignore_errors: true
130122
vars:
131123
pod_test_id: "RHOSO-12752"
132124
pod_status_str: "Running"
133125
pod_nspace: openstack
134126
pod_list:
135127
- openstackclient
136-
137128
tasks:
138129
- name: "Verify Running Pods"
139130
ansible.builtin.import_role:
@@ -143,6 +134,7 @@
143134
- name: "Verify logging pods are running in openshift-operators-redhat"
144135
hosts: controller
145136
gather_facts: no
137+
ignore_errors: true
146138
vars:
147139
pod_test_id: "RHOSO-12673"
148140
pod_status_str: "Running"
@@ -159,6 +151,7 @@
159151
- name: "Verify logging pods are running in openshift-logging"
160152
hosts: controller
161153
gather_facts: no
154+
ignore_errors: true
162155
vars:
163156
pod_test_id: "RHOSO-12676"
164157
pod_status_str: "Running"
@@ -185,6 +178,7 @@
185178
- name: "Verify logging pods are running in minio-dev"
186179
hosts: controller
187180
gather_facts: no
181+
ignore_errors: true
188182
vars:
189183
pod_test_id: "RHOSO-12674"
190184
pod_status_str: "Running"
@@ -201,6 +195,7 @@
201195
- name: "Verify logging pods have complete status in openstack"
202196
hosts: controller
203197
gather_facts: no
198+
ignore_errors: true
204199
vars:
205200
pod_test_id: "RHOSO-12679"
206201
pod_nspace: openstack
@@ -223,7 +218,6 @@
223218
- ssh-known-hosts-edpm-deployment-logging
224219
- telemetry-edpm-deployment-logging
225220
- validate-network-edpm-deployment-logging
226-
227221
tasks:
228222
- name: "Run pods completed tests"
229223
ansible.builtin.import_role:
@@ -233,6 +227,7 @@
233227
- name: "Verify logging services are running in openstack"
234228
hosts: controller
235229
gather_facts: no
230+
ignore_errors: true
236231
vars:
237232
service_test_id: "RHOSO-12749"
238233
service_nspace: openstack
@@ -241,7 +236,6 @@
241236
- nova-metadata-internal
242237
- nova-novncproxy-cell1-public
243238
- nova-public
244-
245239
tasks:
246240
- name: "Run Services tests"
247241
ansible.builtin.import_role:

0 commit comments

Comments
 (0)