-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
OCPBUGS#7608: iPXE for ZTP not default behaviour #56323
Conversation
🤖 Updated build preview is available at: Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/11074 |
/lgtm |
/label peer-review-needed |
/label peer-review-in-progress |
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.
Just one tiny comment. LGTM!
modules/ztp-installation-crs.adoc
Outdated
@@ -22,7 +22,7 @@ The following table lists the installation CRs that are automatically applied by | |||
|
|||
|`BareMetalHost` | |||
|Contains the connection information for the Baseboard Management Controller (BMC) of the target bare-metal host. | |||
|Provides access to the BMC to load and boot the discovery image on the target server by using the Redfish protocol. ZTP supports iPXE and virtual media network booting. | |||
|Provides access to the BMC to load and boot the discovery image on the target server by using the Redfish protocol. |
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.
|Provides access to the BMC to load and boot the discovery image on the target server by using the Redfish protocol. | |
|Provides access to the BMC to load and start the discovery image on the target server by using the Redfish protocol. |
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.
The IBM Style Guide advises that we use "start" or "turn on" instead of "boot" where possible.
/remove-label peer-review-in-progress |
/label peer-review-done |
8d5afcb
to
ff8f15f
Compare
New changes are detected. LGTM label has been removed. |
/label merge-review-needed |
/cherrypick enterprise-4.12 |
/cherrypick enterprise-4.13 |
@sjhala-ccs: new pull request created: #57146 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. |
@sjhala-ccs: new pull request created: #57147 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. |
OCPBUGS#7608: Removing two lines as iPXE network booting for ZTP is not the default behaviour currently.
Version(s):
4.12+
Issue:
https://issues.redhat.com/browse/OCPBUGS-7608
Link to docs preview:
(removed reference from callout 7 of step 4.)
QE review: