Skip to content

Commit

Permalink
tests: ceph-disk destroy needs --purge
Browse files Browse the repository at this point in the history
The former semantic of ceph-disk destroy is now implemented with the
--purge flag. Use that for the ceph-disk suite.

Signed-off-by: Loic Dachary <[email protected]>
  • Loading branch information
ldachary committed Jul 14, 2017
1 parent 8eed436 commit d199ced
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions qa/workunits/ceph-disk/ceph-disk-test.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
# ln -sf /home/ubuntu/ceph/systemd/[email protected] /usr/lib/systemd/system/[email protected]
# ceph-disk.conf will be silently ignored if it is a symbolic link or a hard link /var/log/upstart for logs
# cp /home/ubuntu/ceph/src/upstart/ceph-disk.conf /etc/init/ceph-disk.conf
# id=3 ; ceph-disk deactivate --deactivate-by-id $id ; ceph-disk destroy --zap --destroy-by-id $id
# id=3 ; ceph-disk deactivate --deactivate-by-id $id ; ceph-disk destroy --purge --zap --destroy-by-id $id
# py.test -s -v -k test_activate_dmcrypt_luks ceph-disk-test.py
#
# CentOS 7
Expand Down Expand Up @@ -172,7 +172,7 @@ def destroy_osd(self, uuid):
self.sh("""
set -xe
ceph-disk --verbose deactivate --deactivate-by-id {id}
ceph-disk --verbose destroy --destroy-by-id {id} --zap
ceph-disk --verbose destroy --purge --destroy-by-id {id} --zap
""".format(id=id))

def deactivate_osd(self, uuid):
Expand Down Expand Up @@ -298,7 +298,7 @@ def test_destroy_osd_by_dev_path(self):
assert partition['state'] == 'active'
c.sh("ceph-disk --verbose deactivate " + partition['path'])
c.wait_for_osd_down(osd_uuid)
c.sh("ceph-disk --verbose destroy " + partition['path'] + " --zap")
c.sh("ceph-disk --verbose destroy --purge " + partition['path'] + " --zap")

def test_deactivate_reactivate_dmcrypt_plain(self):
c = CephDisk()
Expand Down

0 comments on commit d199ced

Please sign in to comment.