-
Notifications
You must be signed in to change notification settings - Fork 449
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
Add docs on monitoring WAL failover #18548
Conversation
Files changed: |
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
@jbowens this is the content on WAL failover monitoring I'd like to add directly following merge of #18511 will be rebasing this PR on main once that other PR goes in and then the placement of this content will actually make sense (ideally) opening in parallel for speed since v24.1 public release is next Monday |
✅ 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!
@florence-crl wanted to add some context b/c this PR is "different", this is a followup to #18511 with just the monitoring info I split it into a separate PR since that other one is taking a little time to get through review - that one will also be coming your way ASAP for review of course! |
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 suggestion to format links in liquid. Otherwise LGTM!
b6da131
to
249fca1
Compare
Fixes DOC-10268
Followup to #18511 with the monitoring piece