tree 9c8c34c76906a178c794013334e0642d0355f15e
parent fb86ef6b36bf77bd872d6e3074f6f1d69ffb6f1b
author yatin <ykarel@redhat.com> 1522142376 +0530
committer Vasyl Saienko <vsaienko@mirantis.com> 1659800241 +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