From 1fe8012ce2aec549430212d90ffd728889d6e4aa Mon Sep 17 00:00:00 2001 From: Mark Goddard Date: Wed, 26 Feb 2020 10:06:28 +0000 Subject: Actually disable EPEL, and epel-modular The disable_extra_repos macro accepts a list as its only argument. We were calling it like this to disable EPEL: disable_extra_repos('epel') The macro interpreted this as a request to disable three repos, e, p, l. Thanks Python! Type validation to be improved separately. Additionally, on CentOS 8 the EPEL repository was not included in the repository mapping file, repos.yaml. There is also another EPEL repository on CentOS 8, epel-modular, which is enabled by default after installing epel-release. This change adds mappings for epel and epel-modular repos to repos.yaml, and fixes the disabling of epel in the base image, as well as disabling epel-modular. There are some cases where EPEL is still used (it seemed a bit too easy...), and the repository has been enabled for these images: * bifrost-base (nginx) * ironic-conductor (C7 only, shellinabox) * freezer-base (C7 only, trickle) * gnocchi-base (C8 binary only, python3-boto3) * mariadb (pv) * mongodb (C7 only, mongodb) * nova-spicehtml5proxy (C7 only, spice-html5) * telegraf (C7 only, python2-pip) A few other things were changed: * ironic-conductor does not require the ceph repo * python3-pika is no longer installed in the openstack-base image Related: blueprint remove-epel Change-Id: I3761825239dfc462072383cde6276c4fb3e1bf12 --- docker/nova/nova-spicehtml5proxy/Dockerfile.j2 | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'docker/nova') diff --git a/docker/nova/nova-spicehtml5proxy/Dockerfile.j2 b/docker/nova/nova-spicehtml5proxy/Dockerfile.j2 index b5da34c12..fce71702e 100644 --- a/docker/nova/nova-spicehtml5proxy/Dockerfile.j2 +++ b/docker/nova/nova-spicehtml5proxy/Dockerfile.j2 @@ -5,6 +5,11 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build {% import "macros.j2" as macros with context %} +{# NOTE(mgoddard): EPEL required for spice-html5 package. Not provided by EPEL8 #} +{% if distro_package_manager == 'yum' %} +{{ macros.enable_extra_repos(['epel']) }} +{% endif %} + {% if install_type == 'binary' %} {% if base_package_type == 'rpm' %} -- cgit v1.2.3