tree e71822977f068ac5f649e6553503a0aef70ea519
parent 3f65eb4c2c1cb7288e99762c0156f85e69fde311
author yatin <ykarel@redhat.com> 1522142376 +0530
committer Vasyl Saienko <vsaienko@mirantis.com> 1659800383 +0000

Use addCleanup to detach volume instead of explicit detach

Cinder volume delete [1] is executed during resource cleanup.
So, if test fails due to some reason then it tries to delete a volume
which is "in-use" state. Detach volume should be executed before
trying to delete the volume. This patch adds nova_detach volume
as a cleanup task.

[1] https://github.com/openstack/barbican-tempest-plugin/blob/a6d945d8e677dedf73ccb7b2f7271e8fc96836be/barbican_tempest_plugin/tests/scenario/manager.py#L229

Change-Id: I3457bb6fc22a7bb6a32c8d6f4ea5879aeb8d43c5
Related-Bug: #1755485