Dan Kenigsberg has submitted this change and it was merged. Change subject: virt: periodic: port queryBlockJobs ......................................................................
virt: periodic: port queryBlockJobs Vm/Block Jobs monitoring is a fundamental part of live merge support. Most often, VMs do not have block jobs to monitor, so we should avoid unnecessary work for performance reasons. However, block jobs need to enter the QEMU monitor, so we also need to take countermeasures for possibly blocking operations. The periodic.py aka 'periodic operations' framework can accomodate both needs, so this patch moves the monitoring of the VM block jobs under the periodic.py umbrella, with no intended changes in behaviour nor in the output. The only code changes are limited to the glue code needed to adapt from former VmStatsThread framework. Change-Id: I2bba47881033af1b928195a94456f7130a7ac343 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/37595 Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/periodic.py M vdsm/virt/vm.py 2 files changed, 61 insertions(+), 19 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/37595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bba47881033af1b928195a94456f7130a7ac343 Gerrit-PatchSet: 42 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[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
