From a7c3851d29d0085c3e39f72fbf79244aa64c5f50 Mon Sep 17 00:00:00 2001
From: "Swapnil Kulkarni (coolsvap)" <me@coolsvap.net>
Date: Mon, 25 Jul 2016 08:55:33 +0530
Subject: [PATCH] Customizations for mistral

This patchset contains customization of Dockerfiles
of mistral containers

Change-Id: I0ad1e9aa80b563da17275268ca90b618da350b06
Partially-implements: blueprint third-party-plugin-support
---
 docker/mistral/mistral-api/Dockerfile.j2      | 16 ++++++--------
 docker/mistral/mistral-base/Dockerfile.j2     | 21 +++++++------------
 docker/mistral/mistral-engine/Dockerfile.j2   | 18 +++++++---------
 docker/mistral/mistral-executor/Dockerfile.j2 | 18 +++++++---------
 4 files changed, 30 insertions(+), 43 deletions(-)

diff --git a/docker/mistral/mistral-api/Dockerfile.j2 b/docker/mistral/mistral-api/Dockerfile.j2
index c2b3e3ea8a..ed90c8b0bb 100644
--- a/docker/mistral/mistral-api/Dockerfile.j2
+++ b/docker/mistral/mistral-api/Dockerfile.j2
@@ -1,25 +1,21 @@
 FROM {{ namespace }}/{{ image_prefix }}mistral-base:{{ tag }}
 MAINTAINER {{ maintainer }}
+{% import "macros.j2" as macros with context %}
 
 {% if install_type == 'binary' %}
     {% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
-
-RUN yum -y install \
-        openstack-mistral-api \
-    && yum clean all
-
+       {% set mistral_api_packages = ['openstack-mistral-api'] %}
     {% elif base_distro in ['ubuntu'] %}
-
-RUN apt-get -y install --no-install-recommends \
-        mistral-api \
-    && apt-get clean
-
+        {% set mistral_api_packages = ['mistral-api'] %}
     {% endif %}
+RUN {{ macros.install_packages(mistral_api_packages | customizable("packages")) }}
 {% endif %}
 
 COPY extend_start.sh /usr/local/bin/kolla_mistral_extend_start
 RUN chmod 755 /usr/local/bin/kolla_mistral_extend_start
 
+{% block mistral_api_footer %}{% endblock %}
+{% block footer %}{% endblock %}
 {{ include_footer }}
 
 USER mistral
diff --git a/docker/mistral/mistral-base/Dockerfile.j2 b/docker/mistral/mistral-base/Dockerfile.j2
index 782d31c977..a312867991 100644
--- a/docker/mistral/mistral-base/Dockerfile.j2
+++ b/docker/mistral/mistral-base/Dockerfile.j2
@@ -1,21 +1,16 @@
 FROM {{ namespace }}/{{ image_prefix }}openstack-base:{{ tag }}
 MAINTAINER {{ maintainer }}
 
+{% import "macros.j2" as macros with context %}
+
 {% if install_type == 'binary' %}
 
-    {% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
-
-RUN yum -y install \
-        openstack-mistral-common \
-    && yum clean all
-
-    {% elif base_distro in ['ubuntu'] %}
-
-RUN apt-get -y install --no-install-recommends \
-        mistral-common \
-    && apt-get clean
-
-    {% endif %}
+{% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
+    {% set mistral_base_packages = ['openstack-mistral-common'] %}
+{% elif base_distro in ['ubuntu'] %}
+    {% set mistral_base_packages = ['mistral-common'] %}
+{% endif %}
+RUN {{ macros.install_packages(mistral_base_packages | customizable("packages")) }}
 
 {% elif install_type == 'source' %}
 
diff --git a/docker/mistral/mistral-engine/Dockerfile.j2 b/docker/mistral/mistral-engine/Dockerfile.j2
index 7381290f1c..f12f3d8684 100644
--- a/docker/mistral/mistral-engine/Dockerfile.j2
+++ b/docker/mistral/mistral-engine/Dockerfile.j2
@@ -1,21 +1,19 @@
 FROM {{ namespace }}/{{ image_prefix }}mistral-base:{{ tag }}
 MAINTAINER {{ maintainer }}
 
+{% import "macros.j2" as macros with context %}
+
 {% if install_type == 'binary' %}
     {% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
-
-RUN yum -y install \
-        openstack-mistral-engine \
-    && yum clean all
-
+       {% set mistral_engine_packages = ['openstack-mistral-engine'] %}
     {% elif base_distro in ['ubuntu'] %}
-
-RUN apt-get -y install --no-install-recommends \
-        mistral-engine \
-    && apt-get clean
-
+        {% set mistral_engine_packages = ['mistral-engine'] %}
     {% endif %}
+RUN {{ macros.install_packages(mistral_engine_packages | customizable("packages")) }}
+
 {% endif %}
+{% block mistral_engine_footer %}{% endblock %}
+{% block footer %}{% endblock %}
 
 {{ include_footer }}
 
diff --git a/docker/mistral/mistral-executor/Dockerfile.j2 b/docker/mistral/mistral-executor/Dockerfile.j2
index b5b426cddc..3ab176bd73 100644
--- a/docker/mistral/mistral-executor/Dockerfile.j2
+++ b/docker/mistral/mistral-executor/Dockerfile.j2
@@ -1,21 +1,19 @@
 FROM {{ namespace }}/{{ image_prefix }}mistral-base:{{ tag }}
 MAINTAINER {{ maintainer }}
 
+{% import "macros.j2" as macros with context %}
+
 {% if install_type == 'binary' %}
     {% if base_distro in ['centos', 'fedora', 'oraclelinux', 'rhel'] %}
-
-RUN yum -y install \
-        openstack-mistral-executor \
-    && yum clean all
-
+       {% set mistral_executor_packages = ['openstack-mistral-executor'] %}
     {% elif base_distro in ['ubuntu'] %}
-
-RUN apt-get -y install --no-install-recommends \
-        mistral-executor \
-    && apt-get clean
-
+        {% set mistral_executor_packages = ['mistral-executor'] %}
     {% endif %}
+RUN {{ macros.install_packages(mistral_executor_packages | customizable("packages")) }}
+
 {% endif %}
+{% block mistral_executor_footer %}{% endblock %}
+{% block footer %}{% endblock %}
 
 {{ include_footer }}