Open timvaillancourt opened 1 day 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 5.83333%
with 113 lines
in your changes missing coverage. Please review.
Project coverage is 69.40%. Comparing base (
2e2b223
) to head (03a26d1
). Report is 3 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
As discussed in https://github.com/vitessio/vitess/issues/16377, this PR implements a new
vtctld
and correspondingvttablet
RPC to allow the tags of a tablet to be changedChangeTabletTags
ChangeTags
The new RPC supports both replacing and merging the existing tags. This merging happens on the tablet under a lock
CLI
Related Issue(s)
https://github.com/vitessio/vitess/issues/16377
Checklist
Deployment Notes