Faidon has submitted this change and it was merged.

Change subject: fix jenkins::group class
......................................................................


fix jenkins::group class

It had a subclass and was thus defined as jenkins::group::jenkins.
Also make the group a system group.

Change-Id: I474c54e6142762c9836f057f18c71d3e9f8578fa
---
M modules/jenkins/manifests/group.pp
1 file changed, 5 insertions(+), 8 deletions(-)

Approvals:
  Hashar: Looks good to me, but someone else must approve
  Faidon: Verified; Looks good to me, approved



diff --git a/modules/jenkins/manifests/group.pp 
b/modules/jenkins/manifests/group.pp
index e74a044..0cd150d 100644
--- a/modules/jenkins/manifests/group.pp
+++ b/modules/jenkins/manifests/group.pp
@@ -1,11 +1,8 @@
 class jenkins::group {
-
-  class jenkins {
-    group { "jenkins":
-      name      => "jenkins",
-      ensure    => present,
-      allowdupe => false;
-    }
+  group { 'jenkins':
+    ensure    => present,
+    name      => 'jenkins',
+    system    => true,
+    allowdupe => false,
   }
-
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I474c54e6142762c9836f057f18c71d3e9f8578fa
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>
Gerrit-Reviewer: Faidon <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>

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

Reply via email to