From: Evan Broder <broder@mit.edu>
Date: Wed, 12 Aug 2009 03:01:15 +0000 (-0400)
Subject: Add a safety check to the janitor to make sure we only delete old_ LV.
X-Git-Tag: 0.4.0~1
X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-remote.git/commitdiff_plain/ab1973b52cb6de053cca60b50d058907f88e246b?ds=sidebyside

Add a safety check to the janitor to make sure we only delete old_ LV.

svn path=/trunk/packages/invirt-remote/; revision=2438
---

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