RHDEVDOCS-6199 - Builds for OpenShift: Install and Config Documentation Obsolete

[shivanisathe25]

Version(s):

Version for cherrypicking: build-docs-1.1

Issue:

Link to docs preview: https://84027--ocpdocs-pr.netlify.app/openshift-builds/latest/installing/installing-openshift-builds.html

QE review: @apoorvajagtap @sayan-biswas

• QE has approved this change.

Additional information:

[ocpdocs-previewbot]

🤖 Mon Nov 11 11:29:20 - Prow CI generated the docs preview:

https://84027--ocpdocs-pr.netlify.app/
https://84027--ocpdocs-pr.netlify.app/openshift-builds/latest/installing/installing-openshift-builds.html

[apoorvajagtap]

LGTM.

[shivanisathe25]

/label peer-review-needed

[mburke5678]

@shivanisathe25 Why are these verification steps not part of the Installing Builds modules? I am not sure the verification step should be in its own module.
Also, adding the verification to the install module would remove the redundancy in the verification modules, where the introductory sentence is repeated as the verification step.

[mburke5678]

@shivanisathe25 A few suggestions and a question.

[apoorvajagtap]

LGTM.

[shivanisathe25]

/label peer-review-needed

[shivanisathe25]

/remove-label peer-review-needed

[shivanisathe25]

/label merge-review-needed

[xenolinux]

I added some comments and questions for your consideration. Please add the merge-review label again after addressing changes.

General comment since you are already editing the modules/ob-installing-builds-using-CLI.adoc file:

• We don't usually use the capital letters in the file name. You can modify the file name from ob-installing-builds-using-CLI.adoc to ob-installing-builds-using-cli.adoc.
• In the preview, I noticed that line no 9 You can also install {builds-shortname} by using the CLI may use some tiny changes such as removing "also" and expanding the abbreviation "CLI" because it's a first time occurrence in a module.
  For example: You can install {builds-shortname} by using the command-line interface (CLI).

[adambkaplan]

/lgtm

[openshift-ci]

New changes are detected. LGTM label has been removed.

[openshift-ci]

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

[shivanisathe25]

/label merge-review-needed

[jab-rh]

/cherry-pick build-docs-1.1

[openshift-cherrypick-robot]

@jab-rh: new pull request created: #84772

In response to this:

/cherry-pick build-docs-1.1

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.