From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: blockIoTune: add and use cache
......................................................................
vm: blockIoTune: add and use cache
Getting and setting the block I/O tune values need to go
deep down the QEMU monitor. Thus, this call is costly and risky,
and we should avoid it as much as we can for obvious benefit purposes.
MOM and Engine needs this data, but both pass through Vdsm
to get and set the values.
Thus, it makes sense to add a caching layer for this data inside Vdsm.
Doing so we both increase the system performance reducing the libvirt
load, and by relieving stress on libvirt side we reduce the risk
of thread pool exhaustion.
Bug-Url:
https://bugzilla.redhat.com/1443654
Backport-To: 4.1
Change-Id: Ie50fc8a9395eab60a8b97b998555df1146af5b1b
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M tests/vm_test.py
M vdsm/virt/vm.py
2 files changed, 204 insertions(+), 9 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Verified
Andrej Krejcir: Looks good to me, but someone else must approve
Martin Polednik: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/75767
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie50fc8a9395eab60a8b97b998555df1146af5b1b
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>