diff options
| author | Juan Antonio Osorio Robles <jaosorior@redhat.com> | 2017-07-31 12:27:18 +0300 |
|---|---|---|
| committer | Juan Antonio Osorio Robles <jaosorior@redhat.com> | 2017-07-31 16:44:59 +0300 |
| commit | fd0af55cb105d53e77caa5ac5072a2b2e34fefd4 (patch) | |
| tree | 1db7d3ff59c63f2d7bb6cdf35c597ae780b193db | |
| parent | c89ed0842948e4afa2b100a542e804706f3217db (diff) | |
| download | kolla-fd0af55cb105d53e77caa5ac5072a2b2e34fefd4.tar.gz kolla-fd0af55cb105d53e77caa5ac5072a2b2e34fefd4.zip | |
Add httpd to the Barbican API image
We are running barbican over httpd in TripleO, and would like to
continue doing so when using containerized barbican. Thus, we need this
package in the image as well.
Change-Id: I7001855cffeed264cb45305e2aa6fac139ea05c9
| -rw-r--r-- | docker/barbican/barbican-api/Dockerfile.j2 | 16 | ||||
| -rwxr-xr-x | docker/barbican/barbican-api/extend_start.sh | 13 | ||||
| -rw-r--r-- | releasenotes/notes/use-barbican-with-httpd-d6d7599c07315e32.yaml | 4 |
3 files changed, 32 insertions, 1 deletions
diff --git a/docker/barbican/barbican-api/Dockerfile.j2 b/docker/barbican/barbican-api/Dockerfile.j2 index 39cefda41..4a0d53faa 100644 --- a/docker/barbican/barbican-api/Dockerfile.j2 +++ b/docker/barbican/barbican-api/Dockerfile.j2 @@ -8,18 +8,32 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build {% if install_type == 'binary' %} {% if base_distro in ['centos', 'oraclelinux', 'rhel'] %} {% set barbican_api_packages = [ + 'httpd', + 'mod_ssl', + 'mod_wsgi', 'openstack-barbican-api', 'uwsgi-plugin-python' ] %} + +{{ macros.install_packages(barbican_api_packages | customizable("packages")) }} + +RUN sed -i -r 's,^(Listen 80),#\1,' /etc/httpd/conf/httpd.conf \ + && sed -i -r 's,^(Listen 443),#\1,' /etc/httpd/conf.d/ssl.conf + {% elif base_distro in ['debian', 'ubuntu'] %} {% set barbican_api_packages = [ + 'apache2', 'barbican-api', + 'libapache2-mod-wsgi', 'uwsgi-plugin-python' ] %} - {% endif %} {{ macros.install_packages(barbican_api_packages | customizable("packages")) }} +RUN echo > /etc/apache2/ports.conf + + {% endif %} + {% endif %} COPY extend_start.sh /usr/local/bin/kolla_barbican_extend_start diff --git a/docker/barbican/barbican-api/extend_start.sh b/docker/barbican/barbican-api/extend_start.sh index c541d0343..687c66ddb 100755 --- a/docker/barbican/barbican-api/extend_start.sh +++ b/docker/barbican/barbican-api/extend_start.sh @@ -6,3 +6,16 @@ if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then barbican-db-manage upgrade exit 0 fi + +# Assume the service runs on top of Apache when user is root +if [[ "$(whoami)" == 'root' ]]; then + # NOTE(pbourke): httpd will not clean up after itself in some cases which + # results in the container not being able to restart. (bug #1489676, 1557036) + if [[ "${KOLLA_BASE_DISTRO}" =~ debian|ubuntu ]]; then + # Loading Apache2 ENV variables + . /etc/apache2/envvars + rm -rf /var/run/apache2/* + else + rm -rf /var/run/httpd/* /run/httpd/* /tmp/httpd* + fi +fi diff --git a/releasenotes/notes/use-barbican-with-httpd-d6d7599c07315e32.yaml b/releasenotes/notes/use-barbican-with-httpd-d6d7599c07315e32.yaml new file mode 100644 index 000000000..12782ef74 --- /dev/null +++ b/releasenotes/notes/use-barbican-with-httpd-d6d7599c07315e32.yaml @@ -0,0 +1,4 @@ +--- +other: + - | + HTTPd was added to the barbican-api image. |
