Skip to content

Added node.Retired - #3136

Merged
EmilienM merged 2 commits into
gophercloud:masterfrom
Sharpz7:node-retired
Jul 23, 2024
Merged

EmilienM merged 2 commits into
gophercloud:masterfrom
Sharpz7:node-retired

Conversation

@Sharpz7

@Sharpz7 Sharpz7 commented Jul 19, 2024 •

Copy link
Copy Markdown
Contributor

Prior to starting a PR, please make sure you have read our
contributor tutorial.

Prior to a PR being reviewed, there needs to be a Github issue that the PR
addresses. Replace the brackets and text below with that issue number.

Fixes #3135

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

https://github.com/openstack/ironic/blob/1e52143f07266b53ad47d49a5be04cbf85cccc8b/ironic/objects/node.py#L178

@github-actions github-actions Bot added the semver:minor Backwards-compatible change label Jul 19, 2024

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you for submitting your first PR! Be sure that we will be looking at it but keep in mind
this sometimes takes a while.
Please let the maintainers know if your PR has not got enough attention after a few days.
If any doubt, please consult our PR tutorial.

Comment thread openstack/baremetal/v1/nodes/results.go
@Sharpz7
Sharpz7 requested a review from EmilienM July 19, 2024 14:51
@EmilienM EmilienM added the backport-v2 This PR will be backported to v2 label Jul 19, 2024
@EmilienM

Copy link
Copy Markdown
Contributor

@Sharpz7 unfortunately, the v1 branch doesn't take semver:minor kind of API changes, because it's in maintenance/bugfix only since very recently. We can make exceptions though, but we really encourage folks to migrate to v2 (we have a manual for that).

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 78.738% (+5.7%) from 73.053%
when pulling 505e8cd on Sharpz7:node-retired
into f626223 on gophercloud:master.

@jayofdoom

jayofdoom commented Jul 19, 2024 •

Copy link
Copy Markdown

Thanks for considering an exception @EmilienM -- please grant it this time. We'll follow up and connect with the openstack-prometheus-exporter community to see if we can help with a v2 migration.

@Sharpz7

Sharpz7 commented Jul 19, 2024

Copy link
Copy Markdown
Contributor Author

Thank you @EmilienM for your help :))

@jayofdoom

Copy link
Copy Markdown

FWIW; there seems to be some evidence that field adding has been allowed for v1 backporting in the past: 7d47899

@EmilienM
EmilienM merged commit 59c2aee into gophercloud:master Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v2 This PR will be backported to v2 semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding support for node.retired and node.retired_reason

4 participants