Skip to content

baremetal: add inspection_{started,finished}_at to Node#2545

Merged
mandre merged 1 commit intogophercloud:masterfrom
dtantsur:baremetal-inspection
Feb 3, 2023
Merged

baremetal: add inspection_{started,finished}_at to Node#2545
mandre merged 1 commit intogophercloud:masterfrom
dtantsur:baremetal-inspection

Conversation

@dtantsur
Copy link
Copy Markdown
Contributor

@dtantsur dtantsur commented Feb 2, 2023

Fixes #2544

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

https://docs.openstack.org/api-ref/baremetal/?expanded=show-node-details-detail#show-node-details

@dtantsur dtantsur force-pushed the baremetal-inspection branch from 14d2794 to a4db36a Compare February 2, 2023 15:34
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage: 80.092%. Remained the same when pulling a4db36a on dtantsur:baremetal-inspection into e0878b0 on gophercloud:master.

@mandre mandre added this to the v1.3.0 milestone Feb 2, 2023
Copy link
Copy Markdown
Contributor

@mandre mandre left a comment

Choose a reason for hiding this comment

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

LGTM, just waiting for the Victoria job to finish running before merging. The train job is currently broken, and we might just remove it as upstream has EOL-ed it and we don't see how we can fix it otherwise.

Copy link
Copy Markdown
Contributor

@mandre mandre left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.
On a related news, we've also dropped the permafailing train job in #2546.

@mandre mandre merged commit c48668a into gophercloud:master Feb 3, 2023
@dtantsur dtantsur deleted the baremetal-inspection branch February 3, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

baremetal: missing inspection_{started,finished}_at in Node

3 participants