Skip to content

Add pscale database throttler show/update - #1344

Merged
no-itsbackpack merged 1 commit into
mainfrom
feat/database-throttler
Aug 17, 2026
Merged

Add pscale database throttler show/update#1344
no-itsbackpack merged 1 commit into
mainfrom
feat/database-throttler

Conversation

@no-itsbackpack

@no-itsbackpack no-itsbackpack commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add Databases.GetThrottler / UpdateThrottler API client for database-level Vitess migration throttler (…/databases/{db}/throttler).
  • Add pscale database throttler show|update with --ratio or --configuration keyspace=ratio (mutually exclusive), Vitess/MySQL-only.
  • Document usage in AGENTS.md (distinct from per-DR and vtctld throttlers).

Test plan

  • go test ./internal/planetscale/ -run 'TestDatabases_(Get|Update)Throttler'
  • go test ./internal/cmd/database/ -run TestDatabase_Throttler
  • golangci-lint clean on touched packages
  • Live local: database throttler show main --org big-bang returns keyspaces/configs
  • Live local: Postgres (eagle) and Neki (cat) rejected with Vitess-only error
  • Confirm --ratio + --configuration together is rejected
  • CI green

Expose database-level migration throttler defaults (distinct from
per-DR and vtctld throttlers) and reject non-MySQL databases in the CLI.

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack requested a review from a team as a code owner August 17, 2026 21:01

@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. Approved; Cursor Bugbot completed with no findings, and the Security Agent check was not present. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: cli-reviewer

@no-itsbackpack
no-itsbackpack merged commit d3f4a7a into main Aug 17, 2026
5 checks passed
@no-itsbackpack
no-itsbackpack deleted the feat/database-throttler branch August 17, 2026 21:37
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