openstack/kolla-ansible
Code Issues Proposed changes

Fixing crond crashing with centos/source

Browse Source 
This PS address the issue encountered with the latest master, crond
container's docker file did not have any logic for source type of
installation as a result no binaries were install causing container's
constant restart. Fixing logic in cron's docker file.

Change-Id: I318aa9cb16517e21e0fe58393ee577701331b077
Closes-Bug: #1607336
This commit is contained in:
Serguei Bezverkhi 2016-07-28 08:02:23 -04:00
parent c895034030
commit 2df3d25019
1 changed files with 11 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
docker/cron/Dockerfile.j2
View File

@ -2,21 +2,19 @@ FROM {{ namespace }}/{{ image_prefix }}base:{{ tag }}
MAINTAINER {{ maintainer }} MAINTAINER {{ maintainer }}
{% import "macros.j2" as macros with context %} {% import "macros.j2" as macros with context %}
{% if install_type == 'binary' %} {% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
{% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %} {% set cron_packages = [
{% set cron_packages = [ 'cronie',
'cronie', 'logrotate'
'logrotate' ] %}
] %} {% elif base_distro in ['ubuntu', 'debian'] %}
{% elif base_distro in ['ubuntu', 'debian'] %} {% set cron_packages = [
{% set cron_packages = [ 'cron',
'cron', 'logrotate'
'logrotate' ] %}
] %} {% endif %}
{% endif %}
RUN {{ macros.install_packages(cron_packages | customizable("packages")) }} RUN {{ macros.install_packages(cron_packages | customizable("packages")) }}
{% endif %}
COPY extend_start.sh /usr/local/bin/kolla_extend_start COPY extend_start.sh /usr/local/bin/kolla_extend_start
RUN chmod 755 /usr/local/bin/kolla_extend_start RUN chmod 755 /usr/local/bin/kolla_extend_start