Skip to content

Comments

vtctld: add MoveTables CLI commands#1196

Merged
nickvanw merged 4 commits intomainfrom
nick/vtctld-cli-move-tables
Feb 25, 2026
Merged

vtctld: add MoveTables CLI commands#1196
nickvanw merged 4 commits intomainfrom
nick/vtctld-cli-move-tables

Conversation

@nickvanw
Copy link
Contributor

Summary

  • Adds the move-tables subcommand group under pscale branch vtctld
  • Implements create, show, status, switch-traffic, reverse-traffic, cancel, and complete operations
  • All *bool fields use the cmd.Flags().Changed() guard pattern so server defaults are respected when flags aren't explicitly set

Stacked on #1195.

@nickvanw nickvanw requested a review from a team as a code owner February 24, 2026 00:32
@nickvanw nickvanw force-pushed the nick/vtctld-cli-base branch from 4195ce4 to 8172bd9 Compare February 24, 2026 00:36
@nickvanw nickvanw force-pushed the nick/vtctld-cli-move-tables branch from b416b67 to 965e783 Compare February 24, 2026 00:37
Add the move-tables subcommand group with create, show, status,
switch-traffic, reverse-traffic, cancel, and complete operations.
These allow managing MoveTables workflows on Vitess branches through
the CLI.

Signed-off-by: Nick Van Wiggeren <nick@planetscale.com>
@nickvanw nickvanw force-pushed the nick/vtctld-cli-move-tables branch from 7b7644b to d6a3bf2 Compare February 24, 2026 22:35
Create: --cells, --tablet-types, --exclude-tables, --atomic-copy
SwitchTraffic: --max-replication-lag-allowed
ReverseTraffic: --tablet-types, --max-replication-lag-allowed
Base automatically changed from nick/vtctld-cli-base to main February 25, 2026 00:12
nickvanw and others added 2 commits February 24, 2026 16:12
Only send defer_secondary_keys when the flag is explicitly set so server defaults are preserved. Add tests for both omitted and explicit false values.
@nickvanw nickvanw merged commit 46b2e66 into main Feb 25, 2026
2 checks passed
@nickvanw nickvanw deleted the nick/vtctld-cli-move-tables branch February 25, 2026 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants