Nir Soffer has submitted this change and it was merged. Change subject: vm: Add error logging in diskReplicateFinish ......................................................................
vm: Add error logging in diskReplicateFinish On failures of diskReplicateFinish we had mostly no logs, making debugging much harder. Log error for each failure explaining the error, and include the source disk, drive name, and the libvirt job info. Change-Id: Ibcfe5a6e2f3dcd03ea07df523723e8134239fd59 Bug-Url: https://bugzilla.redhat.com/1325791 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: https://gerrit.ovirt.org/55939 Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Freddy Rolland <[email protected]> --- M vdsm/virt/vm.py 1 file changed, 14 insertions(+), 2 deletions(-) Approvals: Nir Soffer: Verified; Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Francesco Romani: Looks good to me, approved Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/55939 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibcfe5a6e2f3dcd03ea07df523723e8134239fd59 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Idan Shaby <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
