Alexandros Kosiaris has submitted this change and it was merged.

Change subject: Specify mode for nagios_hostgroup and nagios::servicegroup
......................................................................


Specify mode for nagios_hostgroup and nagios::servicegroup

Otherwise the files get generated with an overly restrictive set of only
read/write for the owner, disallowing effectively icinga from starting

Change-Id: Ide71b2fea6db936cdfa6898165380716ec4e5d68
---
M modules/monitoring/manifests/group.pp
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Alexandros Kosiaris: Verified; Looks good to me, approved



diff --git a/modules/monitoring/manifests/group.pp 
b/modules/monitoring/manifests/group.pp
index d13094b..08b1df8 100644
--- a/modules/monitoring/manifests/group.pp
+++ b/modules/monitoring/manifests/group.pp
@@ -20,6 +20,7 @@
         ensure         => $ensure,
         target         => "${config_dir}/puppet_hostgroups.cfg",
         hostgroup_name => $title,
+        mode           => '0444',
         alias          => $description,
     }
 
@@ -28,6 +29,7 @@
         ensure            => $ensure,
         target            => "${config_dir}/puppet_servicegroups.cfg",
         servicegroup_name => $title,
+        mode              => '0444',
         alias             => $description,
     }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ide71b2fea6db936cdfa6898165380716ec4e5d68
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>

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

Reply via email to