Don't bail if the VM renaming at the end of saving doesn't succeed -- we

expect a failure if this is a localhost migration.

Signed-off-by: Ewan Mellor <ewan@xensource.com>
This commit is contained in:
Ewan Mellor committed 2006-11-28 11:35:10 +00:00
1 parent 2e873d3d17
commit 8e37e2b661
1 file changed
+9 -2
+9 -2
View File
@@ -37,6 +37,7 @@ def write_exact(fd, buf, errmsg):
if os.write(fd, buf) != len(buf):
raise XendError(errmsg)
def read_exact(fd, size, errmsg):
buf = ''
while size != 0:
@@ -50,7 +51,6 @@ def read_exact(fd, size, errmsg):
return buf
def save(fd, dominfo, network, live, dst):
write_exact(fd, SIGNATURE, "could not write guest state file: signature")
@@ -97,7 +97,14 @@ def save(fd, dominfo, network, live, dst):
forkHelper(cmd, fd, saveInputHandler, False)
dominfo.destroyDomain()
dominfo.setName(domain_name)
try:
dominfo.setName(domain_name)
except VmError:
# Ignore this. The name conflict (hopefully) arises because we
# are doing localhost migration; if we are doing a suspend of a
# persistent VM, we need the rename, and don't expect the
# conflict. This needs more thought.
pass
except Exception, exn:
log.exception("Save failed on domain %s (%s).", domain_name,
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.