-
Notifications
You must be signed in to change notification settings - Fork 467
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
DOC-8738: Warn users that generating the debug zip can degrade the cluster #17952
Conversation
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify site configuration. |
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
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.
Hi @florence-crl, do you think we can include guidance and document this new flag? We have a few threads and issues related to the range info filter.
Doc issue: #16919
PR: cockroachdb/cockroach#102289
Before I can document #16919, I need to clarify what is the default value. For context, see this slack thread. |
Clarification: cockroachdb/cockroach#104133 changed the default to |
ac25b0f
to
65d9c75
Compare
@dt, PTAL, I included your recommendation under the |
65d9c75
to
187817b
Compare
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.
Nice! Looks good Florence. Thanks for the updates.
…ommendation to filter by time range, and --include-range-info flag.
187817b
to
da9fa8f
Compare
Addresses DOC-8738
(1) Updated cockroach debug zip page with cluster impact warning and recommendation to filter by time range.