Faidon has submitted this change and it was merged.

Change subject: ceph: actually user ceph::key's owner/group/mode
......................................................................


ceph: actually user ceph::key's owner/group/mode

Parameters were added but not really used. Doh!

Change-Id: I048aa858a875ccb8826c714f2ca3890434b639cd
---
M modules/ceph/manifests/key.pp
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Faidon: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ceph/manifests/key.pp b/modules/ceph/manifests/key.pp
index 0d98da5..b177d50 100644
--- a/modules/ceph/manifests/key.pp
+++ b/modules/ceph/manifests/key.pp
@@ -45,9 +45,9 @@
     # ping-pong trickery to securely do permissions, puppet has no umask on 
exec
     file { $keyring:
         ensure  => $ensure,
-        mode    => '0600',
-        owner   => 'root',
-        group   => 'root',
+        owner   => $owner,
+        group   => $group,
+        mode    => $mode,
         backup  => false,
     }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I048aa858a875ccb8826c714f2ca3890434b639cd
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon <fai...@wikimedia.org>
Gerrit-Reviewer: Faidon <fai...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to