Fix #1260 ganeti-cleaner accepts REMOVE_AFTER as env var (#1267)
authorCalum Calder <calumcalder@users.noreply.github.com>
Tue, 19 Sep 2017 11:23:26 +0000 (12:23 +0100)
committerMorg <morg@google.com>
Tue, 19 Sep 2017 11:23:26 +0000 (12:23 +0100)
This commit adds the option to use REMOVE_AFTER env var before running ganeti-cleaner master
to specify how many days to use as threshold before removal.

Signed-off-by: Calum Calder <calumcalder@google.com>
Reviewed-by: Federico Morg Pareschi <morg@google.com>

daemons/ganeti-cleaner.in
doc/examples/ganeti.cron.in
man/ganeti-cleaner.rst

index f4e9bba..dc5abe1 100644 (file)
@@ -106,7 +106,7 @@ cleanup_master() {
 }
 
 # Define how many days archived jobs should be left alone
-REMOVE_AFTER=21
+REMOVE_AFTER=${REMOVE_AFTER:-21}
 
 # Define how many log files to keep around (usually one per day)
 KEEP_LOGS=50
index ad5c79c..69c804e 100644 (file)
@@ -10,7 +10,7 @@ PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin
 */30 * * * * root [ -x @SBINDIR@/ganeti-watcher ] && @SBINDIR@/ganeti-watcher
 
 # Clean job archive (at 01:45 AM)
-45 1 * * * @GNTMASTERUSER@ [ -x @SBINDIR@/ganeti-cleaner ] && @SBINDIR@/ganeti-cleaner master
+45 1 * * * @GNTMASTERUSER@ [ -x @SBINDIR@/ganeti-cleaner ] && REMOVE_AFTER=21 @SBINDIR@/ganeti-cleaner master
 
 # Clean job archive (at 02:45 AM)
 45 2 * * * @GNTNODEDUSER@ [ -x @SBINDIR@/ganeti-cleaner ] && @SBINDIR@/ganeti-cleaner node
index 4eb984c..2603bbf 100644 (file)
@@ -22,8 +22,9 @@ certificates and keys from ``@LOCALSTATEDIR@/run/ganeti/crypto``, as
 well as outdated **ganeti-watcher** information.
 
 When called with ``master`` as argument, it will instead automatically
-remove all files older than 21 days from
-``@LOCALSTATEDIR@/lib/ganeti/queue/archive``.
+remove all files older than $REMOVE_AFTER days from
+``@LOCALSTATEDIR@/lib/ganeti/queue/archive``, with $REMOVE_AFTER
+defaulting to 21.
 
 .. vim: set textwidth=72 :
 .. Local Variables: