From: Steven Valdez Date: Thu, 1 Aug 2013 08:30:05 +0000 (-0400) Subject: Fixed lv deactivation for rename X-Git-Tag: 0.0.6^0 X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-images.git/commitdiff_plain/91cbf1beadaade4ac71d07c0214fc382fb2090c3 Fixed lv deactivation for rename --- diff --git a/debian/changelog b/debian/changelog index 6fd8e36..1dd7308 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +invirt-images (0.0.6) precise; urgency=low + + * Fixing images for new lv setup to deactivate volumes before modifying. + + -- Steven Valdez Thu, 01 Aug 2013 04:29:37 -0400 + invirt-images (0.0.5) precise; urgency=low * Updating version for precise migration. diff --git a/invirt-images b/invirt-images index ea208a5..e6f987d 100755 --- a/invirt-images +++ b/invirt-images @@ -41,19 +41,39 @@ def lvcreate(name, size): return 6 def lvrename(dest, src): - lvr = subprocess.Popen(['lvrename', 'xenvg', src, dest], + lvr = subprocess.Popen(['lvchange', '-an', "xenvg/%s" % (src,)], stderr=subprocess.PIPE, stdout=getOutput()['stdout']) ret = lvr.wait() - if not ret: - return 0 - stderr = lvr.stderr.read() - if 'not found in volume group' in stderr: - return 0 - else: + + if ret: + if verbosity > 0: + print lvr.stderr.read() + return ret + + lvr = subprocess.Popen(['lvrename', "xenvg/%s" % (src,), "xenvg/%s" % (dest,)], + stderr=subprocess.PIPE, + stdout=getOutput()['stdout']) + ret = lvr.wait() + + if ret: + stderr = lvr.stderr.read() + if not ('not found in volume group' in stderr): + if verbosity > 0: + print stderr + return ret + + subprocess.Popen(['lvchange', '-ay', "xenvg/%s" % (dest,)], + stderr=subprocess.PIPE, + stdout=getOutput()['stdout']) + ret = lvr.wait() + + if ret: if verbosity > 0: - print stderr - return ret + print lvr.stderr.read() + + return ret + def lv_random(func, pattern, *args): """