Install sanlock-python on jenkins slaves
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. 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
On Fri, Aug 12, 2016 at 3:34 PM, Adam Litke <alitke@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?
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@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
On 12/08/16 15:39 +0200, Sandro Bonazzola wrote:
On Fri, Aug 12, 2016 at 3:34 PM, Adam Litke <alitke@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
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@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
On Fri, Aug 12, 2016 at 3:41 PM, Adam Litke <alitke@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@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.
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@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
On 12/08/16 15:47 +0200, Sandro Bonazzola wrote:
On Fri, Aug 12, 2016 at 3:41 PM, Adam Litke <alitke@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@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@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
participants (2)
-
Adam Litke -
Sandro Bonazzola