From a970fc0cd7ec4ddb806273b0d7279c52587452a4 Mon Sep 17 00:00:00 2001
From: Lalatendu Mohanty <lmohanty@redhat.com>
Date: Thu, 30 Oct 2014 01:11:02 +0530
Subject: logrotate: gluster logrotate config should not be global

Issue : Previously glusterfs logrotate config file pollutes
global config. So moved the directives inside the curly braces,
 so they don't pollute the global config state.

Change-Id: I8836893dfcdf457d9c5d766612d687bfce64e2ae
BUG: 1126801
"Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>"
"Reviewed-on: http://review.gluster.org/8994"
"Reviewed-by: Niels de Vos <ndevos@redhat.com>"
"Tested-by: Gluster Build System <jenkins@build.gluster.com>"
(cherry picked from commit a5d73daabf6df95bc73b186d92f3e2d1239a6f8a)
Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
Reviewed-on: http://review.gluster.org/9001
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
---
 extras/glusterfs-georep-logrotate | 59 ++++++++++++++++++++++-----------------
 1 file changed, 33 insertions(+), 26 deletions(-)

(limited to 'extras/glusterfs-georep-logrotate')

diff --git a/extras/glusterfs-georep-logrotate b/extras/glusterfs-georep-logrotate
index 85e69d2c0d4..381db8ba464 100644
--- a/extras/glusterfs-georep-logrotate
+++ b/extras/glusterfs-georep-logrotate
@@ -1,36 +1,43 @@
-
-rotate 52
-missingok
-
-compress
-delaycompress
-notifempty
-
 /var/log/glusterfs/geo-replication/*/*.log {
-  sharedscripts
-  postrotate
-  for pid in `ps -aef | grep glusterfs | egrep "\-\-aux-gfid-mount" | awk '{print $2}'`; do
-    /usr/bin/kill -HUP $pid > /dev/null 2>&1 || true
-  done
-  endscript
+    sharedscripts
+    rotate 52
+    missingok
+    compress
+    delaycompress
+    notifempty
+    postrotate
+    for pid in `ps -aef | grep glusterfs | egrep "\-\-aux-gfid-mount" | awk '{print $2}'`; do
+        /usr/bin/kill -HUP $pid > /dev/null 2>&1 || true
+    done
+     endscript
 }
 
 
 /var/log/glusterfs/geo-replication-slaves/*.log {
-  sharedscripts
-  postrotate
-  for pid in `ps -aef | grep glusterfs | egrep "\-\-aux-gfid-mount" | awk '{print $2}'`; do
-    /usr/bin/kill -HUP $pid > /dev/null 2>&1 || true
-  done
-  endscript
+    sharedscripts
+    rotate 52
+    missingok
+    compress
+    delaycompress
+    notifempty
+    postrotate
+    for pid in `ps -aef | grep glusterfs | egrep "\-\-aux-gfid-mount" | awk '{print $2}'`; do
+        /usr/bin/kill -HUP $pid > /dev/null 2>&1 || true
+    done
+    endscript
 }
 
 
 /var/log/glusterfs/geo-replication-slaves/*/*.log {
-  sharedscripts
-  postrotate
-  for pid in `ps -aef | grep glusterfs | egrep "\-\-aux-gfid-mount" | awk '{print $2}'`; do
-    /usr/bin/kill -HUP $pid > /dev/null 2>&1 || true
-  done
-  endscript
+    sharedscripts
+    postrotate
+    rotate 52
+    missingok
+    compress
+    delaycompress
+    notifempty
+    for pid in `ps -aef | grep glusterfs | egrep "\-\-aux-gfid-mount" | awk '{print $2}'`; do
+        /usr/bin/kill -HUP $pid > /dev/null 2>&1 || true
+    done
+    endscript
 }
-- 
cgit