Drop extra loop which modifies Cinder volume status

There is an existing loop which sets the proper value for status
and attach_status right above, so this is doing nothing other than
changing it to the incorrect value.

Co-Authored-By: Mohammed Naser <mnaser@vexxhost.com>
Change-Id: Iea0c1ea0a699b9519f66977391202956f17aac66
(cherry picked from commit 8cd64670ea)
This commit is contained in:
melanie witt 2018-02-01 01:12:08 +00:00 committed by Mohammed Naser
parent e708799c04
commit 825d19abd1
1 changed files with 0 additions and 12 deletions

View File

@ -1387,12 +1387,6 @@ class CinderFixture(fixtures.Fixture):
'size': 1
}
# update the status based on existing attachments
has_attachment = any(
[volume['id'] in attachments
for attachments in self.attachments.values()])
volume['status'] = 'attached' if has_attachment else 'detached'
# Check for our special image-backed volume.
if volume_id == self.IMAGE_BACKED_VOL:
# Make it a bootable volume.
@ -1567,12 +1561,6 @@ class CinderFixtureNewAttachFlow(fixtures.Fixture):
'size': 1
}
# update the status based on existing attachments
has_attachment = any(
[volume['id'] in attachments
for attachments in self.attachments.values()])
volume['status'] = 'attached' if has_attachment else 'detached'
# Check for our special image-backed volume.
if volume_id == self.IMAGE_BACKED_VOL:
# Make it a bootable volume.