Skip to content

add ssh remove-key command#264

Open
vavo wants to merge 1 commit intorunpod:mainfrom
vavo:notrius/issue-258-ssh-remove-key
Open

add ssh remove-key command#264
vavo wants to merge 1 commit intorunpod:mainfrom
vavo:notrius/issue-258-ssh-remove-key

Conversation

@vavo
Copy link
Copy Markdown

@vavo vavo commented Apr 8, 2026

E-258: add ssh remove-key command

  • resolves ssh: add remove-key command to manage SSH keys #258 by adding runpodctl ssh remove-key with --name and --fingerprint.
  • maintains full legacy command compatibility (stderr warnings only, stdout unchanged).
  • keeps runpodctl as the primary binary; drop-in upgrade for existing users.
  • expands ssh key management coverage via the existing graphql updateUserSettings path, including ambiguity handling for duplicate key names.
  • updates ssh command docs and adds unit coverage for command wiring and graphql removal behavior.
  • tests: full unit suite passed locally; live e2e is still blocked in this workspace because no runpod api key is configured.

Test plan:

  • go test ./...
  • go test -tags e2e ./e2e

@vavo vavo mentioned this pull request Apr 8, 2026
2 tasks
@vavo vavo marked this pull request as ready for review April 8, 2026 22:00
@promptless
Copy link
Copy Markdown

promptless bot commented Apr 8, 2026

Promptless prepared a documentation update related to this change.

Triggered by runpod/runpodctl#264

Added documentation for the new runpodctl ssh remove-key command, including usage examples and flag descriptions for --name and --fingerprint.

Review at https://app.gopromptless.ai/suggestions/9e287ca0-cd90-4e51-b161-1981e00a0c48

Copy link
Copy Markdown
Author

vavo commented Apr 8, 2026

live verification completed with a real Runpod API key.

verified flow:

  • build runpodctl
  • ssh add-key with disposable key A
  • ssh add-key with disposable key B
  • ssh list-keys to resolve key B fingerprint
  • ssh remove-key --name for key A
  • ssh remove-key --fingerprint for key B
  • ssh list-keys to confirm both keys were removed

result: pass

cleanup was performed immediately after verification.

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.

ssh: add remove-key command to manage SSH keys

1 participant