Skip to content
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-44731]: Add note about service publishing for HCP on BM #86913

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

lahinson
Copy link
Contributor

@lahinson lahinson commented Jan 10, 2025

Version(s): 4.17+

Issue: https://issues.redhat.com/browse/OCPBUGS-44731

Link to docs preview: https://86913--ocpdocs-pr.netlify.app/openshift-enterprise/latest/hosted_control_planes/hcp-deploy/hcp-deploy-bm.html#hcp-bm-hc_hcp-deploy-bm (see the last bullet point)

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 10, 2025
Copy link

@jparrill jparrill left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 13, 2025
@jiezhao16
Copy link

/lgtm

@lahinson lahinson added peer-review-needed Signifies that the peer review team needs to review this PR branch/enterprise-4.17 branch/enterprise-4.18 labels Jan 13, 2025
@lahinson lahinson added this to the Continuous Release milestone Jan 13, 2025
@skopacz1 skopacz1 added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Jan 13, 2025
Copy link
Contributor

@skopacz1 skopacz1 left a comment

Choose a reason for hiding this comment

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

LGTM, just left one comment for you to consider!

@@ -18,6 +18,8 @@ As you create a hosted cluster, keep the following guidelines in mind:

- A hosted cluster cannot be created in the namespace of a {mce-short} managed cluster.

- The most common service publishing strategy is to expose services through a load balancer. That strategy is the preferred method for exposing the Kubernetes API server. If you create a hosted cluster by using the web console or by using {rh-rhacm-title}, to set a publishing strategy for a service besides the Kubernetes API server, you must manually specify the `servicePublishingStrategy` in the `HostedCluster` custom resource.
Copy link
Contributor

Choose a reason for hiding this comment

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

It's probably fine as is, but it reads a little odd that there's no noun following "the servicePublishingStrategy" based on what i see elsewhere in the docs. Is it possible to add something like "parameter" or "option" there? Up to you

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. I don't think "parameter" or "option" are quite right, so I added "information" as the noun after servicePublishingStrategy.

@skopacz1 skopacz1 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 Jan 13, 2025
@lahinson lahinson force-pushed the ocpbugs-44731-hcp-load-balancer branch from 0a2d26b to 10e60b7 Compare January 13, 2025 20:10
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jan 13, 2025
Copy link

openshift-ci bot commented Jan 13, 2025

New changes are detected. LGTM label has been removed.

Copy link

openshift-ci bot commented Jan 13, 2025

@lahinson: 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.

@lahinson lahinson merged commit fd1f597 into openshift:main Jan 13, 2025
2 checks passed
@lahinson
Copy link
Contributor Author

/cherrypick enterprise-4.17

@lahinson
Copy link
Contributor Author

/cherrypick enterprise-4.18

@openshift-cherrypick-robot

@lahinson: new pull request created: #86966

In response to this:

/cherrypick enterprise-4.17

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.

@openshift-cherrypick-robot

@lahinson: new pull request created: #86967

In response to this:

/cherrypick enterprise-4.18

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.17 branch/enterprise-4.18 peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants