[libvirt] [jenkins-ci PATCH] guests: Install EPEL on CentOS 7 container
Daniel P. Berrangé
berrange at redhat.com
Wed Oct 9 16:03:15 UTC 2019
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....
> + {package_manager} install -y {pkgs} && \\
> + {package_manager} autoremove -y && \\
> + {package_manager} clean all -y
> + """).format(**varmap))
> else:
> sys.stdout.write(textwrap.dedent("""
> RUN {package_manager} update -y && \\
> --
> 2.23.0
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list
mailing list