]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fixed a concurrency issue in VMAX driver
authorXing Yang <xing.yang@emc.com>
Fri, 20 Feb 2015 06:38:09 +0000 (01:38 -0500)
committerXing Yang <xing.yang@emc.com>
Fri, 13 Mar 2015 17:40:21 +0000 (13:40 -0400)
commit8113c8e1b153ad20ebf1692e8a8b903515d974c6
tree254bd66d7d87cb2c71991dd751491465aefe2ef8
parent99c0701ae1f5f3c628363cd5b58a03346e4b8357
Fixed a concurrency issue in VMAX driver

This patch fixed the following problem:

When trying to add a second volume to the same masking view,
the first volume got removed at the same time, causing
the operation on the second volume to fail.

When two attach requests happen at the same time on the same
volume, the second one will fail.

Also fixed a W503 pep8 issue (line break before binary operator)
in emc_vmax_common.py.

Closes-Bug: #1416035
Closes-Bug: #1403160
Change-Id: I52975b399c2bd8e2a91bdd09004ee277e54c9a89
cinder/tests/test_emc_vmax.py
cinder/volume/drivers/emc/emc_vmax_common.py
cinder/volume/drivers/emc/emc_vmax_masking.py