Faidon Liambotis has submitted this change and it was merged.
Change subject: Remove Class['sudo'] -> User['root']
......................................................................
Remove Class['sudo'] -> User['root']
It doesn't make any sense. Presumably the original author wanted the
exact opposite order with the intention of not getting locked out of a
machine.
Change-Id: Id3cac0c8882a665e786f191ab464e4b7ff459dc3
---
M modules/passwords/manifests/init.pp
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Faidon Liambotis: Verified; Looks good to me, approved
diff --git a/modules/passwords/manifests/init.pp
b/modules/passwords/manifests/init.pp
index f60eff2..f1f1063 100644
--- a/modules/passwords/manifests/init.pp
+++ b/modules/passwords/manifests/init.pp
@@ -7,7 +7,6 @@
class passwords::root {
user { 'root':
password => '!',
- before => Class['sudo'],
}
file {
--
To view, visit https://gerrit.wikimedia.org/r/180768
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id3cac0c8882a665e786f191ab464e4b7ff459dc3
Gerrit-PatchSet: 1
Gerrit-Project: labs/private
Gerrit-Branch: master
Gerrit-Owner: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Faidon Liambotis <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
