diff options
| author | Eduardo Gonzalez <dabarren@gmail.com> | 2017-06-12 15:56:58 +0100 |
|---|---|---|
| committer | Eduardo Gonzalez <dabarren@gmail.com> | 2017-06-13 13:44:05 +0200 |
| commit | e759d4d53964755b261d2236b498e881bd49467b (patch) | |
| tree | eee329ff26baab469964c9732885cfe59f36a9f4 /docker | |
| parent | a3ab3f255785b7c4b761859b06952f3a3184face (diff) | |
| download | kolla-e759d4d53964755b261d2236b498e881bd49467b.tar.gz kolla-e759d4d53964755b261d2236b498e881bd49467b.zip | |
Fix kuryr docker images
Kuryr need ovs and bridge packages to
create local bridges and ports for containers.
Creates kuryr user and adds permissions
Remove useless bootstrap tasks
Closes-Bug: #1697475
Change-Id: I4c08a21df263fdefe1fe991cb7ad41cfee65019d
Diffstat (limited to 'docker')
| -rw-r--r-- | docker/kuryr/kuryr-base/Dockerfile.j2 | 7 | ||||
| -rw-r--r-- | docker/kuryr/kuryr-base/extend_start.sh | 6 | ||||
| -rw-r--r-- | docker/kuryr/kuryr-libnetwork/Dockerfile.j2 | 16 |
3 files changed, 24 insertions, 5 deletions
diff --git a/docker/kuryr/kuryr-base/Dockerfile.j2 b/docker/kuryr/kuryr-base/Dockerfile.j2 index 8611a64a9..a7ed3bc09 100644 --- a/docker/kuryr/kuryr-base/Dockerfile.j2 +++ b/docker/kuryr/kuryr-base/Dockerfile.j2 @@ -5,6 +5,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build {% import "macros.j2" as macros with context %} +{{ macros.configure_user(name='kuryr') }} + {% if install_type == 'binary' %} RUN echo '{{ install_type }} not yet available for {{ base_distro }}' \ @@ -22,8 +24,9 @@ ADD kuryr-base-archive /kuryr-base-source # Otherwise, it will be failed. RUN ln -s kuryr-base-source/* kuryr-base \ && sed -i 's|^kuryr-lib===.*$||g' requirements/upper-constraints.txt \ - && {{ macros.install_pip(kuryr_base_pip_packages | customizable("pip_packages")) }} - + && {{ macros.install_pip(kuryr_base_pip_packages | customizable("pip_packages")) }} \ + && mkdir -p /etc/kuryr \ + && chown -R kuryr: /etc/kuryr {% endif %} COPY extend_start.sh /usr/local/bin/kolla_extend_start diff --git a/docker/kuryr/kuryr-base/extend_start.sh b/docker/kuryr/kuryr-base/extend_start.sh index 214675bea..b109ce4f6 100644 --- a/docker/kuryr/kuryr-base/extend_start.sh +++ b/docker/kuryr/kuryr-base/extend_start.sh @@ -1,6 +1,7 @@ #!/bin/bash KURYR_LOG_DIR=/var/log/kolla/kuryr +KURYR_DOCKER_PLUGINS_DIR=/usr/lib/docker/plugins/kuryr if [[ ! -d "${KURYR_LOG_DIR}" ]]; then mkdir -p ${KURYR_LOG_DIR} @@ -9,7 +10,6 @@ if [[ $(stat -c %a ${KURYR_LOG_DIR}) != "755" ]]; then chmod 755 ${KURYR_LOG_DIR} fi -if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then - mkdir -p /usr/lib/docker/plugins/kuryr - exit 0 +if [[ ! -d "${KURYR_DOCKER_PLUGINS_DIR}" ]]; then + mkdir -p ${KURYR_DOCKER_PLUGINS_DIR} fi diff --git a/docker/kuryr/kuryr-libnetwork/Dockerfile.j2 b/docker/kuryr/kuryr-libnetwork/Dockerfile.j2 index 18729576a..7cd9aca24 100644 --- a/docker/kuryr/kuryr-libnetwork/Dockerfile.j2 +++ b/docker/kuryr/kuryr-libnetwork/Dockerfile.j2 @@ -11,6 +11,22 @@ RUN echo '{{ install_type }} not yet available for {{ base_distro }}' \ && /bin/false {% elif install_type == 'source' %} + {% if base_distro in ['centos', 'oraclelinux', 'rhel'] %} + + {% set kuryr_libnetwork_packages = [ + 'bridge-utils', + 'openvswitch' + ] %} + + {% elif base_distro in ['debian', 'ubuntu'] %} + + {% set kuryr_libnetwork_packages = [ + 'bridge-utils', + 'openvswitch-switch' + ] %} + + {% endif %} +{{ macros.install_packages(kuryr_libnetwork_packages | customizable("packages")) }} ADD kuryr-libnetwork-archive /kuryr-libnetwork-source |