Change in vdsm[master]: vm: handle missing domains on recovery

fromani at redhat.com fromani at redhat.com
Tue Mar 25 09:07:11 UTC 2014


Francesco Romani has posted comments on this change.

Change subject: vm: handle missing domains on recovery
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/25275/1//COMMIT_MSG
Commit Message:

Line 7: vm: handle missing domains on recovery
Line 8: 
Line 9: When VDSM tries to reconnect to libvirt, it is possible
Line 10: that the domain lookup fails. This is especially true on
Line 11: recovering.
> Sorry for being a bit slow here, but we should already have a safeguard in 
Ack. Will amend the commit message in the next upload.
Line 12: 
Line 13: This patch add an explicit check in the recovery path to
Line 14: make sure that a VM is either created with a valid libvirt
Line 15: domain handle, or it is reported as Down so the engine


-- 
To view, visit http://gerrit.ovirt.org/25275
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I00ef12883c8035209de0f273925eb8603d6b6da8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list