Re: [libvirt] [jenkins-ci PATCH v2 5/8] guests: Install EPEL repo on all CentOS guests

2019-12-11 Thread Andrea Bolognani
On Wed, 2019-12-11 at 13:25 +0100, Fabiano Fidêncio wrote: > This is the fixup for this patch: > ``` > diff --git a/guests/lcitool b/guests/lcitool > index 803477e..059b789 100755 > --- a/guests/lcitool > +++ b/guests/lcitool > @@ -748,7 +748,7 @@ class Application: > {pack

Re: [libvirt] [jenkins-ci PATCH v2 5/8] guests: Install EPEL repo on all CentOS guests

2019-12-11 Thread Fabiano Fidêncio
On Wed, Dec 11, 2019 at 10:44 AM Fabiano Fidêncio wrote: > > On Wed, Dec 11, 2019 at 10:31 AM Andrea Bolognani wrote: > > > > On Tue, 2019-12-10 at 16:38 +0100, Fabiano Fidêncio wrote: > > > +++ b/guests/playbooks/update/tasks/base.yml > > > @@ -15,7 +15,6 @@ > > > state: latest > > >whe

Re: [libvirt] [jenkins-ci PATCH v2 5/8] guests: Install EPEL repo on all CentOS guests

2019-12-11 Thread Fabiano Fidêncio
On Wed, Dec 11, 2019 at 10:31 AM Andrea Bolognani wrote: > > On Tue, 2019-12-10 at 16:38 +0100, Fabiano Fidêncio wrote: > > +++ b/guests/playbooks/update/tasks/base.yml > > @@ -15,7 +15,6 @@ > > state: latest > >when: > > - os_name == 'CentOS' > > -- os_version == '7' > > Welp, I

Re: [libvirt] [jenkins-ci PATCH v2 5/8] guests: Install EPEL repo on all CentOS guests

2019-12-11 Thread Andrea Bolognani
On Tue, 2019-12-10 at 16:38 +0100, Fabiano Fidêncio wrote: > +++ b/guests/playbooks/update/tasks/base.yml > @@ -15,7 +15,6 @@ > state: latest >when: > - os_name == 'CentOS' > -- os_version == '7' Welp, I just realized you need to apply the same change to the _action_dockerfile()

[libvirt] [jenkins-ci PATCH v2 5/8] guests: Install EPEL repo on all CentOS guests

2019-12-10 Thread Fabiano Fidêncio
EPEL repository should be enabled on all supported CentOS guests as it brings in packages which are part of the "base packages" for libvirt-jenkins-ci. Signed-off-by: Fabiano Fidêncio Reviewed-by: Andrea Bolognani --- guests/playbooks/update/tasks/base.yml | 1 - 1 file changed, 1 deletion(-)