Skip to content

[test_operator] Support merging skiplist with exclude list#4024

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
kstrenkova:merge-test-operator-skip-list
Jul 2, 2026
Merged

[test_operator] Support merging skiplist with exclude list#4024
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
kstrenkova:merge-test-operator-skip-list

Conversation

@kstrenkova

Copy link
Copy Markdown
Contributor

Adding cifmw_test_operator_tempest_exclude_list_merge variable to enable combining list_skipped.yml with
cifmw_test_operator_tempest_exclude_list. When set to true, both sources are merged into a single exclude list, which helps to later remove workarounds that merged these lists manually.

Signed-off-by: Katarina Strenkova kstrenko@redhat.com

@kstrenkova kstrenkova force-pushed the merge-test-operator-skip-list branch from 20c9673 to 7c5945b Compare June 30, 2026 10:06
@kstrenkova kstrenkova requested review from evallesp and michburk June 30, 2026 12:14
@kstrenkova

Copy link
Copy Markdown
Contributor Author

I have tested this change locally (ran the test_operator role, once with the new parameter set to false and once set with true) and it seems to be working as expected :D

The workaround we are trying to remove is present in the ci-framework-jobs repository (search for TODO (anbanerj): Remove this task when tempest supports it), if that would help you understand the bigger picture a bit better.

evallesp
evallesp previously approved these changes Jun 30, 2026

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM in general!

Comment thread roles/test_operator/tasks/tempest-tests.yml
Adding cifmw_test_operator_tempest_exclude_list_merge variable to
enable combining list_skipped.yml with
cifmw_test_operator_tempest_exclude_list. When set to true, both
sources are merged into a single exclude list, which helps to
later remove workarounds that merged these lists manually.

Signed-off-by: Katarina Strenkova kstrenko@redhat.com
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/db2eeab378274974b22f51bad7094f70

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 30m 24s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 34m 23s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 37m 35s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 17m 01s
cifmw-multinode-tempest FAILURE in 32m 19s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 48s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 30s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 44s
✔️ cifmw-molecule-test_operator SUCCESS in 4m 14s

@kstrenkova

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/89d237b68cc84501a08e353264f078bb

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 17m 45s
podified-multinode-edpm-deployment-crc NODE_FAILURE Node(set) request 099-0000132383 failed in 0s
cifmw-crc-podified-edpm-baremetal NODE_FAILURE Node(set) request 099-0000132384 failed in 0s
cifmw-crc-podified-edpm-baremetal-minor-update NODE_FAILURE Node(set) request 099-0000132385 failed in 0s
✔️ cifmw-multinode-tempest SUCCESS in 1h 58m 04s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 54s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 05s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 41s
✔️ cifmw-molecule-test_operator SUCCESS in 4m 09s

@michburk

michburk commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

recheck

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@michburk michburk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@openshift-ci

openshift-ci Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michburk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved label Jul 2, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit f67ae9b into openstack-k8s-operators:main Jul 2, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants