Dzahn has submitted this change and it was merged.

Change subject: modules/coredb_mysql/ sans systemuser
......................................................................


modules/coredb_mysql/ sans systemuser

we want to get rid of generic::systemuser
and replace it with the regular puppet user type
again to remove a layer of abstraction that isn't
really needed

Change-Id: I2f52bf76bcd96849f4c9f2b0df035ca7ec2bba85
---
M modules/coredb_mysql/manifests/base.pp
1 file changed, 11 insertions(+), 4 deletions(-)

Approvals:
  Springle: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/modules/coredb_mysql/manifests/base.pp 
b/modules/coredb_mysql/manifests/base.pp
index 387619b..487fb81 100644
--- a/modules/coredb_mysql/manifests/base.pp
+++ b/modules/coredb_mysql/manifests/base.pp
@@ -2,10 +2,17 @@
 class coredb_mysql::base {
     require coredb_mysql::packages
 
-    generic::systemuser { 'mysql':
-        name  => 'mysql',
-        shell => '/bin/sh',
-        home  => '/home/mysql',
+    group { 'mysql':
+        ensure => present,
+        name   => 'mysql',
+        system => true,
+    }
+
+    user { 'mysql':
+        shell      => '/bin/sh',
+        home       => '/home/mysql',
+        managehome => true,
+        system     => true,
     }
 
     file { '/a/sqldata':

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2f52bf76bcd96849f4c9f2b0df035ca7ec2bba85
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Rush <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Springle <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to