Skip to content

Make --accelerator flag optional for ssh connect#4947

Merged
anton-107 merged 2 commits intomainfrom
ssh-connect-optional-accelerator
Apr 14, 2026
Merged

Make --accelerator flag optional for ssh connect#4947
anton-107 merged 2 commits intomainfrom
ssh-connect-optional-accelerator

Conversation

@anton-107
Copy link
Copy Markdown
Contributor

@anton-107 anton-107 commented Apr 13, 2026

Summary

  • Remove the validation requiring --accelerator when using --name (serverless mode) in ssh connect
  • Add a proactive yellow warning at connect time when --accelerator is omitted, informing users that serverless CPU is in private preview
  • Add a reactive hint appended to the error message when the server fails to start without an accelerator

Test plan

  • Unit tests pass (go test ./experimental/ssh/internal/client/)
  • Build succeeds (make build)
  • Manual test: ./cli ssh connect --name test-conn --profile p shows warning and submits job without --accelerator
  • Existing --accelerator usage is unaffected (validation for valid values still in place)

This pull request was AI-assisted by Isaac.

Serverless CPU compute (without a GPU accelerator) is now supported in
private preview. Remove the validation that required `--accelerator`
when using `--name`, and add a warning for users who may not be enrolled.

Co-authored-by: Isaac
Co-authored-by: Isaac
@anton-107 anton-107 requested review from ilia-db and pietern April 13, 2026 15:22
@anton-107 anton-107 added this pull request to the merge queue Apr 14, 2026
Merged via the queue into main with commit 4adee6a Apr 14, 2026
31 checks passed
@anton-107 anton-107 deleted the ssh-connect-optional-accelerator branch April 14, 2026 07:34
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