]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Roll back if VMAX masking view not created
authorXing Yang <xing.yang@emc.com>
Mon, 19 Jan 2015 18:27:31 +0000 (13:27 -0500)
committerXing Yang <xing.yang@emc.com>
Mon, 2 Feb 2015 19:07:04 +0000 (14:07 -0500)
commite62eaccf7a74a786ce92f22620f840eaeb145d1f
tree28e8871720cb93fe4a268ff14c2817e19810a409
parent09b1ff209172299bf372efe08482bf6b8316817e
Roll back if VMAX masking view not created

In emc_vmax_masking.py, an exception object is created if
there is a failure in the masking view creation logic.
However, the exception is not raised. As a result,
initialize_connection will succeed in spite of a failure
but the VM will not be able to access the disk.
This patch adds rollback logic to handle failures in
creating masking view. If the masking view cannot be
created, the volume will be added back to the default
storage group and an exception will be raised.

Closes-Bug: #1393540
Change-Id: I1a00d8922cc876a943fb7822cd86c580d8e6beac
cinder/tests/test_emc_vmax.py
cinder/volume/drivers/emc/emc_vmax_common.py
cinder/volume/drivers/emc/emc_vmax_masking.py