-
Notifications
You must be signed in to change notification settings - Fork 560
Fixing Baremetal CI #3449
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
base: main
Are you sure you want to change the base?
Fixing Baremetal CI #3449
Conversation
@stephenfin - I think this code is really old. I could totally search through now and find exactly where noauth/httpbasic was broken, but since v1 is functional and correctly uses auth (, I think this is okay to merge? (When master is fixed?) |
Sorry, I don't understand the ask here. The CI is still failing: we hardly want to merge this as-is? |
Sorry, this PR currently has a typo... I'm going to fix it today and it'll pass. |
I only figured out the master issue with blind luck on a different change. My suspicion as to what is happening with the tests is they were designed before the devstack action was created (The README suggests this was the case in the acceptance test folder?), and these configurations are designed to be skipped. I mean... we can totally spin up 3 ironics but that seems insane. (I think that is what would be needed to stop the skips?) |
I don't think we can deploy TBH, I suspect the issue here is the Ironic's devstack plugin. We must be going off the beaten track wrt what Ironic does in their CI. My guess is that it's something to do with the networking configuration (perhaps use of OVN in one place and OVS in another). Would you be able to compare the |
I can get on that - thank you! |
Thanks. Just a case in point RE: Focal vs Noble: https://review.opendev.org/c/openstack/devstack/+/952839 |
Fixes #3343
Continuation from #3383.
Somepoint between May 14th and July 2nd, something changed where baremetal tests started to (mostly) work.