]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Update EMC SMI-S Driver
authorXing Yang <xing.yang@emc.com>
Wed, 6 Feb 2013 18:32:12 +0000 (13:32 -0500)
committerXing Yang <xing.yang@emc.com>
Fri, 15 Feb 2013 16:28:32 +0000 (11:28 -0500)
commitedbfa6a3a5697da0d39a8cf94cdbf2ed0f726b20
treec8bc8bf283baef89206bee23a45a16f45702a7d2
parent90971cd1026728d3061e13843d117e549c0be67c
Update EMC SMI-S Driver

1. Merged with Avishay's changes(https://review.openstack.org/#/c/
19808/3) and removed copy volume<->image from my driver.
2. Also did some refactoring based on Mike's comments
last time (See https://review.openstack.org/#/c/19979/).
Made the following changes in emc_smis_common:
- Added a member variable conn to save get_ecom_connection
and moved exceptions inside the helper function get_ecom_connection.
- Also moved exceptions inside find_pool and get_storage_type.
- Added test cases so get_ecom_connection, find_pool, and
get_storage_type will be covered.
3. Made changes in emc_smis_common to handle both iscsi and fc.

Change-Id: I56db3ba41215489f6afe840ce5310a31f28bf5ae
cinder/tests/test_emc.py
cinder/volume/driver.py
cinder/volume/drivers/emc/emc_smis_common.py
cinder/volume/drivers/emc/emc_smis_iscsi.py