From ab1973b52cb6de053cca60b50d058907f88e246b Mon Sep 17 00:00:00 2001 From: Evan Broder Date: Tue, 11 Aug 2009 23:01:15 -0400 Subject: [PATCH] Add a safety check to the janitor to make sure we only delete old_ LV. svn path=/trunk/packages/invirt-remote/; revision=2438 --- host/usr/sbin/invirt-janitor | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/host/usr/sbin/invirt-janitor b/host/usr/sbin/invirt-janitor index e631972..6edcca9 100755 --- a/host/usr/sbin/invirt-janitor +++ b/host/usr/sbin/invirt-janitor @@ -40,6 +40,14 @@ def cleanup(): lv_path = '/dev/xenvg/%s' % lv try: + # If the LV name doesn't start with old_, we probably + # don't actually want to be deleting it. + # + # Put it in the try block because we still want to delete + # the state file. + if not lv.startswith('old_'): + continue + syslog.syslog(syslog.LOG_INFO, "Cleaning up LV '%s'" % lv_path) # In a perfect world, this should be erroring out with -- 1.7.9.5