Skip to content

HCIDOCS-729-v416: Replace corrupt links v416 #96070

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

Merged
merged 1 commit into from
Jul 15, 2025

Conversation

tshwartz
Copy link
Contributor

@tshwartz tshwartz commented Jul 14, 2025

Version(s):
4.16

Issue:
https://issues.redhat.com/browse/HCIDOCS-729

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:
Approved and published for version 4.17 and later (#95211), but I missed version 4.16.

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

Copy link

openshift-ci bot commented Jul 14, 2025

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

@tshwartz
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Jul 14, 2025
@tmalove
Copy link
Contributor

tmalove commented Jul 14, 2025

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

@openshift-ci openshift-ci bot added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Jul 14, 2025
@@ -33,11 +33,11 @@ For the full procedure, see link:https://docs.oracle.com/en-us/iaas/Content/Obje
+
[IMPORTANT]
====
The Terraform stack includes files for creating cluster resources and custom manifests. The stack also includes a script, and when you apply the stack, the script creates {oci} resources, such as DNS records, an instance, and so on. For a list of the resources, see link:https://github.com/oracle-openshift/oci-openshift/blob/main/terraform-stacks/README.md[Terraform Defined Resources for OpenShift on {oci} README file].
The Terraform stack includes files for creating cluster resources and custom manifests. The stack also includes a script, and when you apply the stack, the script creates {oci} resources, such as DNS records, an instance, and so on. For a list of the resources, see the `terraform-stacks` folder in link:https://github.com/oracle-quickstart/oci-openshift/tree/main[OpenShift on OCI (OSO)].
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The Terraform stack includes files for creating cluster resources and custom manifests. The stack also includes a script, and when you apply the stack, the script creates {oci} resources, such as DNS records, an instance, and so on. For a list of the resources, see the `terraform-stacks` folder in link:https://github.com/oracle-quickstart/oci-openshift/tree/main[OpenShift on OCI (OSO)].
The Terraform stack includes files for creating cluster resources and custom manifests. The stack also includes a script, and when you apply the stack, the script creates {oci} resources, such as DNS records, an instance, and other resources. For a list of the resources, see the `terraform-stacks` folder in link:https://github.com/oracle-quickstart/oci-openshift/tree/main[OpenShift on OCI (OSO)].

See ISG: https://www.ibm.com/docs/en/ibm-style?topic=word-usage#a
Suggested 'resources' as an option, but another term that you think is more accurate will be good.

@tmalove
Copy link
Contributor

tmalove commented Jul 14, 2025

/remove-label peer-review-in-progress
/label peer-review-done

Direct preview link.

@openshift-ci openshift-ci bot 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 labels Jul 14, 2025
@tshwartz
Copy link
Contributor Author

tshwartz commented Jul 15, 2025

Thanks for your review @tmalove . I will not make the change here because I would like v4.16 to be the same as the later versions already published here (#95211). I have created a bug for it instead: https://issues.redhat.com/browse/HCIDOCS-760 and hope that's OK. Please let me know. Thanks!

@tshwartz
Copy link
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 Jul 15, 2025
@tmalove
Copy link
Contributor

tmalove commented Jul 15, 2025

@tshwartz I'm OK with that Talia...thanks for letting me know.

@bscott-rh bscott-rh added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Jul 15, 2025
====
+
--
.. Upload the Terraform stack template link:https://github.com/oracle-openshift/oci-openshift/releases/tag/main[create-cluster-vX.X.X.zip] to the new object storage bucket.
.. Upload the Terraform stacks template link:https://github.com/oracle-quickstart/oci-openshift/tree/main/terraform-stacks[terrform-stacks] to the new object storage bucket.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
.. Upload the Terraform stacks template link:https://github.com/oracle-quickstart/oci-openshift/tree/main/terraform-stacks[terrform-stacks] to the new object storage bucket.
.. Upload the Terraform stacks template link:https://github.com/oracle-quickstart/oci-openshift/tree/main/terraform-stacks[terraform-stacks] to the new object storage bucket.

Copy link
Contributor

@bscott-rh bscott-rh left a comment

Choose a reason for hiding this comment

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

per slack, Talia will resolve this typo in a separate PR.

@bscott-rh bscott-rh added branch/enterprise-4.16 and removed merge-review-in-progress Signifies that the merge review team is reviewing this PR labels Jul 15, 2025
@bscott-rh bscott-rh added this to the Continuous Release milestone Jul 15, 2025
@bscott-rh bscott-rh merged commit 2ad1bf6 into openshift:enterprise-4.16 Jul 15, 2025
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.16 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.

4 participants