Skip to content

feat(BREV-9100): add --region flag to brev create and brev search#393

Open
hakhandelwal11 wants to merge 2 commits into
mainfrom
BREV-9100/add-region-flag-to-brev-create
Open

feat(BREV-9100): add --region flag to brev create and brev search#393
hakhandelwal11 wants to merge 2 commits into
mainfrom
BREV-9100/add-region-flag-to-brev-create

Conversation

@hakhandelwal11

@hakhandelwal11 hakhandelwal11 commented May 19, 2026

Copy link
Copy Markdown
Contributor
  • Adds -r, --region flag to brev create for targeting a specific region/location when launching an instance. The value flows through to cwOptions.Location on the create request.
  • Adds -r, --region flag to brev search (gpu and cpu) for filtering catalog results by region. Matching is case-insensitive substring against AvailableLocations.
  • Exposes available_regions in the brev search --json payload so users can discover valid regions.

@hakhandelwal11 hakhandelwal11 self-assigned this May 19, 2026
@hakhandelwal11 hakhandelwal11 requested a review from a team as a code owner May 19, 2026 11:27
@hakhandelwal11 hakhandelwal11 marked this pull request as draft May 19, 2026 11:27
Comment thread pkg/cmd/gpusearch/gpusearch.go Outdated
@theFong

theFong commented May 19, 2026

Copy link
Copy Markdown
Member

what QA have we done?

@hakhandelwal11 hakhandelwal11 marked this pull request as ready for review May 20, 2026 05:54
@hakhandelwal11 hakhandelwal11 force-pushed the BREV-9100/add-region-flag-to-brev-create branch from b87eab1 to b3a06f5 Compare June 10, 2026 13:35
@hakhandelwal11

Copy link
Copy Markdown
Contributor Author

what QA have we done?

@theFong ,
We did a local build of the CLI and ran through the flows to validate behavior

  • ./brev search gpu --region us-east-1: confirmed exact and partial (substring, case-insensitive) matches against AvailableLocations.
  • ./brev gpu create --region <valid>: instance launches, verified Region lands on the workspace.
  • ./brev gpu create --region <bogus>: hits validateRegionExists and errors out before any API mutation.
  • ./brev gpu create --instance-type <t> --region <r-not-supported-by-t>: hits validateRegion with the "not available for instance type(s)…" error.
  • ./brev gpu create --instance-type <unknown> --region <r>: hits the unknown-type branch.

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