Page MenuHomeDevCentral

Add sudo for nasqueron-dev-docker group
ClosedPublic

Authored by DorianWinty on May 23 2022, 17:30.
Tags
None
Referenced Files
F2747658: D2697.id6834.diff
Fri, Mar 29, 06:35
Unknown Object (File)
Sat, Mar 16, 12:02
Unknown Object (File)
Wed, Mar 13, 20:25
Unknown Object (File)
Wed, Mar 13, 19:03
Unknown Object (File)
Wed, Mar 13, 15:05
Unknown Object (File)
Tue, Mar 12, 05:43
Unknown Object (File)
Tue, Mar 12, 05:16
Unknown Object (File)
Tue, Mar 12, 01:24
Subscribers
None

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

DorianWinty created this revision.
dereckson retitled this revision from Add sudo for nasqueron-dev-docker user to Add sudo for nasqueron-dev-docker group.May 23 2022, 17:32
dereckson edited the summary of this revision. (Show Details)
dereckson edited the test plan for this revision. (Show Details)
roles/core/sudo/files/nasqueron-dev-docker
7 ↗(On Diff #6833)

roles/core is the role with content for every server.

Here, file should be in roles/paas-docker/devel/files/

Generally, if the configuration file belongs to roles/<role>/<unit>, it should be in roles/<role>/<unit>/files (and we need to create the folder files when not already existing).

DorianWinty marked an inline comment as done.
DorianWinty edited the summary of this revision. (Show Details)

correct

roles/paas-docker/devel/account.sls
7

?!

We can probably delete the description line.

14

We can drop that one too, as it's trivial enough not to need explanation.

20

Path to edit here too with the new location.

DorianWinty marked 3 inline comments as done.

correct

This revision is now accepted and ready to land.May 23 2022, 18:14
This revision was automatically updated to reflect the committed changes.