-
Notifications
You must be signed in to change notification settings - Fork 10
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
feature_tests: add openstack-network-exporter feature tests #231
base: master
Are you sure you want to change the base?
feature_tests: add openstack-network-exporter feature tests #231
Conversation
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2ac2a112dd4c4a398ad35959d1738f64 ✔️ feature-verification-tests-noop SUCCESS in 4s |
866ec4f
to
75a9bac
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/457b6c239a9546f8a1d667c01836d0ec ✔️ feature-verification-tests-noop SUCCESS in 4s |
I think the reason this isn't passing is that the openstack-operater and telemetry-operator are out-of-sync after the PR that removed the prom external route. This results in the prom metrics not being visable. This should be resolved when openstack-k8s-operators/openstack-operator#1326 is merged |
recheck |
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.
Looks fine to me, but it must pass CI and have the dependancy (openstack-k8s-operators/openstack-operator#1326) merged first.
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e6928f8982d047888cd353db02cd0913 ✔️ feature-verification-tests-noop SUCCESS in 5s |
recheck |
The dependancy has merged and the previous failure appears arbitrary. |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f88a874861a24c1cba20e923ebc7dbb5 ✔️ feature-verification-tests-noop SUCCESS in 4s |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d099ea630693454ebe89fdcfe13923ab Warning: |
Either this PR should not depend on the T-O one or vice-versa Edit: I think that this PR should depend on the telemetry operator PR. I'll update the PR descriptions accordingly and recheck |
recheck |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/73d4fb66c4f14a47b6b82d2066655580 ✔️ feature-verification-tests-noop SUCCESS in 5s |
@atyronesmith I believe you've hit this issue: https://issues.redhat.com/browse/OSPRH-14391 |
Once the following PR merges, it should fix the CI issue you're hitting: #232 |
Add tests for operator of openstack-network-exporter Signed-off-by: Aaron T. Smith <[email protected]>
75a9bac
to
3f5a76b
Compare
I expect that this PR will pass CI now that the blocker was resolved. |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/7f70be49ac454d329af058fa3dffea17 ✔️ feature-verification-tests-noop SUCCESS in 5s |
Add tests for operator of openstack-network-exporter
Depends-On: openstack-k8s-operators/openstack-operator#1326
Depends-On: openstack-k8s-operators/telemetry-operator#627