diff options
| author | Marcin Juszkiewicz <marcin@juszkiewicz.com.pl> | 2019-08-16 13:12:31 +0200 |
|---|---|---|
| committer | Marcin Juszkiewicz <marcin@juszkiewicz.com.pl> | 2019-08-19 18:17:06 +0200 |
| commit | f4e0bbd3fb31fa6d24b0116e32f662691645a00d (patch) | |
| tree | d9c080032dcb52c5efa7d8c2001fdb6f0c743a20 | |
| parent | be2bc668667b623f580af2a28fe7caaa15ef06b8 (diff) | |
| download | kolla-f4e0bbd3fb31fa6d24b0116e32f662691645a00d.tar.gz kolla-f4e0bbd3fb31fa6d24b0116e32f662691645a00d.zip | |
fluentd: there is no 'td-agent' for Debian/x86-64
As there is no 'td-agent' package image build fails on 'x86-64'.
We have to wait for TD Agent 4.
Change-Id: I03d38628eee0f0823a58cba3a70df22f29022956
| -rw-r--r-- | docker/fluentd/Dockerfile.j2 | 18 | ||||
| -rw-r--r-- | docker/macros.j2 | 2 |
2 files changed, 8 insertions, 12 deletions
diff --git a/docker/fluentd/Dockerfile.j2 b/docker/fluentd/Dockerfile.j2 index 3f6bd7056..98b07ea62 100644 --- a/docker/fluentd/Dockerfile.j2 +++ b/docker/fluentd/Dockerfile.j2 @@ -5,6 +5,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build {% import "macros.j2" as macros with context %} +{% set fluentd_user = 'fluentd' %} + {% if base_package_type == 'rpm' %} {{ macros.configure_user(name='fluentd', groups='mysql') }} @@ -32,7 +34,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build 'ruby', 'ruby-dev', ] %} - {% if base_arch == 'x86_64' %} + {% if base_arch == 'x86_64' and base_distro == 'ubuntu' %} + {% set fluentd_user = 'td-agent' %} {{ macros.configure_user(name='td-agent', groups='mysql') }} {% set fluentd_packages = fluentd_packages + [ 'td-agent' @@ -45,10 +48,6 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build {{ macros.install_packages(fluentd_packages | customizable("packages")) }} -{% if base_package_type == 'deb' and base_arch != 'x86_64' %} -RUN /usr/bin/gem install fluentd --no-rdoc --no-ri -{% endif %} - # Distro specific files and operations {% if base_package_type == 'rpm' %} COPY rpm_fluentd_sudoers /etc/sudoers.d/kolla_fluentd_sudoers @@ -56,13 +55,14 @@ RUN chmod 440 /etc/sudoers.d/kolla_fluentd_sudoers \ && mkdir -p /var/run/fluentd \ && chown -R fluentd: /etc/fluentd /var/run/fluentd {% elif base_package_type == 'deb' %} - {% if base_arch == 'x86_64' %} + {% if base_arch == 'x86_64' and base_distro == 'ubuntu' %} COPY fluentd_sudoers /etc/sudoers.d/kolla_fluentd_sudoers RUN chmod 440 /etc/sudoers.d/kolla_fluentd_sudoers \ && sed -i -e "s/USER=td-agent/USER=root/" -e "s/GROUP=td-agent/GROUP=root/" /etc/init.d/td-agent \ && rm -f /etc/td-agent/td-agent.conf \ && chown -R td-agent: /etc/td-agent {% else %} +RUN /usr/bin/gem install fluentd --no-rdoc --no-ri COPY rpm_fluentd_sudoers /etc/sudoers.d/kolla_fluentd_sudoers RUN chmod 440 /etc/sudoers.d/kolla_fluentd_sudoers \ && mkdir -p /etc/fluentd \ @@ -106,8 +106,4 @@ RUN tar -xvf /tmp/fluentd-monasca.tar.gz -C /tmp \ {% block fluentd_footer %}{% endblock %} {% block footer %}{% endblock %} -{% if base_package_type == 'rpm' or (base_package_type == 'deb' and base_arch != 'x86_64') %} -USER fluentd -{% elif base_package_type == 'deb' and base_arch == 'x86_64' %} -USER td-agent -{% endif %} +USER {{ fluentd_user }} diff --git a/docker/macros.j2 b/docker/macros.j2 index 25d3a3752..50adf8bb6 100644 --- a/docker/macros.j2 +++ b/docker/macros.j2 @@ -111,7 +111,7 @@ RUN apt-get update \ {%- if base_distro in ['centos', 'oraclelinux', 'rhel'] -%} ulimit -n 65536 && gem install --minimal-deps {{ plugins | join(' ') }} {%- elif base_distro in ['debian', 'ubuntu'] -%} - {%- if base_arch == 'x86_64' -%} + {%- if base_arch == 'x86_64' and base_distro == 'ubuntu' -%} ulimit -n 65536 && td-agent-gem install {{ plugins | join(' ') }} {%- else -%} ulimit -n 65536 && gem install --minimal-deps {{ plugins | join(' ') }} |