Merge "Introduce clean_package_cache"

docker/base/Dockerfile.j2

@@ -156,9 +156,9 @@ RUN rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-7
     rpm --import {{ key }} \
 {% endfor -%}
 {%- if base_centos_yum_repo_keys|customizable('centos_yum_repo_keys')|length == 0 %}RUN {% else %}    && {% endif -%}
-    yum -y update --security --sec-severity=Important --sec-severity=Critical \
+    yum -y update --security --sec-severity=Important --sec-severity=Critical{% if clean_package_cache %} \
     && yum clean all \
-    && rm -rf /var/cache/yum
+    && rm -rf /var/cache/yum{% endif %}
 
     {% endif %}
     {# Endif for base_distro centos #}
@@ -180,9 +180,9 @@ RUN yum-config-manager --enable rhel-7-server-optional-rpms \
     && yum-config-manager --enable rhel-7-server-rhceph-2-osd-rpms \
     && yum-config-manager --enable rhel-7-server-rhceph-2-mon-rpms \
     && yum-config-manager --enable rhel-7-server-rhceph-2-tools-rpms \
-    && yum -y update --security --sec-severity=Important --sec-severity=Critical \
+    && yum -y update --security --sec-severity=Important --sec-severity=Critical{% if clean_package_cache %} \
     && yum clean all \
-    && rm -rf /var/cache/yum
+    && rm -rf /var/cache/yum{% endif %}
 {% endblock %}
 
     {% endif %}
@@ -210,9 +210,9 @@ RUN yum -y install \
     && yum-config-manager --enable ol7_optional_latest ol7_addons \
     && yum -y install \
            yum-plugin-priorities \
-    && yum -y update --security --sec-severity=Important --sec-severity=Critical \
+    && yum -y update --security --sec-severity=Important --sec-severity=Critical{% if clean_package_cache %} \
     && yum clean all \
-    && rm -rf /var/cache/yum
+    && rm -rf /var/cache/yum{% endif %}
 {% endblock %}
 
     {% endif %}

docker/macros.j2

@@ -1,14 +1,16 @@
 {% macro install_packages(packages) -%}
 {% if packages is defined and packages|length > 0 -%}
     {% if base_package_type == 'rpm' -%}
-        RUN {{ distro_package_manager }} -y install {{ packages | join(' ') }} && {{ distro_package_manager }} clean all && rm -rf /var/cache/{{ distro_package_manager }}
+        RUN {{ distro_package_manager }} -y install {{ packages | join(' ') }}
+        {%- if clean_package_cache %} \
+    && {{ distro_package_manager }} clean all && rm -rf /var/cache/{{ distro_package_manager }}{% endif %}
     {%- elif base_package_type == 'deb' -%}
         {#-
           debian_package_install is a utility method to build up an appropriate
           set of commands to install packages in a debian-based environment that
           may include URL links to a .deb package
         -#}
-        RUN {{ debian_package_install(packages) }}
+        RUN {{ debian_package_install(packages, clean_package_cache) }}
     {%- endif %}
 {%- endif %}
 {%- endmacro %}

kolla/common/config.py

@@ -283,7 +283,9 @@ _BASE_OPTS = [
     cfg.BoolOpt('squash-cleanup', default=True,
                 help='Remove source image from Docker after squashing'),
     cfg.StrOpt('squash-tmp-dir',
-               help='Temporary directory to be used during squashing')
+               help='Temporary directory to be used during squashing'),
+    cfg.BoolOpt('clean_package_cache', default=True,
+                help='Clean all package cache.')
 ]
 
 

kolla/image/build.py

@@ -715,6 +715,8 @@ class KollaWorker(object):
             package_manager = 'apt'
         self.distro_package_manager = package_manager
 
+        self.clean_package_cache = self.conf.clean_package_cache
+
         # Determine base packaging type for use in Dockerfiles.
         if self.conf.base_package_type:
             self.base_package_type = self.conf.base_package_type
@@ -933,7 +935,8 @@ class KollaWorker(object):
                       'distro_python_version': self.distro_python_version,
                       'distro_package_manager': self.distro_package_manager,
                       'rpm_setup': self.rpm_setup,
-                      'build_date': build_date}
+                      'build_date': build_date,
+                      'clean_package_cache': self.clean_package_cache}
             env = jinja2.Environment(  # nosec: not used to render HTML
                 loader=jinja2.FileSystemLoader(self.working_dir))
             env.filters.update(self._get_filters())

kolla/template/methods.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 
-def debian_package_install(packages):
+def debian_package_install(packages, clean_package_cache=True):
     """Jinja utility method for building debian-based package install command.
 
     apt-get is not capable of installing .deb files from a URL and the
@@ -48,6 +48,7 @@ def debian_package_install(packages):
         cmds.append('apt-get -y install --no-install-recommends {}'.format(
             ' '.join(reg_packages)
         ))
+        if clean_package_cache:
             cmds.append('apt-get clean')
             cmds.append('rm -rf /var/lib/apt/lists/*')
 

releasenotes/notes/clean_package_cache-fa08d1808a2f2b49.yaml

@@ -0,0 +1,6 @@
+---
+features:
+  - |
+    clean_package_cache is a parameter (set to True by default for backward
+    compatibility) that controls the package cache cleaning when building the
+    images.