Skip to content

OSDOCS-16614: Documented VIP optional with user-managed LB feature #104065

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.

Sign up for GitHub

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

Jump to bottom
Open
dfitzmau wants to merge 1 commit into
base: main
Choose a base branch
from
Open

OSDOCS-16614: Documented VIP optional with user-managed LB feature #104065

dfitzmau wants to merge 1 commit into from

Conversation

@dfitzmau
Copy link
Contributor

@dfitzmau dfitzmau commented Dec 17, 2025 •
edited
Loading

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Dec 17, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 17, 2025 •
edited by openshift-ci bot
Loading

@dfitzmau: This pull request references OSDOCS-16614 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the epic to target either version "4.22." or "openshift-4.22.", but it targets "openshift-4.21" instead.

Details

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16614

Link to docs preview:

  • SME has approved this change.
  • QE has approved this change.

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 17, 2025
@dfitzmau dfitzmau force-pushed the OSDOCS-16614 branch 2 times, most recently from ed2f3c4 to d58bc86 Compare December 17, 2025 12:53
@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 17, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Dec 17, 2025 •
edited
Loading

🤖 Thu Dec 18 12:36:16 - Prow CI generated the docs preview:

https://104065--ocpdocs-pr.netlify.app/openshift-dedicated/latest/networking/networking_operators/dns-operator.html
https://104065--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/upi/installing-bare-metal.html
https://104065--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/upi/installing-bare-metal-network-customizations.html
https://104065--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/upi/installing-restricted-networks-bare-metal.html
https://104065--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/k8s_nmstate/k8s-nmstate-updating-node-network-config.html
https://104065--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/networking_operators/dns-operator.html
https://104065--ocpdocs-pr.netlify.app/openshift-rosa-hcp/latest/networking/networking_operators/dns-operator.html
https://104065--ocpdocs-pr.netlify.app/openshift-rosa/latest/networking/networking_operators/dns-operator.html

@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 Dec 17, 2025
ocpdocs-vale-bot
ocpdocs-vale-bot reviewed Dec 17, 2025
View reviewed changes
modules/installation-bare-metal-dns-record-type.adoc
* `Internal`: The default value. Setting this value causes the cluster infrastructure to automatically create and maintain the necessary DNS records.
* `External`: You can use this value only if you set the `loadBalancer.type` parameter to `UserManaged`. The cluster does not manage the DNS records. You must manually configure DNS records on an external DNS server.

.Prerequisites
Copy link
Collaborator

@ocpdocs-vale-bot ocpdocs-vale-bot Dec 17, 2025

Choose a reason for hiding this comment

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

🤖 [error] AsciiDocDITA.BlockTitle: Block titles can only be assigned to examples, figures, and tables in DITA.

ocpdocs-vale-bot
ocpdocs-vale-bot reviewed Dec 17, 2025
View reviewed changes
modules/installation-bare-metal-dns-record-type.adoc

* If you intend on setting `dnsRecordsType.External` in the `infrastructure.config.openshift.io` CR , you must initially configure cluster nodes to use the specific external server for DNS resolution.

.Procedure
Copy link
Collaborator

@ocpdocs-vale-bot ocpdocs-vale-bot Dec 17, 2025

Choose a reason for hiding this comment

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

🤖 [error] AsciiDocDITA.BlockTitle: Block titles can only be assigned to examples, figures, and tables in DITA.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 17, 2025 •
edited by openshift-ci bot
Loading

@dfitzmau: This pull request references OSDOCS-16614 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the epic to target either version "4.22." or "openshift-4.22.", but it targets "openshift-4.21" instead.

Details

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16614

Link to docs preview:

  • SME has approved this change.
  • QE has approved this change.

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 17, 2025 •
edited by openshift-ci bot
Loading

@dfitzmau: This pull request references OSDOCS-16614 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the epic to target either version "4.22." or "openshift-4.22.", but it targets "openshift-4.21" instead.

Details

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16614

Link to docs preview:

  • SME has approved this change.
  • QE has approved this change.

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci
Copy link

openshift-ci bot commented Dec 18, 2025

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

cybertron
cybertron suggested changes Dec 19, 2025
View reviewed changes
Copy link
Member

@cybertron cybertron left a comment

Choose a reason for hiding this comment

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

This can't be set via the CR, it needs to be done in install-config at initial deployment.

modules/installation-bare-metal-dns-record-type.adoc
:FeatureName: Configuring the DnsRecordsType parameter
include::snippets/technology-preview.adoc[]

To use the parameter, you must enable the `OnPremDNSRecords` feature gate in the `config.yaml` file.
Copy link
Member

@cybertron cybertron Dec 19, 2025

Choose a reason for hiding this comment

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

The featuregate is part of the TechPreview featureset. There's no explicit enablement required.

modules/installation-bare-metal-dns-record-type.adoc
= Configuring the DnsRecordsType parameter

[role="_abstract"]
You can use the `DnsRecordsType` parameter in your `infrastructure.config.openshift.io` custom resource (CR) to set if the internal DNS service or an external source provides the necessary records for `api`, `api-int`, and `ingress` DNS records.
Copy link
Member

@cybertron cybertron Dec 19, 2025

Choose a reason for hiding this comment

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

This can't be configured via the CR. It can only be set at install time in install-config.yaml.

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

Reviewers

2 more reviewers

@ocpdocs-vale-bot ocpdocs-vale-bot ocpdocs-vale-bot left review comments

@cybertron cybertron cybertron requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dfitzmau @openshift-ci-robot @ocpdocs-previewbot @cybertron @ocpdocs-vale-bot