Skip to content

Conversation

@lahinson
Copy link
Contributor

@lahinson lahinson commented Oct 24, 2024

Version(s): 4.17 only

Issue: https://issues.redhat.com/browse/OSDOCS-12093

Link to docs preview: https://84101--ocpdocs-pr.netlify.app/openshift-enterprise/latest/welcome/learn_more_about_openshift.html#self-managed-hcp

QE review:

  • QE has approved this change.

Additional information: Do not merge until the MCE 2.7 release on Nov. 5.

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 24, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 24, 2024

🤖 Wed Nov 06 22:00:20 - Prow CI generated the docs preview:

https://84101--ocpdocs-pr.netlify.app/openshift-enterprise/latest/welcome/learn_more_about_openshift.html

@lahinson lahinson force-pushed the osdocs-12093-hcp-welcome branch from eb7243d to 53b3920 Compare October 24, 2024 18:12
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 24, 2024
@lahinson
Copy link
Contributor Author

@xenolinux - When you're back from PTO next week, PTAL at the Welcome page and let me know if you think I should highlight different topics or any additional topics. Thank you!

Copy link
Contributor

@xenolinux xenolinux left a comment

Choose a reason for hiding this comment

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

Thanks for the welcome page PR.
The table looks great!

@jiezhao16
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 28, 2024
@lahinson lahinson added the peer-review-needed Signifies that the peer review team needs to review this PR label Oct 28, 2024
@mburke5678 mburke5678 added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Oct 28, 2024
@mburke5678
Copy link
Contributor

@lahinson Just one comment. Otherwise LGTM

@mburke5678 mburke5678 added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR peer-review-needed Signifies that the peer review team needs to review this PR labels Oct 28, 2024
@mburke5678 mburke5678 added this to the Continuous Release milestone Oct 28, 2024
@lahinson lahinson force-pushed the osdocs-12093-hcp-welcome branch from 53b3920 to af68373 Compare October 28, 2024 19:32
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 28, 2024
@openshift-ci
Copy link

openshift-ci bot commented Oct 28, 2024

New changes are detected. LGTM label has been removed.

@lahinson lahinson marked this pull request as draft November 4, 2024 19:48
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2024
@lahinson lahinson marked this pull request as ready for review November 6, 2024 21:02
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 6, 2024
@lahinson lahinson force-pushed the osdocs-12093-hcp-welcome branch from af68373 to 0694566 Compare November 6, 2024 21:47
@lahinson lahinson force-pushed the osdocs-12093-hcp-welcome branch from 0694566 to a3a8d0e Compare November 6, 2024 21:49
@openshift-ci
Copy link

openshift-ci bot commented Nov 6, 2024

@lahinson: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

@lahinson lahinson merged commit ac7420b into openshift:enterprise-4.17 Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.17 ok-to-merge peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants