-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Exposed replica information for monitoring readable secondaries #8322
Conversation
Merge with master
rebase with master
@denzilribeiro Could you please review this PR for exposing replica information to monitor readable secondaries? |
This won't work for Hyperscale btw as neither of these have rows in sys.dm_db_replica_states so will only work for Business critical. |
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 suggestions to make accross the board for MI/DB
Required for all PRs:
Modified queries in azuresqlqueries.go to include replica_ID and is_primary_replica for monitoring readable secondaries.