Closed vitess-bot[bot] closed 1 week ago
Hello @mattlord, there are conflicts in this backport.
Please address them in order to merge this Pull Request. You can execute the snippet below to reset your branch and resolve the conflict manually.
Make sure you replace origin
by the name of the vitessio/vitess remote
git fetch --all
gh pr checkout 17176 -R vitessio/vitess
git reset --hard origin/release-21.0
git cherry-pick -m 1 31b956b637373ca8b14249e6f854d5eadc271775
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 60.00000%
with 10 lines
in your changes missing coverage. Please review.
Project coverage is 67.51%. Comparing base (
5fbbb8e
) to head (66711ea
). Report is 2 commits behind head on release-21.0.
Files with missing lines | Patch % | Lines |
---|---|---|
go/vt/vtctl/workflow/materializer.go | 64.70% | 6 Missing :warning: |
go/vt/vtctl/workflow/utils.go | 50.00% | 4 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 #17174