-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use service selector for openstackclient pod #186
Use service selector for openstackclient pod #186
Conversation
Recently the openstackclient pod moved to using app to service as a selector, this change updates the check in the tests to use the right label.
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/b490188b59b046d18da802e506004601 ❌ data-plane-adoption-github-rdo-centos-9-crc-single-node FAILURE in 1h 23m 38s |
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/b25fc4093bec42c7baf8c65da38ae819 ❌ data-plane-adoption-github-rdo-centos-9-crc-single-node FAILURE in 1h 19m 40s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
but waiting for merge on https://review.rdoproject.org/r/c/rdo-jobs/+/50667
and recheck here
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/948411c5c039439f913e9e918e579a20 ❌ data-plane-adoption-github-rdo-centos-9-crc-single-node FAILURE in 1h 20m 15s |
Merging as at least we have one working job to gate PRs with this fix, investigating now the failure reason for the single-node job |
the new extracted job passes here (the older job fails on placement for cert issue which is cix'd and almost fixed OSPCIX-104 |
Recently the openstackclient pod moved to using app to service as
a selector, this change updates the check in the tests to use the right
label.
It appears this change in openstack-operator was the trigger:
openstack-k8s-operators/openstack-operator#502