aboutsummaryrefslogtreecommitdiff
path: root/handlers/sys.in
diff options
context:
space:
mode:
authorMicah Anderson <micah@riseup.net>2010-10-01 18:09:43 -0400
committerMicah Anderson <micah@riseup.net>2010-10-12 11:48:50 -0400
commit9271e58a24a80b00ec57e9f4ed99ddd2ed80e3e0 (patch)
tree93eff843130859d5495a0d9a8887c26ee459c460 /handlers/sys.in
parenta80d6f4a0cc96ebd4125ccc016d7dbb256ba992d (diff)
downloadbackupninja-9271e58a24a80b00ec57e9f4ed99ddd2ed80e3e0.tar.gz
backupninja-9271e58a24a80b00ec57e9f4ed99ddd2ed80e3e0.tar.bz2
older versions (lenny) of lvm2 have a bug in vgcfgbackup that makes it so when you specify multiple volume groups to backup, it gives an error, saying you must set the %s template for the name. We are doing that, it just cannot handle it properly.
This change routes around this issue, which has been fixed in later lvm2 releases, by passing only one argument to vgcfgbackup by taking all the volume groups that are found and running them through a for loop
Diffstat (limited to 'handlers/sys.in')
-rwxr-xr-xhandlers/sys.in7
1 files changed, 5 insertions, 2 deletions
diff --git a/handlers/sys.in b/handlers/sys.in
index a92663e..fcf3e31 100755
--- a/handlers/sys.in
+++ b/handlers/sys.in
@@ -649,8 +649,11 @@ function doLvmBackup () {
debug "$VGS --options vg_name --noheadings | @SED@ 's/^[ ]*//' | @SED@ 's/[ ]*$//' | tr '\n' ' '"
vgs=`$VGS --options vg_name --noheadings | @SED@ 's/^[ ]*//' | @SED@ 's/[ ]*$//' | tr '\n' ' '`
debug "Let's try to backup LVM metadata for detected volume groups: $vgs"
- debug "$VGCFGBACKUP --file \"${lvmdir}\"/\'%s\' $vgs"
- output=`$VGCFGBACKUP --file "${lvmdir}"/'%s' $vgs`
+ for vg in $vgs
+ do
+ debug "$VGCFGBACKUP --file \"${lvmdir}\"/\'%s\' $vg"
+ output=`$VGCFGBACKUP --file "${lvmdir}"/'%s' $vg`
+ done
exit_code=$?
debug $output
case $exit_code in