Akosiaris has submitted this change and it was merged.

Change subject: Fix owner/perms ganglia_new::monitor::aggregator
......................................................................


Fix owner/perms ganglia_new::monitor::aggregator

As always, not specified so using puppetmaster's filesystem perms

Change-Id: Ibf8c22bc7f4d12c3a15a17e3fa1c63a7532a2b5e
---
M modules/ganglia_new/manifests/monitor/aggregator.pp
1 file changed, 10 insertions(+), 6 deletions(-)

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



diff --git a/modules/ganglia_new/manifests/monitor/aggregator.pp 
b/modules/ganglia_new/manifests/monitor/aggregator.pp
index 4d4cd10..9769a48 100644
--- a/modules/ganglia_new/manifests/monitor/aggregator.pp
+++ b/modules/ganglia_new/manifests/monitor/aggregator.pp
@@ -9,13 +9,17 @@
                        ensure => directory,
                        mode => 0555;
                "/etc/init/ganglia-monitor-aggregator.conf":
-                       source => 
"puppet:///modules/$module_name/upstart/ganglia-monitor-aggregator.conf",
-                       before => Service["ganglia-monitor-aggregator"],
-                       mode => 0444;
+                       owner  => 'root',
+                       group  => 'root',
+                       mode   => '0444',
+                       source => 
"puppet:///modules/${module_name}/upstart/ganglia-monitor-aggregator.conf",
+                       before => Service['ganglia-monitor-aggregator'];
                "/etc/init/ganglia-monitor-aggregator-instance.conf":
-                       source => 
"puppet:///modules/$module_name/upstart/ganglia-monitor-aggregator-instance.conf",
-                       before => Service["ganglia-monitor-aggregator"],
-                       mode => 0444;
+                       owner  => 'root',
+                       group  => 'root',
+                       mode   => '0444',
+                       source => 
"puppet:///modules/${module_name}/upstart/ganglia-monitor-aggregator-instance.conf",
+                       before => Service['ganglia-monitor-aggregator'];
        }
 
        generic::upstart_job { "ganglia-monitor-aggregator-instance": }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibf8c22bc7f4d12c3a15a17e3fa1c63a7532a2b5e
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Akosiaris <[email protected]>
Gerrit-Reviewer: Akosiaris <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to