]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commit
purge-docker: fix ceph-osd-zap name container v3.0.23
authorGuillaume Abrioux <gabrioux@redhat.com>
Fri, 2 Feb 2018 10:55:18 +0000 (11:55 +0100)
committerGuillaume Abrioux <gabrioux@redhat.com>
Fri, 2 Feb 2018 21:14:03 +0000 (22:14 +0100)
commit28c11c56fe47823b1e047b2928c8d065b2e58b24
treea1596470624f2c00645883309f59c6009465e287
parent3c83368685b4a6e576b025b012d01343e141aba9
purge-docker: fix ceph-osd-zap name container

the `zap ceph osd disks` task should iter on `resolved_parent_device`
instead of `combined_devices_list` which contain only the base device
name (vs. full path name in `combined_devices_list`).

this fixes the issue where docker complain about container name because
of illegal characters such as `/` :
```
"/usr/bin/docker-current: Error response from daemon: Invalid container
name (ceph-osd-zap-magna074-/dev/sdb1), only [a-zA-Z0-9][a-zA-Z0-9_.-]
are allowed.","See '/usr/bin/docker-current run --help'."
""
```

having the the basename of the device path is enough for the container
name.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1540137
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 3b2f6c34e42eae4033a209d819620211dc68c34b)
infrastructure-playbooks/purge-docker-cluster.yml