-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
OSDOCS-4872: change variant rhcos to openshift #58302
Conversation
@jldohmann: This pull request references OSDOCS-4872 which is a valid jira issue. 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. |
🤖 Updated build preview is available at: Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/14173 |
@mike-nguyen could you PTAL for QE ack? this updates the SNO docs to reflect the changes in Butane support. it doesn't look like these docs existed pre-4.12, so i've marked them as 4.12+ |
I'm not too familiar with single node mode so I can't comment on the dropping of root ssh login. If SNO is more like OCP now with only allowing core user login then the changes LGTM. Do you know if SNO has QE that can look over these changes? |
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.
Looking pretty good. One question for you, as well as two suggested changes for while you're working on the file.
@yunjiang29 could you PTAL for QE ack, or do you know who could take a look for QE ack? this PR updates the SNO docs to reflect the changes in Butane support. thank you! |
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
@jldohmann: This pull request references OSDOCS-4872 which is a valid jira issue. 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. |
New changes are detected. LGTM label has been removed. |
I am leaving the hardcoded 4.12 version in the |
/cherrypick enterprise-4.13 |
/cherrypick enterprise-4.12 |
@jldohmann: new pull request created: #58847 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. |
@jldohmann: #58302 failed to apply on top of branch "enterprise-4.12":
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. |
Version(s): 4.12+
Issue: https://issues.redhat.com/browse/OSDOCS-4872
Link to docs preview: http://file.rdu.redhat.com/jdohmann/OSDOCS-4872/installing/installing_sno/install-sno-installing-sno.html#create-custom-live-rhcos-iso_install-sno-installing-sno-with-the-assisted-installer
QE review:
Additional information: