-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Update gathering-data-specific-features.adoc #96197
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
base: main
Are you sure you want to change the base?
Conversation
Below is step 5: 5. Attach the compressed file to your support case on the the Customer Support page of the Red Hat Customer Portal. In the above sentence word `the` `the` is mentioned twice. Which is not required. We need to remove one word from it. Here is the updated look: 5. Attach the compressed file to your support case on the Customer Support page of the Red Hat Customer Portal. Also, in step 3 command structure is wrong: 3. Run the oc adm must-gather command with one or more -image or -image-stream arguments. For example, the following command gathers both the default cluster data and information specific to KubeVirt: $ oc adm must-gather \ --image-stream=openshift/must-gather \ --image=quay.io/kubevirt/must-gather Here, the `- -` sign should be below `oc`. Command will as it is, but it is not as per our standard documentation rule. Hence we need to change it. Here is the updated look: 3. Run the oc adm must-gather command with one or more -image or -image-stream arguments. For example, the following command gathers both the default cluster data and information specific to KubeVirt: $ oc adm must-gather \ --image-stream=openshift/must-gather \ --image=quay.io/kubevirt/must-gather
@prithvipatil97: 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. |
/label peer-review-needed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Hello @JoeAldinger Regards, |
@anuragthehatter could you help find a QE review for this, please and thanks. |
Below is step 5:
the
the
is mentioned twice. Which is not required.Also, in step 3 command structure is wrong:
- -
sign should be belowoc
.Here is the updated look:
Version(s):
RHOCP 4.20, RHOCP 4.19, RHOCP 4.18, RHOCP 4.17, RHOCP 4.16, RHOCP 4.15, RHOCP 4.14, RHOCP 4.13, RHOCP 4.12
Issue:
https://issues.redhat.com/browse/OSDOCS-15334
Link to docs preview:
https://96197--ocpdocs-pr.netlify.app/openshift-dedicated/latest/support/gathering-cluster-data.html
https://96197--ocpdocs-pr.netlify.app/openshift-enterprise/latest/support/gathering-cluster-data.html
https://96197--ocpdocs-pr.netlify.app/openshift-rosa-hcp/latest/support/gathering-cluster-data.html
https://96197--ocpdocs-pr.netlify.app/openshift-rosa/latest/support/gathering-cluster-data.html
QE review:
Additional information: