Revert "docker: Retry OSD disk prepare to workaround race condition"

diff --git a/roles/ceph-osd/tasks/docker/start_docker_osd.yml b/roles/ceph-osd/tasks/docker/start_docker_osd.yml
index 2b0aad3bd47f67cf95b4a60c95cf580c0e0e891f..3423a94523bbbad4d18115d05ed79027fc6f8f6d 100644 (file)
--- a/roles/ceph-osd/tasks/docker/start_docker_osd.yml
+++ b/roles/ceph-osd/tasks/docker/start_docker_osd.yml
@@ -21,12 +21,10 @@
 # starting the next task
 - name: prepare ceph osd disk
   shell: |
-    NAME="ceph-osd-prepare-{{ ansible_hostname }}-dev{{ item.0 | regex_replace('/', '') }}"
-    docker ps -a | grep -sq $NAME && docker rm $NAME || true
     docker run --net=host \
     --pid=host \
     --privileged=true \
-    --name=$NAME \
+    --name="ceph-osd-prepare-{{ ansible_hostname }}-dev{{ item.0 | regex_replace('/', '') }}" \
     -v /etc/ceph:/etc/ceph \
     -v /var/lib/ceph/:/var/lib/ceph/ \
     -v /dev:/dev \
@@ -43,20 +41,13 @@
     - item.1.get("rc", 0) != 0
     - ceph_osd_docker_prepare_env is defined
     - not osd_containerized_deployment_with_kv
-  # https://github.com/ceph/ceph-ansible/issues/1495
-  register: prepare_osd_disk
-  until: prepare_osd_disk.rc == 0
-  retries: 5
-  delay: 3
 
 - name: prepare ceph osd disk with kv_store
   shell: |
-    NAME="ceph-osd-prepare-{{ ansible_hostname }}-dev-{{ item.0 | regex_replace('/', '') }}"
-    docker ps -a | grep -sq $NAME && docker rm $NAME || true
     docker run  --net=host   \
     --pid=host \
     --privileged=true \
-    --name=$NAME \
+    --name="ceph-osd-prepare-{{ ansible_hostname }}-dev-{{ item.0 | regex_replace('/', '') }}" \
     -v /dev:/dev \
     -v /etc/localtime:/etc/localtime:ro \
     -e "OSD_DEVICE={{ item.0 }}" \
@@ -75,11 +66,6 @@
     - item.1.get("rc", 0) != 0
     - ceph_osd_docker_prepare_env is defined
     - osd_containerized_deployment_with_kv
-  # https://github.com/ceph/ceph-ansible/issues/1495
-  register: prepare_osd_disk_kv
-  until: prepare_osd_disk_kv.rc == 0
-  retries: 5
-  delay: 3
 
 - name: generate ceph osd docker run script
   become: true