Cron packages missing for 'source' install type

Bug #1607598 reported by Blake Caldwell
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla
Fix Released
Undecided
Unassigned

Bug Description

The changes to the cron dockerfile template in the following commit doesn't handle the 'source' install type.

----
commit 9ad412a41810f20e24a3da19f7c1bbf5fdcc96d1
Author: Abe Music <email address hidden>
Date: Mon Jul 25 08:58:26 2016 -0500

    Customizations for cron

    This patchset contains customization of Dockerfile
    of cron container

    Change-Id: I7066b281d2a546e52ab0fdc2722da2e0026fc3a7
    Partially-implements: blueprint third-party-plugin-support
----

For this container, there should be no difference between 'binary' and 'source'. Propose the following patch:

----
diff --git a/docker/cron/Dockerfile.j2 b/docker/cron/Dockerfile.j2
index b0d7965..fa8f771 100644
--- a/docker/cron/Dockerfile.j2
+++ b/docker/cron/Dockerfile.j2
@@ -2,7 +2,6 @@ FROM {{ namespace }}/{{ image_prefix }}base:{{ tag }}
 MAINTAINER {{ maintainer }}
 {% import "macros.j2" as macros with context %}

-{% if install_type == 'binary' %}
     {% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
         {% set cron_packages = [
             'cronie',
@@ -16,7 +15,6 @@ MAINTAINER {{ maintainer }}
     {% endif %}

 RUN {{ macros.install_packages(cron_packages | customizable("packages")) }}
-{% endif %}

 COPY extend_start.sh /usr/local/bin/kolla_extend_start
 RUN chmod 755 /usr/local/bin/kolla_extend_start
----

Changed in kolla:
assignee: nobody → Eduardo Gonzalez (egonzalez90)
assignee: Eduardo Gonzalez (egonzalez90) → nobody
Revision history for this message
Eduardo Gonzalez (egonzalez90) wrote :
Changed in kolla:
status: New → Fix Released
milestone: none → newton-rc1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.