[libvirt] [jenkins-ci PATCH] guests: Install EPEL on CentOS 7 container
Fabiano Fidêncio
fidencio at redhat.com
Wed Oct 9 16:23:55 UTC 2019
On Wed, Oct 9, 2019 at 6:03 PM Daniel P. Berrangé <berrange at redhat.com> wrote:
>
> On Wed, Oct 09, 2019 at 05:52:27PM +0200, Fabiano Fidêncio wrote:
> > Since commit d63d6a59cc we're installing EPEL on CentOS7 in order to
> > have both "ninja" and "python36-*" packages in the system.
> >
> > However, we've forgot to add the very code to the lcitool so the
> > containers would also have EPEL release installed.
> >
> > Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
> > ---
> > guests/lcitool | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/guests/lcitool b/guests/lcitool
> > index a3ef137..0d11d37 100755
> > --- a/guests/lcitool
> > +++ b/guests/lcitool
> > @@ -714,6 +714,14 @@ class Application:
> > {package_manager} autoremove -y && \\
> > {package_manager} clean all -y
> > """).format(**varmap))
> > + else if os_name == "CentOS" and os_version == "7":
> > + sys.stdout.write(textwrap.dedent("""
> > + RUN {package_manager} update -y && \\
> > + {package_manager} install -y epel-release && \\
>
> How does this actually work ?
>
> Isn't there a chicken & egg problem here. The 'epel-release' RPM is
> in the EPEL repos, so you have to already have EPEL enabled in order
> to enable EPEL surely....
epel-release is part of "extras" repository*, which is enabled by default.
*: http://mirror.centos.org/centos/7/extras/x86_64/Packages/
[snip]
Best Regards,
--
Fabiano Fidêncio
More information about the libvir-list
mailing list