Mark Bergsma has submitted this change and it was merged.
Change subject: Revert "Deprecate $name param to systemuser in favor of $title"
......................................................................
Revert "Deprecate $name param to systemuser in favor of $title"
This reverts commit 4f08671606fb07d48834fe7179c19247ddd6f72f
Change-Id: I782c891f53a5b28aa4e3465c984cf3e8ee010159
---
M manifests/generic-definitions.pp
M manifests/gerrit.pp
2 files changed, 7 insertions(+), 6 deletions(-)
Approvals:
Mark Bergsma: Verified; Looks good to me, approved
diff --git a/manifests/generic-definitions.pp b/manifests/generic-definitions.pp
index 7ea5fbd..702ba99 100644
--- a/manifests/generic-definitions.pp
+++ b/manifests/generic-definitions.pp
@@ -24,22 +24,22 @@
}
# Creates a system username with associated group, random uid/gid, and
/bin/false as shell
-define systemuser($name = undef, $home=undef, $managehome=true,
$shell="/bin/false", $groups=undef, $default_group=$name, $ensure=present) {
- $username = $name ? { undef => $title, default => $name }
+define systemuser($name, $home=undef, $managehome=true, $shell="/bin/false",
$groups=undef, $default_group=$name, $ensure=present) {
+ # FIXME: deprecate $name parameter in favor of just using $title
- if $default_group == $username {
+ if $default_group == $name {
group { $default_group:
name => $default_group,
ensure => present;
}
}
- user { $username:
+ user { $name:
require => Group[$default_group],
- name => $username,
+ name => $name,
gid => $default_group,
home => $home ? {
- undef => "/var/lib/${username}",
+ undef => "/var/lib/${name}",
default => $home
},
managehome => $managehome,
diff --git a/manifests/gerrit.pp b/manifests/gerrit.pp
index 2fb5126..808d344 100644
--- a/manifests/gerrit.pp
+++ b/manifests/gerrit.pp
@@ -404,6 +404,7 @@
# replication. See role::gerrit::production::replicationdest
class gerrit::replicationdest( $sshkey, $extra_groups = undef, $slaveuser =
"gerritslave" ) {
systemuser { $slaveuser:
+ name => $slaveuser,
groups => $extra_groups,
shell => "/bin/bash";
}
--
To view, visit https://gerrit.wikimedia.org/r/67417
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I782c891f53a5b28aa4e3465c984cf3e8ee010159
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mark Bergsma <[email protected]>
Gerrit-Reviewer: Mark Bergsma <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
