Gehel has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/370530 )

Change subject: discovery-stats user should be a member of wikidev group
......................................................................

discovery-stats user should be a member of wikidev group

Since Admin::Groupmembers seems to check only secondary groups, defining the
primariy group of discovery-stats as wikidev should make things work as
expected.

Bug: T172740
Change-Id: I95e6f4f91a7b9969268efaa22f84eea38bcd7d52
---
M modules/statistics/manifests/discovery.pp
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/30/370530/1

diff --git a/modules/statistics/manifests/discovery.pp 
b/modules/statistics/manifests/discovery.pp
index 94dec47..fcc5474 100644
--- a/modules/statistics/manifests/discovery.pp
+++ b/modules/statistics/manifests/discovery.pp
@@ -22,7 +22,7 @@
         shell      => '/bin/bash',
         managehome => false,
         system     => true,
-        groups     => $group,
+        gid        => $group,
     }
 
     # This file will render at

-- 
To view, visit https://gerrit.wikimedia.org/r/370530
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I95e6f4f91a7b9969268efaa22f84eea38bcd7d52
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Gehel <guillaume.leder...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to