From: Takashi Kajinami <tkajinam@redhat.com>
Date: Sun, 20 Feb 2022 09:57:46 +0000 (+0900)
Subject: Fix missing mode/owner/group enforcement of polling.yaml
X-Git-Tag: 20.2.0~2^2
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=8ac9135920cd0cca41e3b70ceae33ae63211785a;p=puppet-modules%2Fpuppet-ceilometer.git

Fix missing mode/owner/group enforcement of polling.yaml

... and avoid hard-coding user/group but use the definition in
the params class.

Change-Id: Ia5f1e356902ad544fe7eb17e53b3b274475d40a8
---

diff --git a/manifests/agent/notification.pp b/manifests/agent/notification.pp
index 828674a..3b10c8f 100644
--- a/manifests/agent/notification.pp
+++ b/manifests/agent/notification.pp
@@ -156,7 +156,7 @@ class ceilometer::agent::notification (
       selinux_ignore_defaults => true,
       mode                    => '0640',
       owner                   => 'root',
-      group                   => 'ceilometer',
+      group                   => $::ceilometer::params::group,
       tag                     => 'ceilometer-yamls',
     }
   }
@@ -177,7 +177,7 @@ class ceilometer::agent::notification (
       selinux_ignore_defaults => true,
       mode                    => '0640',
       owner                   => 'root',
-      group                   => 'ceilometer',
+      group                   => $::ceilometer::params::group,
       tag                     => 'ceilometer-yamls',
     }
   }
diff --git a/manifests/agent/polling.pp b/manifests/agent/polling.pp
index b8ba7e0..734ce95 100644
--- a/manifests/agent/polling.pp
+++ b/manifests/agent/polling.pp
@@ -186,6 +186,9 @@ class ceilometer::agent::polling (
       ensure                  => present,
       path                    => $::ceilometer::params::polling,
       content                 => $polling_content,
+      mode                    => '0640',
+      owner                   => 'root',
+      group                   => $::ceilometer::params::group,
       selinux_ignore_defaults => true,
       tag                     => 'ceilometer-yamls',
     }