Skip to content

OSDOCS-17200: "Installing Bare Metal" fixing broken link#105177

Merged
jeana-redhat merged 1 commit intoopenshift:enterprise-4.12from
DCChadwick:osdocs17200bug
Jan 28, 2026
Merged

OSDOCS-17200: "Installing Bare Metal" fixing broken link#105177
jeana-redhat merged 1 commit intoopenshift:enterprise-4.12from
DCChadwick:osdocs17200bug

Conversation

@DCChadwick
Copy link
Copy Markdown
Contributor

@DCChadwick DCChadwick commented Jan 21, 2026

Version(s): 4.12

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jan 21, 2026
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 21, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12, 4.13, 4.14, 4.15

Issue: OSDOCS-17200

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

1 similar comment
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 21, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12, 4.13, 4.14, 4.15

Issue: OSDOCS-17200

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 Jan 21, 2026
@ocpdocs-previewbot
Copy link
Copy Markdown

ocpdocs-previewbot commented Jan 21, 2026

🤖 Thu Jan 22 12:37:44 - Prow CI generated the docs preview:
https://105177--ocpdocs-pr.netlify.app
Complete list of updated preview URLs: artifacts/updated_preview_urls.txt

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 21, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12, 4.14, 4.15

Issue: OSDOCS-17200

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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.

@DCChadwick
Copy link
Copy Markdown
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci Bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jan 21, 2026
@ShaunaDiaz ShaunaDiaz removed the merge-review-needed Signifies that the merge review team needs to review this PR label Jan 21, 2026
Copy link
Copy Markdown
Contributor

@ShaunaDiaz ShaunaDiaz left a comment

Choose a reason for hiding this comment

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

The links work, however, I do not believe that RHEL 9 is supported on OpenShift 4.12 for compute nodes? Might want to double-check that this would be a valid link for FIPS since it is a regulatory issue. You might want to have 4.12 point to the RHEL 8 docs.

@DCChadwick
Copy link
Copy Markdown
Contributor Author

Thanks, Shauna! I'm investigating.

Comment thread modules/installation-bare-metal-config-yaml.adoc Outdated
@DCChadwick DCChadwick force-pushed the osdocs17200bug branch 4 times, most recently from 0efcdfd to 8aeffda Compare January 22, 2026 01:28
@DCChadwick
Copy link
Copy Markdown
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci Bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jan 22, 2026
@ShaunaDiaz ShaunaDiaz removed the merge-review-needed Signifies that the merge review team needs to review this PR label Jan 22, 2026
@ShaunaDiaz ShaunaDiaz changed the base branch from enterprise-4.12 to enterprise-4.15 January 22, 2026 12:27
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 22, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12, 4.13, 4.14, 4.15

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

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-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 22, 2026
@ShaunaDiaz ShaunaDiaz changed the base branch from enterprise-4.15 to enterprise-4.12 January 22, 2026 12:27
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 22, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12, 4.13, 4.14, 4.15

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

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.

@ShaunaDiaz
Copy link
Copy Markdown
Contributor

ShaunaDiaz commented Jan 22, 2026

Ah @DCChadwick is this PR just for 4.12? We can merge in that case. If you want to do all versions at once, this PR would have to be pulled against 4.15 and picked backwards.

@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@openshift openshift deleted a comment from openshift-ci-robot Jan 22, 2026
@ShaunaDiaz
Copy link
Copy Markdown
Contributor

/retest

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jan 22, 2026

@DCChadwick: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/validate-portal 088c1b4 link true /test validate-portal
ci/prow/validate-asciidoc 088c1b4 link true /test validate-asciidoc

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.

@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 25, 2026
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 28, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12, 4.13, 4.14, 4.15

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

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

openshift-ci-robot commented Jan 28, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

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.

@DCChadwick
Copy link
Copy Markdown
Contributor Author

@ShaunaDiaz Hi! I revisited this and realized I rushed it. Long story short, this PR is for 4.12 only. I'll build another PR off 4.15 to address 4.14 and 4.15.

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 28, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

Note: The link is in the IMPORTANT block under step 12.

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

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

openshift-ci-robot commented Jan 28, 2026

@DCChadwick: This pull request references OSDOCS-17200 which is a valid jira issue.

Details

In response to this:

Version(s): 4.12

Issue: OSDOCS-17200

Link to docs preview: https://105177--ocpdocs-pr.netlify.app/openshift-enterprise/latest/installing/installing_bare_metal/installing-bare-metal

The fixed link appear in the two IMPORTANT blocks that begin "To enable FIPS..." (one is in the "Optional configuration parameters" table and the other is step 12 of the "Sample install-config.yaml file for bare metal" section).

QE review:

  • QE has approved this change.

Additional information:

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.

@DCChadwick
Copy link
Copy Markdown
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci Bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jan 28, 2026
@jeana-redhat jeana-redhat added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Jan 28, 2026
@jeana-redhat
Copy link
Copy Markdown
Contributor

Thanks, Shauna! I'm investigating.

What was the outcome here? the published 4.12 docs seem to reference RHEL 9 - was that incorrect and now this will correct the version compatibility as well as updating the links?

@DCChadwick
Copy link
Copy Markdown
Contributor Author

Thanks, Shauna! I'm investigating.

What was the outcome here? the published 4.12 docs seem to reference RHEL 9 - was that incorrect and now this will correct the version compatibility as well as updating the links?

@jeana-redhat When I updated the link, I accidentally linked to the RHEL 9 documentation, but I've since updated that updated link and it now points to the RHEL 8 documentation:

link in this PR = https://docs.redhat.com/en/documentation/red_hat_enterprise_linux/8/html/security_hardening/switching-rhel-to-fips-mode_security-hardening[Switching RHEL to FIPS mode]

That "...linux/8/html..." makes all the difference in the world. I just overlooked it when I previously sent this to review (I had a "9" in that position).

Copy link
Copy Markdown
Contributor

@jeana-redhat jeana-redhat left a comment

Choose a reason for hiding this comment

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

/remove-label merge-review-in-progress
/remove-label merge-review-needed

@openshift-ci openshift-ci Bot 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 Jan 28, 2026
@jeana-redhat jeana-redhat merged commit 67e7088 into openshift:enterprise-4.12 Jan 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.12 jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. 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.

7 participants