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
Merge upstream 2020-12-04 #114
Conversation
If an existing Ironic node, and a BMH share a MAC address, link them together.
This adds a new `manifest-lint` target in the Makefile to run manifest validation checks via ./hack/manifestlint.sh script.
Add a Makefile target for running manifest validation
Look up existing nodes by MAC address
It is not safe to assume in ValidateManagementAccess() that the ironic state will eventually converge to manageable, available, or active. The state may in fact be something (like e.g. adopt failed) that requires action on the part of the actual provisioning function for the current state to resolve. We previously weren't allowing that, because ValidateManagementAccess() was always forcing a requeue whenever it encountered a node in another state. Since b7b10d7 we call ValidateManagementAccess() in substantially every provisioning state, so there are even more node states (e.g. inspect failed) that we could get stuck in. Move the code for handling the verifying state so that it runs only in that state, and allow all other states from manageable on to be handled by the host state machine.
Test that ValidateManagementAccess() requeues when the node is in the Enroll or Validating states.
Avoid getting stuck in ValidateManagementAccess due to node state
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dhellmann, zaneb The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
No description provided.