Page MenuHomePhabricator

Do not require people to be explicitly added to the bastiononly group
Closed, ResolvedPublic

Description

Currently people who need access to stat* or other boxes need to be added manually to the particular target group and also to the bastiononly group, and a lot of times people trip up on the latter. Why is this manual addition required? Shouldn't giving access to any other group automatically imply access to bastion? If not why not?

Event Timeline

yuvipanda raised the priority of this task from to Needs Triage.
yuvipanda updated the task description. (Show Details)
yuvipanda added a project: acl*sre-team.
yuvipanda subscribed.

Change 227327 had a related patch set uploaded (by Alex Monk):
Add all groups to general bastions, mostly empty bastiononly group

https://gerrit.wikimedia.org/r/227327

The main challenge with this is ensuring that while (almost?) all groups imply bastion access, only ops gets their sudo rules applied on bastions. Most users shouldn't have root access on bastions just because their group is intended to give them full sudo access on other machines.

Dzahn triaged this task as Medium priority.Oct 19 2015, 11:22 PM
Dzahn subscribed.

Change 244471 had a related patch set uploaded (by Alex Monk):
admin: allow all active users to be applied

https://gerrit.wikimedia.org/r/244471

Change 227327 abandoned by Alex Monk:
Add all groups to general bastions, mostly empty bastiononly group

Reason:
in favour of I8f984e51

https://gerrit.wikimedia.org/r/227327

Change 301149 had a related patch set uploaded (by Alex Monk):
Replace manually-maintained bastiononly group with the new 'all-users'

https://gerrit.wikimedia.org/r/301149

Change 301149 merged by Dzahn:
Replace manually-maintained bastiononly group with the new 'all-users'

https://gerrit.wikimedia.org/r/301149

Merged that one after it had a couple +1, checked on bastions. It really just created the "dkg" user as expected. No other change. And now there is no more "bastiononly" group.

I think that resolves the ticket.

Change 244471 abandoned by Rush:
admin: allow all active users to be applied

Reason:
hooray, already done

https://gerrit.wikimedia.org/r/244471