Install sanlock-python on jenkins slaves

Adam Litke alitke at redhat.com
Fri Aug 12 13:54:57 UTC 2016


On 12/08/16 15:47 +0200, Sandro Bonazzola wrote:
>On Fri, Aug 12, 2016 at 3:41 PM, Adam Litke <alitke at redhat.com> wrote:
>
>> On 12/08/16 15:39 +0200, Sandro Bonazzola wrote:
>>
>>> On Fri, Aug 12, 2016 at 3:34 PM, Adam Litke <alitke at redhat.com> wrote:
>>>
>>> Hi,
>>>>
>>>> I would like to remove an old hack in the test code that was put in
>>>> place at a time when sanlock was not widely available on everyone's
>>>> development machines.  Since it is now installable on all supported
>>>> platforms I would like to make it a BuildRequires and remove the hack.
>>>> See the patch [1] here.
>>>>
>>>>
>>> isn't mom on standard-ci?
>>>
>>
>> This is for vdsm... and I forgot to supply the patch reference:
>> [1] https://gerrit.ovirt.org/#/c/61996/2
>
>
>So you just need to add sanlock-python to automation/*package*
>
>And it will be installed in the mock chroot.

Thanks!  I'll give that a try.

>
>
>
>>
>>
>>
>>>
>>>
>>>
>>>> In order for this patch to pass CI, I need those machines updated with
>>>> the new BuildRequires: sanlock-python.  Could you please install it?
>>>> Thanks!
>>>>
>>>> Note that we do NOT need sanlock (the daemon).  We just need the
>>>> python module in order to satisfy imports.
>>>>
>>>> Thanks!
>>>>
>>>> --
>>>> Adam Litke
>>>> _______________________________________________
>>>> Infra mailing list
>>>> Infra at ovirt.org
>>>> http://lists.ovirt.org/mailman/listinfo/infra
>>>>
>>>>
>>>
>>>
>>> --
>>> Sandro Bonazzola
>>> Better technology. Faster innovation. Powered by community collaboration.
>>> See how it works at redhat.com
>>>
>>
>> --
>> Adam Litke
>>
>
>
>
>-- 
>Sandro Bonazzola
>Better technology. Faster innovation. Powered by community collaboration.
>See how it works at redhat.com

-- 
Adam Litke



More information about the Infra mailing list