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
[release-4.4] Bug 1843230: Read BMH status from annotation #70
[release-4.4] Bug 1843230: Read BMH status from annotation #70
Conversation
(cherry-picked from commit 301f313)
If we are copying the Status in from a baremetalhost.metal3.io/status annotation, return and requeue immediately after writing it rather than try to continue to do other things, which may result in a write conflict. Also, invert the output of the hostHasStatus() method, so that it no longer returns the exact opposite of what it says on the tin. (cherry picked from commit 7417d87)
@zaneb: This pull request references Bugzilla bug 1843230, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Hi @zaneb. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign @russellb |
/ok-to-test |
/test e2e-metal-ipi |
Signed-off-by: Doug Hellmann <dhellmann@redhat.com> (cherry picked from commit e15ccce)
/test e2e-metal-ipi |
This is going to need to include metal3-io#546 as well |
Signed-off-by: Doug Hellmann <dhellmann@redhat.com> (cherry picked from commit 7f3d1f8)
Add functions for marshalling and unmarshalling the data in the status annotation. These will be useful in test code. Signed-off-by: Doug Hellmann <dhellmann@redhat.com> (cherry picked from commit b6a368b)
If the status annotation is a partial version of the status struct it will not include the LastUpdated value. This results in an infinite loop, since hostHasStatus() only looks at the LastUpdated value to see if it is filled in. This change removes the loop condition by ensuring LastUpdated is set if the value from the annotation is empty. Signed-off-by: Doug Hellmann <dhellmann@redhat.com> (cherry picked from commit 7ef20ab)
/bugzilla refresh |
@dhellmann: This pull request references Bugzilla bug 1843230, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
The 4.5 bugzilla is still ON_QA, so it won't let us merge a backport to 4.4 until that is verified. |
/bugzilla refresh |
@dhellmann: This pull request references Bugzilla bug 1843230, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/bugzilla refresh |
@dhellmann: This pull request references Bugzilla bug 1843230, which is valid. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/lgtm |
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dhellmann, stbenjam, 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 |
/label cherry-pick-approved |
@honza: The label(s) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@zaneb: All pull requests linked via external trackers have merged: openshift/baremetal-operator#70. Bugzilla bug 1843230 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Backport of #68 to release-4.4, so so we can populate masters' IP information to make the node/machine link.