From 3f15f96f630ed9006ab99ac92b601c1e902442de Mon Sep 17 00:00:00 2001 From: Evan Broder Date: Tue, 18 Nov 2008 11:55:50 -0500 Subject: [PATCH] I know this is the wrong answer, but doing lvchange -a n twice before deleting an LV tends to make it less likely to error out svn path=/trunk/packages/invirt-remote-host/; revision=1716 --- debian/changelog | 7 +++++++ files/usr/sbin/invirt-lvm | 4 ++++ 2 files changed, 11 insertions(+) diff --git a/debian/changelog b/debian/changelog index 6cbf7e6..0739c67 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,10 @@ +invirt-remote-host (0.0.8) unstable; urgency=low + + * I know this is the wrong answer, but doing lvchange -a n twice before + deleting an LV tends to make it less likely to error out + + -- Evan Broder Tue, 18 Nov 2008 11:55:22 -0500 + invirt-remote-host (0.0.7) unstable; urgency=low * Actually remove the unused vmboot remctl diff --git a/files/usr/sbin/invirt-lvm b/files/usr/sbin/invirt-lvm index 00766f6..6367216 100755 --- a/files/usr/sbin/invirt-lvm +++ b/files/usr/sbin/invirt-lvm @@ -44,6 +44,10 @@ if subcommand == "lvremove": def error(): print >>sys.stderr, "Error removing LV %s\n" % lvname sys.exit(1) + # I know this is the wrong answer, but sometimes the first + # lvchange -a n fails for no particularly good reason, so this is + # a pretty good workaround + call(["/sbin/lvchange", "-a", "n", lvpath]) rv = call(["/sbin/lvchange", "-a", "n", lvpath]) if rv != 0: error() -- 1.7.9.5