Bug #6610

[API] [Webshell] shell accounts should be created even if the user has not added an SSH key

Added by Ward Vandewege over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Start date:
07/15/2015
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Subtasks

Task #6633: Review 6610-login-without-ssh-keyResolvedTom Clegg

Associated revisions

Revision 97374cec
Added by Tom Clegg over 6 years ago

Merge branch '6610-login-without-ssh-key' closes #6610

History

#1 Updated by Tom Clegg over 6 years ago

  • Target version changed from Bug Triage to 2015-07-22 sprint

#2 Updated by Tom Clegg over 6 years ago

  • Status changed from New to In Progress
  • Story points set to 0.5

#3 Updated by Tom Clegg over 6 years ago

  • Subject changed from [Shell] shell accounts do not appear to be created unless there is an ssh key for the user to [API] [Webshell] shell accounts should be created even if the user has not added an SSH key
  • Category set to API
  • Assigned To set to Tom Clegg

#4 Updated by Peter Amstutz over 6 years ago

Does it make sense to include VMs without ssh keys even when webshell is not enabled in the workbench configuration?

#5 Updated by Peter Amstutz over 6 years ago

Peter Amstutz wrote:

Does it make sense to include VMs without ssh keys even when webshell is not enabled in the workbench configuration?

Answer: yes, because on the API server side it doesn't know anything about webshell.

Here's a better comment:

On the workbench side, if webshell is turned off and you don't have any public keys associated with any of the logins, then there should be a warning telling you to upload an ssh key. However if you feel that's out of the scope of the ticket, forget about it.

#6 Updated by Peter Amstutz over 6 years ago

Disregard previous comment, misunderstood the purpose of this change. LGTM.

#7 Updated by Tom Clegg over 6 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:97374cec874aaaaeb92eeb962bf580bdba199be9.

Also available in: Atom PDF