Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Added support for baremetal portgroups #3290

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

Merged
merged 4 commits into from
Mar 5, 2025

Conversation

Sharpz7
Copy link
Contributor

@Sharpz7 Sharpz7 commented Feb 4, 2025

Fixes #3285

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

Built from https://docs.openstack.org/api-ref/baremetal/#portgroups-portgroups (For Ironic this is considered reliable)

@github-actions github-actions bot added edit:baremetal This PR updates baremetal code semver:minor Backwards-compatible change labels Feb 4, 2025
@coveralls
Copy link

coveralls commented Feb 4, 2025

Coverage Status

coverage: 78.74% (+0.01%) from 78.729%
when pulling a475a62 on Sharpz7:baremetal-portgroups
into e7b58a9 on gophercloud:main.

Copy link

@jayofdoom jayofdoom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 seems like a faithful implementation of portgroups

@github-actions github-actions bot added semver:major Breaking change and removed semver:minor Backwards-compatible change labels Feb 5, 2025
@EmilienM EmilienM merged commit c860416 into gophercloud:main Mar 5, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
edit:baremetal This PR updates baremetal code semver:major Breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Portgroups from Baremetal API
4 participants