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