Skip to content

BRE2-976: allow setting of launchable parameters#425

Merged
drewmalin merged 1 commit into
mainfrom
dm/launchableparms
Jul 22, 2026
Merged

BRE2-976: allow setting of launchable parameters#425
drewmalin merged 1 commit into
mainfrom
dm/launchableparms

Conversation

@drewmalin

@drewmalin drewmalin commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Allow launchable parameters to be set:

Missing parms:

❯ ./brev create -l env-3Ge8dxSYkYLewLVDsNhKAmEiVDD
Deploying launchable: "fancy parms"
Instance type: n2d-highcpu-2
Storage: 256
Build mode: VM
Required Parameters:
    B
    D    (allowed: aaa, bbb, ccc)
Optional Parameters:
    A    (default: default)
    C    (allowed: 123, 456, 789; default: 456)

invalid launchable parameters:
  - missing required parameter "B"; provide --param B=VALUE
  - missing required parameter "D"; provide --param D=VALUE

Invalid value:

❯ ./brev create -l env-3Ge8dxSYkYLewLVDsNhKAmEiVDD --param B=hi --param D=a
Deploying launchable: "fancy parms"
Instance type: n2d-highcpu-2
Storage: 256
Build mode: VM
Required Parameters:
    B
    D    (allowed: aaa, bbb, ccc)
Optional Parameters:
    A    (default: default)
    C    (allowed: 123, 456, 789; default: 456)

invalid launchable parameters:
  - invalid value "a" for "D" (allowed: aaa, bbb, ccc)

Unknown parameter:

❯ ./brev create -l env-3Ge8dxSYkYLewLVDsNhKAmEiVDD --param B=hi --param D=aaa --param E=aaa
Deploying launchable: "fancy parms"
Instance type: n2d-highcpu-2
Storage: 256
Build mode: VM
Required Parameters:
    B
    D    (allowed: aaa, bbb, ccc)
Optional Parameters:
    A    (default: default)
    C    (allowed: 123, 456, 789; default: 456)

invalid launchable parameters:
  - unknown parameter "E"

@drewmalin
drewmalin requested a review from a team as a code owner July 22, 2026 23:04
@drewmalin
drewmalin merged commit 2a7f53b into main Jul 22, 2026
9 checks passed
@drewmalin
drewmalin deleted the dm/launchableparms branch July 22, 2026 23:57
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