Skip to content

[OSDOCS-17908]: Configuring ingress end point for a hosted cluster#107009

Merged
lahinson merged 2 commits intoopenshift:mainfrom
lahinson:osdocs-17908-hcp-ingress-controller
Mar 2, 2026
Merged

[OSDOCS-17908]: Configuring ingress end point for a hosted cluster#107009
lahinson merged 2 commits intoopenshift:mainfrom
lahinson:osdocs-17908-hcp-ingress-controller

Conversation

@lahinson
Copy link
Copy Markdown
Contributor

@lahinson lahinson commented Feb 19, 2026

Version(s): 4.21+

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

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information: This PR adds instructions about how to configure the endpoint for a nodeport at cluster creation time so that load balancer configuration is easier. I also made a few updates to the files to ensure that they are compliant with the upcoming migration to DITA.

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Feb 19, 2026
@lahinson lahinson added this to the Continuous Release milestone Feb 19, 2026
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 23, 2026
@lahinson lahinson force-pushed the osdocs-17908-hcp-ingress-controller branch from 08bb347 to 41e41ef Compare February 23, 2026 20:54
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 23, 2026
@lahinson lahinson force-pushed the osdocs-17908-hcp-ingress-controller branch 3 times, most recently from b6b13d1 to 1d49244 Compare February 24, 2026 15:46
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 24, 2026
@lahinson lahinson added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 26, 2026
@lahinson lahinson force-pushed the osdocs-17908-hcp-ingress-controller branch from 1d49244 to 281d557 Compare February 26, 2026 14:58
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Feb 26, 2026
@lahinson lahinson force-pushed the osdocs-17908-hcp-ingress-controller branch from 281d557 to c24291f Compare February 26, 2026 15:18
@lahinson
Copy link
Copy Markdown
Contributor Author

@mburke5678 Thanks for your review! I'm going to have QE give this one more check to ensure that my latest updates are accurate. I'll remove this from the merge review queue for the moment.

@lahinson lahinson removed merge-review-in-progress Signifies that the merge review team is reviewing this PR merge-review-needed Signifies that the merge review team needs to review this PR labels Feb 26, 2026
@mburke5678
Copy link
Copy Markdown
Contributor

A few comments. Otherwise LGTM

Copy link
Copy Markdown

@prb112 prb112 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 Feb 26, 2026
@lahinson lahinson force-pushed the osdocs-17908-hcp-ingress-controller branch from c24291f to 6312529 Compare March 2, 2026 15:40
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 2, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 2, 2026

New changes are detected. LGTM label has been removed.

@lahinson
Copy link
Copy Markdown
Contributor Author

lahinson commented Mar 2, 2026

Per QE review from the IBM Z team, I'm removing the ingress endpoint step from the IBM Z module, as it is not yet supported on that platform.

@lahinson lahinson force-pushed the osdocs-17908-hcp-ingress-controller branch from 6312529 to f66033a Compare March 2, 2026 16:00
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 2, 2026

@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 b9e8b49 into openshift:main Mar 2, 2026
2 checks passed
@lahinson
Copy link
Copy Markdown
Contributor Author

lahinson commented Mar 2, 2026

/cherrypick enterprise-4.21

@lahinson
Copy link
Copy Markdown
Contributor Author

lahinson commented Mar 2, 2026

/cherrypick enterprise-4.22

@openshift-cherrypick-robot
Copy link
Copy Markdown

@lahinson: new pull request created: #107710

Details

In response to this:

/cherrypick enterprise-4.21

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
Copy link
Copy Markdown

@lahinson: new pull request created: #107711

Details

In response to this:

/cherrypick enterprise-4.22

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.

----
#...
spec:
operatorConfiguration:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry for being late. This PR looks good. The only thing is the formatting of this snippet. Can you move the lines after "spec" to left? The indentation between spec and the following lines should be only two spaces.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will do - thanks @mgencur!

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

Labels

branch/enterprise-4.21 branch/enterprise-4.22 do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. 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.

8 participants