Yaniv Bronhaim has submitted this change and it was merged. Change subject: Live Merge: Ignore libvirt block job events ......................................................................
Live Merge: Ignore libvirt block job events Since VDSM needs to handle missed block job completion events that may have been emitted while the daemon was stopped, it is best to not rely on the events at all and initiate all cleanup operations from the existing polling function. Since a volume chain sync can be a relatively expensive operation involving HSM calls, run it in a separate thread. Depending on the current state of the storage, the sync may need to be retried. Change-Id: Ibbcdca4c0c0e45e9323ecfef9ce2fce10d8451a5 Signed-off-by: Adam Litke <[email protected]> Reviewed-on: http://gerrit.ovirt.org/30046 Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-by: Federico Simoncelli <[email protected]> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1124970 Reviewed-on: http://gerrit.ovirt.org/31049 Reviewed-by: Yaniv Bronhaim <[email protected]> --- M vdsm/clientIF.py M vdsm/virt/vm.py 2 files changed, 125 insertions(+), 77 deletions(-) Approvals: Adam Litke: Verified Yaniv Bronhaim: Looks good to me, approved Federico Simoncelli: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/31049 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibbcdca4c0c0e45e9323ecfef9ce2fce10d8451a5 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 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: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
