Skip to content

Fixing Baremetal CI #3500

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixing Baremetal CI #3500

wants to merge 1 commit into from

Conversation

Sharpz7
Copy link
Contributor

@Sharpz7 Sharpz7 commented Sep 3, 2025

Fixes #3343

Continuation from #3383 and #3449.

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

image

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

Coverage Status

coverage: 63.81%. remained the same
when pulling 9d2ef73 on Sharpz7:baremetal-ci-fix
into 7c53a81 on gophercloud:main.

@Sharpz7
Copy link
Contributor Author

Sharpz7 commented Sep 3, 2025

We need to decide if IRONIC_BAREMETAL_BASIC_OPS=False is acceptable. Apparently, we don't do anything requiring it... the question is will that hold in the long term / not cost us problems later.

cc @jayofdoom

@jayofdoom
Copy link

IRONIC_BAREMETAL_BASIC_OPS=True sets up nested baremetal nodes. We shouldn't need to do that, and if we get to the point where we want to move things through states we should use fake-hardware IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport-v2 This PR will be backported to v2 edit:actions This PR updates GitHub Actions code edit:baremetal This PR updates baremetal code edit:testinfra This PR updates testing infrastructure code semver:patch No API change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

functional-baremetal tests being skipped.
3 participants
@Sharpz7 @coveralls @jayofdoom