Alexandros Kosiaris has submitted this change and it was merged. Change subject: puppetmaster: cleanups in gitsync ......................................................................
puppetmaster: cleanups in gitsync Specify owner, group for file resources Remove redundant dependency Change-Id: I567a60c8355209e6a464529a5d3879011c3a9c01 --- M modules/puppetmaster/manifests/gitsync.pp 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: jenkins-bot: Verified Dzahn: Looks good to me, approved diff --git a/modules/puppetmaster/manifests/gitsync.pp b/modules/puppetmaster/manifests/gitsync.pp index b46e2fe..fff429f 100644 --- a/modules/puppetmaster/manifests/gitsync.pp +++ b/modules/puppetmaster/manifests/gitsync.pp @@ -16,6 +16,8 @@ file { '/usr/local/bin/git-sync-upstream': ensure => present, content => template('puppetmaster/git-sync-upstream.erb'), + owner => 'root', + group => 'root', mode => '0555', } @@ -30,7 +32,8 @@ file { '/etc/logrotate.d/git-sync-upstream': ensure => present, source => 'puppet:///modules/puppetmaster/git-sync-upstream.logrotate', + owner => 'root', + group => 'root', mode => '0444', - require => Cron['rebase_operations_puppet'], } } -- To view, visit https://gerrit.wikimedia.org/r/209263 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I567a60c8355209e6a464529a5d3879011c3a9c01 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Alexandros Kosiaris <akosia...@wikimedia.org> Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org> Gerrit-Reviewer: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits