]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commit
purge-docker: fix ceph-osd-zap name container
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:09:11 +0000 (22:09 +0100)
commit3b2f6c34e42eae4033a209d819620211dc68c34b
tree5a3163a45ac8fc9169203b87abda6f0eccebbac3
parentdd0c98c5a2e9e26bca60e00564ea2018984545f6
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>
infrastructure-playbooks/purge-docker-cluster.yml