Skip to content

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

Sharpz7
Copy link
Contributor

@Sharpz7 Sharpz7 commented Jul 3, 2025

Fixes #3343

Continuation from #3383.

Somepoint between May 14th and July 2nd, something changed where baremetal tests started to (mostly) work.

  • Master is broken, not sure why
  • Noauth + httpbasic tests all get skipped: My guess is there was a behavior change.

@github-actions github-actions bot added edit:baremetal This PR updates baremetal code semver:patch No API change backport-v2 This PR will be backported to v2 labels Jul 3, 2025
@coveralls
Copy link

coveralls commented Jul 3, 2025

Coverage Status

coverage: 63.725% (+0.02%) from 63.702%
when pulling 7c12090 on Sharpz7:baremetal-ci-fixes
into f98c2f9 on gophercloud:main.

@Sharpz7
Copy link
Contributor Author

Sharpz7 commented Jul 3, 2025

@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?)

@stephenfin
Copy link
Contributor

@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?

@Sharpz7
Copy link
Contributor Author

Sharpz7 commented Jul 8, 2025

@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.

@github-actions github-actions bot added semver:major Breaking change and removed semver:patch No API change backport-v2 This PR will be backported to v2 labels Jul 9, 2025
@github-actions github-actions bot added edit:actions This PR updates GitHub Actions code edit:testinfra This PR updates testing infrastructure code labels Jul 10, 2025
@Sharpz7
Copy link
Contributor Author

Sharpz7 commented Jul 10, 2025

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?)

@stephenfin
Copy link
Contributor

stephenfin commented Jul 10, 2025

I don't think we can deploy master on Ubuntu 22.04: it's not a supported runtime for 2025.2 (Flamingo). While it gets things passing right now, there's a good chance this will break at some point in the near future as DevStack removes/inadvertently breaks support for Focal.

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 local.conf we use versus those used in the upstream Ironic gates to see if there are any obvious differences? I will do so at some point myself if not, but it's just a case of finding the time.

@Sharpz7
Copy link
Contributor Author

Sharpz7 commented Jul 10, 2025

I can get on that - thank you!

@stephenfin
Copy link
Contributor

Thanks.

Just a case in point RE: Focal vs Noble: https://review.opendev.org/c/openstack/devstack/+/952839

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
edit:actions This PR updates GitHub Actions code edit:baremetal This PR updates baremetal code edit:testinfra This PR updates testing infrastructure code semver:major Breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

functional-baremetal tests being skipped.
3 participants