Skip to content

Add external keyspace creation - #1428

Merged
no-itsbackpack merged 4 commits into
mainfrom
egomez/keyspace-create-external
Sep 18, 2026
Merged

no-itsbackpack merged 4 commits into
mainfrom
egomez/keyspace-create-external

Conversation

@no-itsbackpack

Copy link
Copy Markdown
Contributor

Adds pscale keyspace create-external with datasource linting, optional cluster sizing, and wait support. pscale size cluster list --external lists only valid external keyspace sizes, and cluster-size completion uses the same filtered list.

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack marked this pull request as ready for review September 18, 2026 17:52
@no-itsbackpack
no-itsbackpack requested a review from a team as a code owner September 18, 2026 17:52

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c13403a. Configure here.

Comment thread internal/cmd/keyspace/create_external.go

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk: medium. Left a non-blocking comment and did not approve because Cursor Bugbot was present but completed as skipped, so that required review signal did not finish successfully. Human review is needed; no reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: cli-reviewer

no-itsbackpack and others added 3 commits September 18, 2026 13:49
The dry-run only failed on connection errors, so a source with per-table
lint errors was reported as compatible. Type lint_errors so the response
carries them and list them in human output.

Co-authored-by: Cursor <cursoragent@cursor.com>
Replace pscale workflow as the Vitess table-move path in the embedded
agent guide. Document keyspace create-external and branch vtctld
move-tables, including next_steps and approval gates.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keep move-tables as the preferred path without telling agents the old
command is forbidden.

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack merged commit 6aded38 into main Sep 18, 2026
4 checks passed
@no-itsbackpack
no-itsbackpack deleted the egomez/keyspace-create-external branch September 18, 2026 19:24
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.

3 participants