From: intrigeri <intrigeri@boum.org>
Date: Sat, 8 Jun 2013 06:58:49 +0000 (+0200)
Subject: Add option to include rdiff-backup output in reports.
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=1c6677ebddff17f5efdaddebf19a89ec15d58b3d;p=backupninja.git

Add option to include rdiff-backup output in reports.

Thanks to David Gasaway <dave@gasaway.org> for the patch.
---

diff --git a/examples/example.rdiff b/examples/example.rdiff
index e8ce542..323b3eb 100644
--- a/examples/example.rdiff
+++ b/examples/example.rdiff
@@ -53,6 +53,20 @@
 ## Default:
 # ignore_version = no
 
+## should backupninja write program output as Info messages rather than Debug
+## messages? (default: no)
+## Usually rdiff-backup output (for increment expiration and backup) is written
+## to output as Debug messages; this option causes backupninja to use Info-level
+## messages instead. Since backup reports include Info messages, this option is
+## useful to receive output like rdiff-backup session statistics in reports. In
+## addition, since rdiff-backup has a habit of using a zero exit code when
+## non-fatal errors are encountered (causing backupninja to conclude the backup
+## was entirely successful), this option is useful for inspecting non-fatal
+## filesystem and permission errors from rdiff-backup.
+##
+## Default:
+# output_as_info = no
+
 ######################################################
 ## source section
 ## (where the files to be backed up are coming from)
diff --git a/handlers/rdiff.in b/handlers/rdiff.in
index c3c8d1d..471a3d7 100644
--- a/handlers/rdiff.in
+++ b/handlers/rdiff.in
@@ -85,6 +85,7 @@ getconf testconnect yes
 getconf nicelevel 0
 getconf bwlimit
 getconf ignore_version no
+getconf output_as_info no
 
 setsection source
 getconf type; sourcetype=$type
@@ -182,7 +183,11 @@ if [ "$keep" != yes ]; then
    if [ $test = 0 ]; then
       output="`su -c "$removestr" 2>&1`"
       if [ $? = 0 ]; then
-         debug $output
+         if [ "$output_as_info" == "yes" ]; then
+            info $output
+         else
+            debug $output
+         fi
          info "Removing backups older than $keep days succeeded."
       else
          warning $output
@@ -268,7 +273,11 @@ debug "$execstr"
 if [ $test = 0 ]; then
    output=`nice -n $nicelevel su -c "$execstr" 2>&1`
    if [ $? = 0 ]; then
-      debug $output
+      if [ "$output_as_info" == "yes" ]; then
+         info $output
+      else
+         debug $output
+      fi
       info "Successfully finished backing up source $label"
    else
       error $output