]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
Revert "osd: container remove --pid=host"
authorGuillaume Abrioux <gabrioux@redhat.com>
Thu, 14 Feb 2019 09:23:19 +0000 (10:23 +0100)
committermergify[bot] <mergify[bot]@users.noreply.github.com>
Thu, 14 Feb 2019 10:34:37 +0000 (10:34 +0000)
This reverts commit bb2bbeb94115692ea072a1f6e2aed2c6c5336cec.

Looks like when not passing `--pid=host` we are facing some issues when
deploying more than 2 OSDs in containerized environment.

At the moment, we are still troubleshooting this issue but we prefer to
revert this commit so it doesn't block any PR in the CI.

As soon as we have a fix; we will push a new PR to remove `--pid=host`
(a revert of revert...)

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
roles/ceph-osd/templates/ceph-osd-run.sh.j2

index 364b048cf2c1b681db0720b1ab1d99329ffaadab..5678d62bfad305da8ac56141be80af6271d219d2 100644 (file)
@@ -74,6 +74,7 @@ fi
   --rm \
   --net=host \
   --privileged=true \
+  --pid=host \
   --memory={{ ceph_osd_docker_memory_limit }} \
   {% if (container_binary == 'docker' and ceph_docker_version.split('.')[0] is version_compare('13', '>=')) or container_binary == 'podman' -%}
   --cpus={{ ceph_osd_docker_cpu_limit }} \