Closed vitess-bot[bot] closed 1 week 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 50.00000%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 66.42%. Comparing base (
3444a1d
) to head (7fee093
). Report is 1 commits behind head on release-20.0.
Files with missing lines | Patch % | Lines |
---|---|---|
go/vt/topo/zk2topo/zk_conn.go | 50.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
This is a backport of #17136