Paths

Table of Contentst

Differential D2697

Add sudo for nasqueron-dev-docker group
ClosedPublic
Actions

Authored by DorianWinty on May 23 2022, 17:30.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 21, 15:15

	Unknown Object (File)
	Wed, Dec 18, 06:01

	Unknown Object (File)
	Thu, Dec 12, 05:13

	Unknown Object (File)
	Thu, Dec 5, 01:52

	Unknown Object (File)
	Mon, Dec 2, 17:47

	Unknown Object (File)
	Mon, Nov 25, 16:03

	Unknown Object (File)
	Fri, Nov 22, 17:54

	Unknown Object (File)
	Nov 22 2024, 11:35

Subscribers

None

Details

Reviewers

Maniphest Tasks

T1724: Split Docker engines between production (docker-001) and dev (dwellers)

Commits

rOPSe7ff9d92481c: Add sudo for nasqueron-dev-docker group

Summary

Test Plan

Deploy on Dwellers.

sudo whoami

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

DorianWinty requested review of this revision.May 23 2022, 17:30

DorianWinty created this revision.

Harbormaster completed remote builds in B4251: Diff 6833.May 23 2022, 17:30

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)

dereckson added a task: T1724: Split Docker engines between production (docker-001) and dev (dwellers).

dereckson added inline comments.May 23 2022, 17:35

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).

correct

Harbormaster completed remote builds in B4252: Diff 6834.May 23 2022, 17:47

a

Harbormaster completed remote builds in B4253: Diff 6835.May 23 2022, 17:49

dereckson added inline comments.May 23 2022, 17:57

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.

correct

Harbormaster completed remote builds in B4254: Diff 6836.May 23 2022, 18:02

r

Harbormaster completed remote builds in B4255: Diff 6837.May 23 2022, 18:08

dereckson accepted this revision.May 23 2022, 18:14

This revision is now accepted and ready to land.May 23 2022, 18:14

Closed by commit rOPSe7ff9d92481c: Add sudo for nasqueron-dev-docker group (authored by DorianWinty). · Explain WhyMay 23 2022, 18:14

This revision was automatically updated to reflect the committed changes.

DorianWinty added a commit: rOPSe7ff9d92481c: Add sudo for nasqueron-dev-docker group.

Revision Contents
Changeset List

Path

Size

roles/

paas-docker/

devel/

16 lines

files/

nasqueron-dev-docker.sudoers

17 lines

Diff 6838

roles/paas-docker/devel/account.sls

Loading...

roles/paas-docker/devel/files/nasqueron-dev-docker.sudoers

Loading...