Giuseppe Lavagetto has submitted this change and it was merged.
Change subject: Gerrit: Remove $extra_groups from replicationdest, nothing uses
it
......................................................................
Gerrit: Remove $extra_groups from replicationdest, nothing uses it
Change-Id: I90d5aa0c5c0c4546686a517d45848f977f5cb006
---
M modules/gerrit/manifests/replicationdest.pp
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
Giuseppe Lavagetto: Looks good to me, approved
QChris: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/modules/gerrit/manifests/replicationdest.pp
b/modules/gerrit/manifests/replicationdest.pp
index 6d8af67..9612b94 100644
--- a/modules/gerrit/manifests/replicationdest.pp
+++ b/modules/gerrit/manifests/replicationdest.pp
@@ -1,6 +1,6 @@
# Setup the `gerritslave` account on any host that wants to receive
# replication. See role::gerrit::production::replicationdest
-class gerrit::replicationdest( $ssh_key, $extra_groups = [], $slaveuser =
'gerritslave' ) {
+class gerrit::replicationdest( $ssh_key, $slaveuser = 'gerritslave' ) {
group { $slaveuser:
ensure => present,
@@ -10,7 +10,6 @@
user { $slaveuser:
name => $slaveuser,
- groups => $extra_groups,
shell => '/bin/bash',
managehome => true,
system => true,
--
To view, visit https://gerrit.wikimedia.org/r/223169
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I90d5aa0c5c0c4546686a517d45848f977f5cb006
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Chad <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: QChris <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
