Update OCP documentation links to latest versions#3517
Update OCP documentation links to latest versions#3517sebrandon1 wants to merge 1 commit intoopenshift:masterfrom
Conversation
This commit updates OpenShift Container Platform documentation URLs to point to the latest available versions. Updated by: ocp-doc-checker More info: https://github.com/sebrandon1/ocp-doc-checker
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sebrandon1 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
| common across both methods: | ||
| * The key used in the *cloud-private-key* [secret](../README.md#Usage) and the key used when creating the cluster should | ||
| be added to the [ssh-agent](https://docs.redhat.com/en/documentation/openshift_container_platform/4.15/html/installing_on_azure/installing-azure-default#ssh-agent-using_installing-azure-default). | ||
| be added to the [ssh-agent](https://docs.redhat.com/en/documentation/openshift_container_platform/4.16/html/installing_on_azure/installing-azure-default#ssh-agent-using_installing-azure-default). |
There was a problem hiding this comment.
Should we use this link instead? Is this the same module in 4.20 as the 4.15?
https://docs.redhat.com/en/documentation/openshift_container_platform/4.20/html/installing_on_azure/installer-provisioned-infrastructure#ssh-agent-using_installing-azure-preparing-ipi
There was a problem hiding this comment.
@mburke5678 is it still safe to use the latest placeholder in the links to resolve the latest?
| * [Red Hat Insights cost management](https://docs.redhat.com/en/documentation/cost_management_service/1-latest) | ||
| * [Red Hat OpenShift Local](https://developers.redhat.com/products/openshift-local/overview) | ||
| * [OpenShift monitoring of user defined project](https://docs.redhat.com/en/documentation/openshift_container_platform/4.10/html/monitoring/enabling-monitoring-for-user-defined-projects) | ||
| * [OpenShift monitoring of user defined project](https://docs.redhat.com/en/documentation/openshift_container_platform/4.13/html/monitoring/enabling-monitoring-for-user-defined-projects) |
There was a problem hiding this comment.
Should we use this link instead? Is this the same module in 4.20 as the 4.15?
https://docs.redhat.com/en/documentation/openshift_container_platform/4.20/html/monitoring/configuring-user-workload-monitoring#enabling-monitoring-for-user-defined-projects-uwm_preparing-to-configure-the-monitoring-stack-uwm
There was a problem hiding this comment.
So my tool just looks for if the URL + version + #anchor stays the same and nothing else changes. If the actual URL + version + anchor needs a change like that I'd say it'd be a separate change.
There was a problem hiding this comment.
Looks like 4.13 is that last version where that URL + #anchor stays the same.
There was a problem hiding this comment.
There was a problem hiding this comment.
Agreed! I think that might have to be a separate change though because the underlying URL + #anchor is different than the original. I can help open up a separate PR after this one merges if needed?
|
@sebrandon1: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
jrvaldes
left a comment
jrvaldes
left a comment
There was a problem hiding this comment.
@sebrandon1 Thanks for opening this PR. LGTM
will defer from @mburke5678 for approval
3 participants
This PR updates OpenShift Container Platform documentation URLs to point to the latest available versions.
Changes
Tool Information
This PR was created automatically by scanning the repository with
ocp-doc-checker, which identifies and updates outdated OpenShift documentation links.Repository: https://github.com/sebrandon1/ocp-doc-checker
Tracking Issue: sebrandon1/ocp-doc-checker#18