Bug #16689

[arvados-login-sync] user groups are not being updated

Added by Javier Bértoli about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Login
Target version:
Start date:
09/03/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

As an admin, when a user is allowed shell access in workbench, the user is created in shell nodes by arvados-login-sync.

When the user's groups are modified after it was created, those changes are ignored by arvados-login-sync.


Subtasks

Task #16695: review 16689-shell-sync-groupsResolvedWard Vandewege

Associated revisions

Revision 8f8329e7
Added by Peter Amstutz about 1 year ago

Merge branch '16689-shell-sync-groups' refs #16689

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

Revision 1771152d (diff)
Added by Peter Amstutz about 1 year ago

Fix test failure, refs #16689

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

History

#1 Updated by Peter Amstutz about 1 year ago

  • Assigned To set to Javier Bértoli

#2 Updated by Peter Amstutz about 1 year ago

  • Target version changed from 2020-08-26 Sprint to 2020-09-09 Sprint

#4 Updated by Peter Amstutz about 1 year ago

  • Assigned To changed from Javier Bértoli to Peter Amstutz

#5 Updated by Peter Amstutz about 1 year ago

16689-shell-sync-groups @ 3241db378301b3d507e928776d5e3e511c38a998

Based on the 16803-shell-sync-tokens branch just to avoid potential merge conflicts.

  • Add/remove groups as needed
  • Stop sending output of adduser to devnull because that made debugging annoying.

#6 Updated by Ward Vandewege about 1 year ago

Peter Amstutz wrote:

16689-shell-sync-groups @ 3241db378301b3d507e928776d5e3e511c38a998

Based on the 16803-shell-sync-tokens branch just to avoid potential merge conflicts.

  • Add/remove groups as needed
  • Stop sending output of adduser to devnull because that made debugging annoying.

LGTM, thanks.

#7 Updated by Peter Amstutz about 1 year ago

  • Status changed from New to Resolved

#8 Updated by Peter Amstutz about 1 year ago

  • Release set to 25

Also available in: Atom PDF