Yaniv Bronhaim has submitted this change and it was merged. Change subject: vm: consider 'action' when handling I/O errors ......................................................................
vm: consider 'action' when handling I/O errors When VDSM receives an I/O error event from libvirt, it does not consider the value of the 'action' callback field, and thus considers the guest CPU stopped. This is true in the default configuration, which translates to the disk write error policy 'stop', but not necessarily true if the said policy is 'enospace'. In this case the reported state of the CPU goes out of sync. Since the error policy may be changed through an hook, the safest thing to do is actually check the 'action' value as reported by libvirt in the IOError callback. This patch adds this check, and additional logging in the case the guest CPU is reported running after I/O errors. Change-Id: I9299399c3e5bef7e6e3111aa35e3483827ad57da Bug-Url: https://bugzilla.redhat.com/1064630 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: http://gerrit.ovirt.org/25157 Reviewed-by: Dan Kenigsberg <[email protected]> Tested-by: Dan Kenigsberg <[email protected]> Reviewed-on: http://gerrit.ovirt.org/26023 Reviewed-by: Vinzenz Feenstra <[email protected]> Reviewed-by: Michal Skrivanek <[email protected]> Reviewed-by: Yaniv Bronhaim <[email protected]> --- M vdsm/clientIF.py M vdsm/vm.py 2 files changed, 17 insertions(+), 9 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, approved Vinzenz Feenstra: Looks good to me, but someone else must approve Francesco Romani: Verified Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/26023 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9299399c3e5bef7e6e3111aa35e3483827ad57da Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.4 Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[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
