Open GuptaManan100 opened 4 days ago
Hello reviewers! :wave: Please follow this checklist when reviewing this Pull Request.
release notes (needs details)
label if users need to know about this change.-
), and have a clear help text.Jobs
should be named in order to mark it as required
.required
, the maintainer team must be notified._vt
tables and RPCs need to be backward compatible.vtctl
command output order should be stable and awk
-able.Attention: Patch coverage is 71.42857%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 69.53%. Comparing base (
95f2e3e
) to head (bd70d2d
).
Files with missing lines | Patch % | Lines |
---|---|---|
go/vt/vtorc/inst/instance_dao.go | 20.00% | 4 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
As pointed out in https://github.com/vitessio/vitess/issues/16828, it would be a good idea to also publish the number of errant GTIDs in each tablet as a metric. This PR accomplishes this goal.
The new metric looks like -
Related Issue(s)
Checklist
Deployment Notes