This is a multi-part message in MIME format.
--------------1E2632E836D8A74038541936
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
On 09/11/2017 01:16 PM, Eyal Edri wrote:
On Mon, Sep 11, 2017 at 2:02 PM, Francesco Romani <fromani(a)redhat.com
<mailto:fromani@redhat.com>> wrote:
Hi everyone,
https://gerrit.ovirt.org/#/q/topic:drivemonitor_event+status:open
<
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
<
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.
Sure thing. Will not merge before OST and CI both pass. But it is
totally reviewable while we wait! :)
Bests,
--
Francesco Romani
Senior SW Eng., Virtualization R&D
Red Hat
IRC: fromani github: @fromanirh
--------------1E2632E836D8A74038541936
Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: 8bit
<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=utf-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-cite-prefix">On 09/11/2017 01:16 PM, Eyal Edri
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CACuV_Hoq2i+v+WLYaoGLF1wF1veZRdXzGnJb+b_oyJm5wLiGqQ@mail.gmail.com">
<div dir="ltr"><br>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Mon, Sep 11, 2017 at 2:02 PM,
Francesco Romani <span dir="ltr"><<a
href="mailto:fromani@redhat.com" target="_blank"
moz-do-not-send="true">fromani(a)redhat.com</a>&gt;</span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">Hi
everyone,<br>
<br>
<br>
<a
href="https://gerrit.ovirt.org/#/q/topic:drivemonitor_event+status:o...
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://gerrit.ovirt.org/#/q/<wbr&...
is<br>
ready for review. It is the first part of the series
needed<br>
<br>
to consume the BLOCK_THRESHOLD event available with
libvirt >= 3.2.0 and<br>
QEMU >= 2.3.0.<br>
<br>
Once completed, this patchset will allow Vdsm to avoid
polling, thus<br>
greatly improving the system performance and<br>
<br>
eventually close <a
href="https://bugzilla.redhat.com/show_bug.cgi?id=1181665"
rel="noreferrer" target="_blank"
moz-do-not-send="true">https://bugzilla.redhat.com/<wbr&g...
<br>
<br>
Please note that:<br>
<br>
1. CI fails because the workers are not yet updated to
CentOS 7.4 (not<br>
yet released AFAIK!) which will provide libvirt >=
3.2.0.<br>
</blockquote>
<div><br>
</div>
<div>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</div>
<div>before merging, otherwise any patch that will be merged
afterwards will fail and CI won't work.</div>
<div><br>
</div>
<div>AFAIK, it should be out this week.</div>
<div> <br>
</div>
</div>
<br>
</div>
</div>
</blockquote>
<br>
Sure thing. Will not merge before OST and CI both pass. But it is
totally reviewable while we wait! :)<br>
<br>
Bests,<br>
<pre class="moz-signature" cols="72">--
Francesco Romani
Senior SW Eng., Virtualization R&D
Red Hat
IRC: fromani github: @fromanirh</pre>
</body>
</html>
--------------1E2632E836D8A74038541936--