-
Notifications
You must be signed in to change notification settings - Fork 111
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
Add proposal for supporting external introspection #155
Conversation
2559a71
to
415501e
Compare
415501e
to
dd21880
Compare
/retitle Add proposal for supporting external introspection |
@dhellmann @andfasano updated based on your feedback, thanks! |
When inspect.metal3.io=disabled is specified as an annotation we skip inspection and return complete immediately from the Inspecting state. Partially-Implements: metal3-io/metal3-docs#155
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
89ba787
to
86ca369
Compare
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
Add a proposal for an API to optionally disable introspection and update hardware status from some external inspection data if such data exists.
86ca369
to
03a5158
Compare
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
I'm happy with this draft. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dhellmann, hardys 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 |
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
Adds documentation for these interfaces, more information can be found in metal3-io/metal3-docs#155
When inspect.metal3.io=disabled is specified as an annotation we skip inspection and return complete immediately from the Inspecting state. Partially-Implements: metal3-io/metal3-docs#155
Adds documentation for these interfaces, more information can be found in metal3-io/metal3-docs#155
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
Adds documentation for these interfaces, more information can be found in metal3-io/metal3-docs#155
As described in metal3-io/metal3-docs#155 This enables addition of only the hardware part of the status, unlike the status annotation which can only be set on the first reconcile, since it allows all fields of the status to be modified.
Adds documentation for these interfaces, more information can be found in metal3-io/metal3-docs#155
This was discussed in the community meeting today, we agreed to give a week for lazy-consensus and merge if there are no objections. |
Ok the lazy consensus period completed and there were no objections - @andfasano perhaps you can add the lgtm please? |
/lgtm |
@andfasano: adding LGTM is restricted to approvers and reviewers in OWNERS files. 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. |
@dhellmann @maelk perhaps you could merge this? Seems we have a small list of approvers in the OWNERS, we should look at expanding that. |
/lgtm |
Add a proposal for an API to optionally disable introspection - this is related to the LiveImage proposal, where we may wish to skip inspection either to boot an appliance image faster, or because the image booted does it's own inspection.