From: dtrishkin <dtrishkin@mirantis.com>
Date: Mon, 2 Feb 2015 14:23:51 +0000 (+0300)
Subject: Fix access to ceilometer log directory
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F35%2F2835%2F1;p=openstack-build%2Fceilometer-build.git

Fix access to ceilometer log directory

Change-Id: Ie11f52b872cf7ded3ec76475ae6194854d14537b
Related-bug: #1400240
---

diff --git a/debian/ceilometer-common.postinst b/debian/ceilometer-common.postinst
index 27ca0a2..271dd31 100644
--- a/debian/ceilometer-common.postinst
+++ b/debian/ceilometer-common.postinst
@@ -9,9 +9,8 @@ if [ "$1" = "configure" ]; then
         adduser --system --home /var/lib/ceilometer --ingroup ceilometer --no-create-home --shell /bin/false ceilometer
     fi
 
-    chown -R ceilometer:adm /var/log/ceilometer
-    chmod 0750 /var/log/ceilometer
-    chown -R ceilometer:ceilometer /var/lib/ceilometer /etc/ceilometer
+    chmod 0775 /var/log/ceilometer
+    chown -R ceilometer:ceilometer /var/lib/ceilometer /etc/ceilometer /var/log/ceilometer
 
     if grep -qE "^(sql_)?connection.*sqlite.*" /etc/ceilometer/ceilometer.conf
     then
diff --git a/rpm/SPECS/openstack-ceilometer.spec b/rpm/SPECS/openstack-ceilometer.spec
index a4496e1..2953954 100644
--- a/rpm/SPECS/openstack-ceilometer.spec
+++ b/rpm/SPECS/openstack-ceilometer.spec
@@ -304,7 +304,7 @@ popd
 # Setup directories
 install -d -m 755 %{buildroot}%{_sharedstatedir}/ceilometer
 install -d -m 755 %{buildroot}%{_sharedstatedir}/ceilometer/tmp
-install -d -m 755 %{buildroot}%{_localstatedir}/log/ceilometer
+install -d -m 775 %{buildroot}%{_localstatedir}/log/ceilometer
 
 # Install config files
 install -d -m 755 %{buildroot}%{_sysconfdir}/ceilometer
@@ -502,7 +502,7 @@ fi
 %config(noreplace) %attr(-, root, ceilometer) %{_sysconfdir}/ceilometer/event_definitions.yaml
 %config(noreplace) %{_sysconfdir}/logrotate.d/%{name}
 
-%dir %attr(0755, ceilometer, root) %{_localstatedir}/log/ceilometer
+%dir %attr(0775, ceilometer, ceilometer) %{_localstatedir}/log/ceilometer
 %dir %attr(0755, ceilometer, root) %{_localstatedir}/run/ceilometer
 
 %{_bindir}/ceilometer-agent-ipmi