Dan Kenigsberg has submitted this change and it was merged. Change subject: Live Merge: Fix backing chain xml parsing ......................................................................
Live Merge: Fix backing chain xml parsing The backingStore element is optional inside of a disk XML block. It will be missing for disks with no backing files (ie. new VM based on the blank template). Handle this case by breaking out of the loop rather than returning an error. Change-Id: I6128d1eb1265f7ede1ab420ac7b5342bc7f2db9e Signed-off-by: Adam Litke <[email protected]> Reviewed-on: http://gerrit.ovirt.org/29006 Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Adam Litke: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29006 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6128d1eb1265f7ede1ab420ac7b5342bc7f2db9e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Greg Padgett <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
