<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 12, 2016 at 3:41 PM, Adam Litke <span dir="ltr"><<a href="mailto:alitke@redhat.com" target="_blank">alitke@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 12/08/16 15:39 +0200, Sandro Bonazzola wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Fri, Aug 12, 2016 at 3:34 PM, Adam Litke <<a href="mailto:alitke@redhat.com" target="_blank">alitke@redhat.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
I would like to remove an old hack in the test code that was put in<br>
place at a time when sanlock was not widely available on everyone's<br>
development machines. Since it is now installable on all supported<br>
platforms I would like to make it a BuildRequires and remove the hack.<br>
See the patch [1] here.<br>
<br>
</blockquote>
<br>
isn't mom on standard-ci?<br>
</blockquote>
<br></span>
This is for vdsm... and I forgot to supply the patch reference:<br>
[1] <a href="https://gerrit.ovirt.org/#/c/61996/2" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>1996/2</a></blockquote><div><br></div><div>So you just need to add sanlock-python to automation/*package*</div><div><br></div><div>And it will be installed in the mock chroot.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In order for this patch to pass CI, I need those machines updated with<br>
the new BuildRequires: sanlock-python. Could you please install it?<br>
Thanks!<br>
<br>
Note that we do NOT need sanlock (the daemon). We just need the<br>
python module in order to satisfy imports.<br>
<br>
Thanks!<br>
<br>
--<br>
Adam Litke<br>
______________________________<wbr>_________________<br>
Infra mailing list<br>
<a href="mailto:Infra@ovirt.org" target="_blank">Infra@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/infra</a><br>
<br>
</blockquote>
<br>
<br>
<br>
-- <br>
Sandro Bonazzola<br>
Better technology. Faster innovation. Powered by community collaboration.<br>
See how it works at <a href="http://redhat.com" rel="noreferrer" target="_blank">redhat.com</a><br>
</blockquote>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Adam Litke<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Sandro Bonazzola<br>Better technology. Faster innovation. Powered by community collaboration.<br>See how it works at <a href="http://redhat.com" target="_blank">redhat.com</a><br></div></div></div></div>
</div></div>