Tim Landscheidt has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/324729

Change subject: labstore: Use explicit groups for file resources
......................................................................

labstore: Use explicit groups for file resources

Currently, the file resources for /usr/local/sbin/block-for-export and
/usr/local/sbin/nfs-mount-manager have no group attributes causing
Puppet to take the (numerical) group the source file has on the
puppetmaster, i. e. gitpuppet in the case of a newly launched
puppetmaster.

This change adds explicit group attributes for those file resources.

Bug: T152095
Change-Id: I94d170931870be88802a299348e1d0fe962108bd
---
M modules/labstore/manifests/nfs_mount.pp
1 file changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/29/324729/1

diff --git a/modules/labstore/manifests/nfs_mount.pp 
b/modules/labstore/manifests/nfs_mount.pp
index 05da537..386878f 100644
--- a/modules/labstore/manifests/nfs_mount.pp
+++ b/modules/labstore/manifests/nfs_mount.pp
@@ -99,7 +99,8 @@
     if !defined(File['/usr/local/sbin/nfs-mount-manager']) {
         file { '/usr/local/sbin/nfs-mount-manager':
             ensure => present,
-            owner  => root,
+            owner  => 'root',
+            group  => 'root',
             mode   => '0655',
             source => 'puppet:///modules/labstore/nfs-mount-manager',
         }
@@ -134,7 +135,8 @@
                 # This script will block until the NFS volume is available
                 file { '/usr/local/sbin/block-for-export':
                     ensure => present,
-                    owner  => root,
+                    owner  => 'root',
+                    group  => 'root',
                     mode   => '0555',
                     source => 'puppet:///modules/labstore/block-for-export',
                 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I94d170931870be88802a299348e1d0fe962108bd
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Tim Landscheidt <t...@tim-landscheidt.de>

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

Reply via email to