Skip to content

Conversation

@xenolinux
Copy link
Contributor

@xenolinux xenolinux commented Nov 13, 2025

Version(s): 4.20+

Issue:

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:
This PR should not be merged until HCP for OCP 4.20 is GA (tentative GA date: Dec 3)

@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 13, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Nov 13, 2025

🤖 Tue Nov 18 09:14:34 - Prow CI generated the docs preview:

https://102493--ocpdocs-pr.netlify.app/openshift-enterprise/latest/hosted_control_planes/hcp-machine-config.html

Copy link

@gaol gaol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have some minor comments, would you please check ?

@xenolinux xenolinux added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 14, 2025
@gaol
Copy link

gaol commented Nov 14, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 14, 2025
@xenolinux
Copy link
Contributor Author

/test validate-asciidoc

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 15, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 15, 2025

New changes are detected. LGTM label has been removed.

@xenolinux xenolinux added the merge-review-needed Signifies that the merge review team needs to review this PR label Nov 15, 2025
@dfitzmau dfitzmau added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Nov 17, 2025
@dfitzmau
Copy link
Contributor

Hi, @xenolinux . Nice work. Feel free to merge this PR when the certain commented items are addressed. Thanks!

@dfitzmau dfitzmau removed the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Nov 17, 2025
@xenolinux
Copy link
Contributor Author

Hi, @xenolinux . Nice work. Feel free to merge this PR when the certain commented items are addressed. Thanks!

Ack. Thanks for the review.

@openshift-ci
Copy link

openshift-ci bot commented Nov 18, 2025

@xenolinux: all tests passed!

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.20 branch/enterprise-4.21 do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. ok-to-merge size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants