-
Notifications
You must be signed in to change notification settings - Fork 1.8k
OCPBUGS-52653 updated the tar code in step 3 and also added sentence in step 3 about unique cluster ID #91615
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
a19398f
to
76268f8
Compare
@@ -75,12 +75,11 @@ Because this command picks a random control plane node by default, the pod might | |||
Contact Red Hat Support for the recommended resources to gather. | |||
==== | |||
|
|||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. For example, on a computer that uses a Linux | |||
operating system, run the following command: | |||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. Make sure you provide the date and cluster ID for a unique must-gather. For moe information on how to fiind the cluster ID, see the Red Hat Knowledge base solution link:https://access.redhat.com/solutions/5280291[How to find the cluster-id or name on Opeshift cluster]. For example, on a computer that uses a Linux operating system, run the following command: | |||
+ |
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.
There is a typo fiind the cluster ID
, please update , thanks
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.
@zhouying7780 Can you please review the docs again under "Links to doc preview" above. Thanks.
modules/support-gather-data.adoc
Outdated
+ | ||
[source,terminal] | ||
---- | ||
$ tar cvaf must-gather.tar.gz must-gather.local.5421342344627712289/ <1> | ||
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-$CLUSTERID.tar.gz must-gather.local.426787175854303238 <1> | ||
---- | ||
<1> Make sure to replace `must-gather-local.5421342344627712289/` with the actual directory name. |
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.
here should be the same as the example : must-gather.local.426787175854303238
76268f8
to
2357de0
Compare
2357de0
to
120edde
Compare
@wgabor0427 The typo and for |
33376bd
to
421df5a
Compare
99e4dd0
to
aa4f0ee
Compare
/lgtm |
/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.
Left a few comments; otherwise LGTM
@@ -22,11 +22,11 @@ $ oc adm must-gather -- gather_network_logs | |||
==== | |||
By default, the `must-gather` tool collects the OVN `nbdb` and `sbdb` databases from all of the nodes in the cluster. Adding the `-- gather_network_logs` option to include additional logs that contain OVN-Kubernetes transactions for OVN `nbdb` database. | |||
==== | |||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. For example, on a computer that uses a Linux operating system, run the following command: | |||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. Make sure you provide the date and cluster ID for a unique must-gather. For more information on how to find the cluster ID, see the Red Hat Knowledge base solution link:https://access.redhat.com/solutions/5280291[How to find the cluster-id or name on OpenShift cluster]. For example, on a computer that uses a Linux operating system, run the following command: |
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.
. Create a compressed file from the `must-gather` directory that was just created in your working directory. Make sure you provide the date and cluster ID for a unique must-gather. For more information on how to find the cluster ID, see the Red Hat Knowledge base solution link:https://access.redhat.com/solutions/5280291[How to find the cluster-id or name on OpenShift cluster]. For example, on a computer that uses a Linux operating system, run the following command: | |
. Create a compressed file from the `must-gather` directory that was just created in your working directory. Make sure you provide the date and cluster ID for a unique must-gather data. For more information about how to find the cluster ID, see link:https://access.redhat.com/solutions/5280291[How to find the cluster-id or name on OpenShift cluster]. For example, on a computer that uses a Linux operating system, run the following command: |
- "....a unique must-gather": Assuming this is poiniting to a unique must-gather data. If not, please replace with the suitable name/term.
- "...Knowledge base solution" can be omitted
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.
- Replaced "for a unique must-gather" with "for the unique must-gather data."
- Deleted "Knowledge base solution"
+ | ||
[source,terminal] | ||
---- | ||
$ tar cvaf must-gather.tar.gz must-gather.local.472290403699006248 <1> | ||
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-$CLUSTERID.tar.gz must-gather.local.426787175854303238 <1> |
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.
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-$CLUSTERID.tar.gz must-gather.local.426787175854303238 <1> | |
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-<cluster_id>.tar.gz <must_gather_local_dir> <1> |
- I'd suggest to use a user-replaced variable for cluster id. I believe
$
is not needed for <cluster_id>. You might want to confirm with SME/QE. - because
must-gather.local.426787175854303238
is replaceable, suggestion to use a placeholder --<must_gather_local_dir>
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.
- Added the user-replaced variable <cluster_id>
- Added the placeholder <must_gather_local_dir>
+ | ||
[source,terminal] | ||
---- | ||
$ tar cvaf must-gather.tar.gz must-gather.local.472290403699006248 <1> | ||
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-$CLUSTERID.tar.gz must-gather.local.426787175854303238 <1> | ||
---- | ||
<1> Replace `must-gather-local.472290403699006248` with the actual directory name. |
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.
<1> Replace `must-gather-local.472290403699006248` with the actual directory name. | |
<1> Replace `<must_gather_local_dir>` with the actual directory name. |
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.
Done
@@ -371,14 +371,14 @@ $ oc adm must-gather \ | |||
<2> The must-gather image for KubeVirt | |||
|
|||
ifndef::openshift-origin[] | |||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. For example, on a computer that uses a Linux | |||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. Make sure you provide the date and cluster ID for a unique must-gather. For more information on how to find the cluster ID, see the Red Hat Knowledge base solution link:https://access.redhat.com/solutions/5280291[How to find the cluster-id or name on OpenShift cluster]. For example, on a computer that uses a Linux |
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.
Same comments as earlier
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.
Made same edits in this file
operating system, run the following command: | ||
+ | ||
[source,terminal] | ||
---- | ||
$ tar cvaf must-gather.tar.gz must-gather.local.5421342344627712289/ <1> | ||
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-$CLUSTERID.tar.gz must-gather.local.426787175854303238 <1> |
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.
Same comments as earlier
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.
Made same edits in this file
---- | ||
<1> Make sure to replace `must-gather-local.5421342344627712289/` with the | ||
<1> Make sure to replace `must-gather.local.426787175854303238` with the |
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.
<1> Make sure to replace `must-gather.local.426787175854303238` with the | |
<1> Make sure to replace `<must_gather_local_dir>` with the actual directory name. |
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.
Made changes
---- | ||
<1> Make sure to replace `must-gather-local.5421342344627712289/` with the | ||
<1> Make sure to replace `must-gather.local.426787175854303238` with the | ||
actual directory name. |
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.
actual directory name. |
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.
Made the same changes to this file as above
modules/support-gather-data.adoc
Outdated
@@ -75,14 +75,13 @@ Because this command picks a random control plane node by default, the pod might | |||
Contact Red Hat Support for the recommended resources to gather. | |||
==== | |||
|
|||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. For example, on a computer that uses a Linux | |||
operating system, run the following command: | |||
. Create a compressed file from the `must-gather` directory that was just created in your working directory. Make sure you provide the date and cluster ID for a unique must-gather. For more information on how to find the cluster ID, see the Red Hat Knowledge base solution link:https://access.redhat.com/solutions/5280291[How to find the cluster-id or name on Opeshift cluster]. For example, on a computer that uses a Linux operating system, run the following command: |
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.
Same comments
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.
Done
modules/support-gather-data.adoc
Outdated
$ tar cvaf must-gather-`date +"%m-%d-%Y-%H-%M-%S"`-$CLUSTERID.tar.gz must-gather.local.426787175854303238 <1> | ||
---- | ||
<1> Make sure to replace `must-gather-local.5421342344627712289/` with the actual directory name. | ||
<1> Make sure to replace `must-gather-local.426787175854303238/` with the actual directory name. |
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.
Same comments
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.
Done
aa4f0ee
to
d86b187
Compare
New changes are detected. LGTM label has been removed. |
8d54a4d
to
fd44598
Compare
/retest |
@wgabor0427: 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. |
@xenolinux I've made the changes you suggested. Does this need to go through QE again? |
Thanks @wgabor0427 |
Version(s):
4.12+
Issue:
https://issues.redhat.com/browse/OCPBUGS-52653
Link to docs preview:
https://91615--ocpdocs-pr.netlify.app/openshift-dedicated/latest/support/gathering-cluster-data.html
https://91615--ocpdocs-pr.netlify.app/openshift-enterprise/latest/support/gathering-cluster-data.html
https://91615--ocpdocs-pr.netlify.app/openshift-rosa-hcp/latest/support/gathering-cluster-data.html
https://91615--ocpdocs-pr.netlify.app/openshift-rosa/latest/support/gathering-cluster-data.html
QE review:
The following topic steps should have the same information: