-
Notifications
You must be signed in to change notification settings - Fork 451
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-6747: Docs for Improve network observability inside CockroachDB #17824
Conversation
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-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.
ac87844
to
a18f647
Compare
0bdd3cf
to
11afcdd
Compare
11afcdd
to
a6f988b
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.
Thanks @mdlinville for the careful review!
a6f988b
to
e323199
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.
I did not make a separate Network Partitions
section. I did modify the description of a network partition to match the changes @mdlinville suggested to the description in the troubleshooting doc.
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.
Great doc! 👏
…f-hosted-deployments. (3) Updated Troubleshoot Self-Hosted Setup page, Network partitioning section.
e323199
to
ca941d2
Compare
Addresses DOC-6747
(1) Updated Network page and images. (2) Updated sidebar-data for self-hosted-deployments. (3) Updated Troubleshoot Self-Hosted Setup page, Network partitioning section.