[ovirt-devel] [review][vdsm] please review https://gerrit.ovirt.org/#/q/topic:drivemonitor_event+status:open

Eyal Edri eedri at redhat.com
Mon Sep 11 11:16:39 UTC 2017


On Mon, Sep 11, 2017 at 2:02 PM, Francesco Romani <fromani at redhat.com>
wrote:

> Hi everyone,
>
>
> https://gerrit.ovirt.org/#/q/topic:drivemonitor_event+status:open is
> ready for review. It is the first part of the series needed
>
> to consume the BLOCK_THRESHOLD event available with libvirt >= 3.2.0 and
> QEMU >= 2.3.0.
>
> Once completed, this patchset will allow Vdsm to avoid polling, thus
> greatly improving the system performance and
>
> eventually close https://bugzilla.redhat.com/show_bug.cgi?id=1181665
>
>
> Please note that:
>
> 1. CI fails because the workers are not yet updated to CentOS 7.4 (not
> yet released AFAIK!) which will provide libvirt >= 3.2.0.
>

You probably know that already, but just to be sure, please wait for
official CentOS 7.4 be out and that we'll verify OST works well with it
before merging, otherwise any patch that will be merged afterwards will
fail and CI won't work.

AFAIK, it should be out this week.


>
> 2. Few more simple patches will be needed to enable/disable monitoring
> in specific flows where we cannot use events (e.g. LSM)
>
> 3. I did initial verification successfully, installing fedora 25 on thin
> provisioned disk without issue.
>
>
> --
> Francesco Romani
> Senior SW Eng., Virtualization R&D
> Red Hat
> IRC: fromani github: @fromanirh
>
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
>
>


-- 

Eyal edri


ASSOCIATE MANAGER

RHV DevOps

EMEA VIRTUALIZATION R&D


Red Hat EMEA <https://www.redhat.com/>
<https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20170911/e03fdb62/attachment-0001.html>


More information about the Devel mailing list