Merge "Remove access_mode 'rw' setting in drivers"

This commit is contained in:
Jenkins 2016-02-11 06:14:44 +00:00 committed by Gerrit Code Review
commit 0a04f5a395
21 changed files with 22 additions and 83 deletions

View File

@ -3297,8 +3297,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_domains.called)
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_controller.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3341,8 +3340,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_domains.called)
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_controller.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3386,8 +3384,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_domains.called)
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_ctrl.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3506,8 +3503,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_domains.called)
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_controller.called)
expected = {'access_mode': 'ro',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3549,8 +3545,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_controller.called)
self.assertTrue(mock_is_virtualport_mode.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3598,8 +3593,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_controller_port_iscsi_config.called)
self.assertTrue(mock_find_active_controller.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3677,8 +3671,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_controller.called)
self.assertTrue(mock_find_controller_port_iscsi_config.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3722,8 +3715,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_ctrl.called)
self.assertTrue(mock_find_controller_port_iscsi_config.called)
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3765,8 +3757,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_ctrl_port.called)
self.assertTrue(mock_find_active_controller.called)
self.assertTrue(mock_find_iscsi_config.called)
expected = {'access_mode': 'ro',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':
@ -3811,8 +3802,7 @@ class DellSCSanAPITestCase(test.TestCase):
self.assertTrue(mock_find_controller_port_iscsi_config.called)
# Since we're feeding the same info back multiple times the information
# will be duped.
expected = {'access_mode': 'rw',
'target_discovered': False,
expected = {'target_discovered': False,
'target_iqn':
u'iqn.2002-03.com.compellent:5000d31000fcbe43',
'target_iqns':

View File

@ -176,7 +176,6 @@ class InfortrendCLITestData(object):
'target_discovered': True,
'target_lun': fake_lun_map[0],
'target_wwn': fake_target_wwpns[0:2],
'access_mode': 'rw',
'initiator_target_map': test_initiator_target_map,
},
}
@ -192,7 +191,6 @@ class InfortrendCLITestData(object):
'target_discovered': True,
'target_lun': fake_lun_map[0],
'target_wwn': [fake_target_wwpns[1]],
'access_mode': 'rw',
'initiator_target_map': test_initiator_target_map_specific_channel,
},
}
@ -215,7 +213,6 @@ class InfortrendCLITestData(object):
'target_discovered': True,
'target_lun': fake_lun_map[0],
'target_wwn': test_target_wwpns_map_multipath_r_model[:],
'access_mode': 'rw',
'initiator_target_map':
test_initiator_target_map_multipath_r_model,
},
@ -234,7 +231,6 @@ class InfortrendCLITestData(object):
'target_discovered': True,
'target_lun': fake_lun_map[0],
'target_wwn': [x.lower() for x in fake_target_wwpns[0:2]],
'access_mode': 'rw',
'initiator_target_map': test_initiator_target_map_zoning,
},
}
@ -252,7 +248,6 @@ class InfortrendCLITestData(object):
'target_discovered': True,
'target_lun': fake_lun_map[0],
'target_wwn': [x.lower() for x in fake_target_wwpns[1:3]],
'access_mode': 'rw',
'initiator_target_map': test_initiator_target_map_zoning_r_model,
},
}

View File

@ -191,7 +191,6 @@ ISCSI_CONNECTION_INFO = {
"driver_volume_type": "iscsi",
"data": {
"target_discovered": False,
"access_mode": "rw",
"discard": True,
"target_luns": [1, 1, 1, 1],
"target_iqns": [TARGET_IQN, TARGET_IQN, TARGET_IQN, TARGET_IQN],
@ -207,7 +206,6 @@ FC_CONNECTION_INFO = {
"target_wwn": FC_WWNS,
"target_lun": 1,
"target_discovered": True,
"access_mode": "rw",
"initiator_target_map": INITIATOR_TARGET_MAP,
"discard": True,
},

View File

@ -356,8 +356,7 @@ class TegileISCSIDriverTestCase(test.TestCase):
'_api_executor',
fake_tegile_backend):
self.assertEqual(
{'data': {'access_mode': 'rw',
'auth_method': 'CHAP',
{'data': {'auth_method': 'CHAP',
'discard': False,
'target_discovered': (False,),
'auth_password': 'test',
@ -388,8 +387,7 @@ class TegileFCDriverTestCase(test.TestCase):
with mock.patch.object(tegile_driver,
'_api_executor',
fake_tegile_backend):
self.assertEqual({'data': {'access_mode': 'rw',
'encrypted': False,
self.assertEqual({'data': {'encrypted': False,
'initiator_target_map': {
'21000024ff59bb6e':
['21000024ff578701'],

View File

@ -1040,7 +1040,6 @@ class XIOISEDriverTestCase(object):
exp_result = {"driver_volume_type": "iscsi",
"data": {"target_lun": '1',
"volume_id": '1',
"access_mode": 'rw',
"target_discovered": False,
"target_iqn": ISE_IQN,
"target_portal": ISE_ISCSI_IP1 + ":3260"}}
@ -1048,7 +1047,6 @@ class XIOISEDriverTestCase(object):
exp_result = {"driver_volume_type": "fibre_channel",
"data": {"target_lun": '1',
"volume_id": '1',
"access_mode": 'rw',
"target_discovered": True,
"initiator_target_map": ISE_INIT_TARGET_MAP,
"target_wwn": ISE_TARGETS}}
@ -1071,7 +1069,6 @@ class XIOISEDriverTestCase(object):
exp_result = {"driver_volume_type": "iscsi",
"data": {"target_lun": '1',
"volume_id": '1',
"access_mode": 'rw',
"target_discovered": False,
"target_iqn": ISE_IQN,
"target_portal": ISE_ISCSI_IP1 + ":3260"}}
@ -1079,7 +1076,6 @@ class XIOISEDriverTestCase(object):
exp_result = {"driver_volume_type": "fibre_channel",
"data": {"target_lun": '1',
"volume_id": '1',
"access_mode": 'rw',
"target_discovered": True,
"initiator_target_map": ISE_INIT_TARGET_MAP,
"target_wwn": ISE_TARGETS}}
@ -1102,7 +1098,6 @@ class XIOISEDriverTestCase(object):
exp_result = {"driver_volume_type": "iscsi",
"data": {"target_lun": '1',
"volume_id": '2',
"access_mode": 'rw',
"target_discovered": False,
"target_iqn": ISE_IQN,
"target_portal": ISE_ISCSI_IP1 + ":3260",
@ -1113,7 +1108,6 @@ class XIOISEDriverTestCase(object):
exp_result = {"driver_volume_type": "fibre_channel",
"data": {"target_lun": '1',
"volume_id": '2',
"access_mode": 'rw',
"target_discovered": True,
"initiator_target_map": ISE_INIT_TARGET_MAP,
"target_wwn": ISE_TARGETS}}

View File

@ -2431,9 +2431,6 @@ class ISCSIDriver(VolumeDriver):
present meaning no authentication, or auth_method == `CHAP`
meaning use CHAP with the specified credentials.
:access_mode: the volume access mode allow client used
('rw' or 'ro' currently supported)
:discard: boolean indicating if discard is supported
In some of drivers that support multiple connections (for multipath
@ -2550,7 +2547,6 @@ class ISCSIDriver(VolumeDriver):
'target_iqn': 'iqn.2010-10.org.openstack:volume-00000001',
'target_portal': '127.0.0.0.1:3260',
'volume_id': 1,
'access_mode': 'rw',
'discard': False,
}
}
@ -2571,7 +2567,6 @@ class ISCSIDriver(VolumeDriver):
'target_lun': 1,
'target_luns': [1, 1],
'volume_id': 1,
'access_mode': 'rw',
'discard': False,
}
}
@ -2655,14 +2650,12 @@ class FakeISCSIDriver(ISCSIDriver):
def initialize_connection(self, volume, connector):
return {
'driver_volume_type': 'iscsi',
'data': {'access_mode': 'rw'},
'discard': False,
}
def initialize_connection_snapshot(self, snapshot, connector):
return {
'driver_volume_type': 'iscsi',
'data': {'access_mode': 'rw'}
}
def terminate_connection(self, volume, connector, **kwargs):
@ -2833,7 +2826,6 @@ class FibreChannelDriver(VolumeDriver):
'target_discovered': True,
'target_lun': 1,
'target_wwn': '1234567890123',
'access_mode': 'rw',
'discard': False,
}
}
@ -2846,7 +2838,6 @@ class FibreChannelDriver(VolumeDriver):
'target_discovered': True,
'target_lun': 1,
'target_wwn': ['1234567890123', '0987654321321'],
'access_mode': 'rw',
'discard': False,
}
}

View File

@ -1427,7 +1427,6 @@ class StorageCenterApi(object):
# Our mutable process object.
pdata = {'active': -1,
'up': -1,
'access_mode': 'rw',
'ip': ip,
'port': port}
# Our output lists.
@ -1436,7 +1435,7 @@ class StorageCenterApi(object):
iqns = []
# Process just looks for the best port to return.
def process(lun, iqn, address, port, readonly, status, active):
def process(lun, iqn, address, port, status, active):
"""Process this mapping information.
:param lun: SCSI Lun.
@ -1467,7 +1466,6 @@ class StorageCenterApi(object):
# but we don't actually need the state to be
# up at this point.
if pdata['up'] == -1:
pdata['access_mode'] = 'rw' if readonly is False else 'ro'
if active:
pdata['active'] = len(iqns) - 1
if status == 'Up':
@ -1491,7 +1489,6 @@ class StorageCenterApi(object):
# The lun, ro mode and status are in the mapping.
LOG.debug('mapping: %s', mapping)
lun = mapping.get('lun')
ro = mapping.get('readOnly', False)
status = mapping.get('status')
# Get our IQN from our mapping.
iqn = self._get_iqn(mapping)
@ -1509,7 +1506,7 @@ class StorageCenterApi(object):
portnumber = dom.get('portNumber')
# We have all our information. Process this portal.
process(lun, iqn, ipaddress, portnumber,
ro, status, isactive)
status, isactive)
# Else we are in legacy mode.
elif iqn:
# Need to get individual ports
@ -1523,7 +1520,7 @@ class StorageCenterApi(object):
portnumber = cpconfig.get('portNumber')
# We have all our information. Process this portal.
process(lun, iqn, ipaddress, portnumber,
ro, status, isactive)
status, isactive)
# We've gone through all our mappings.
# Make sure we found something to return.
@ -1553,7 +1550,6 @@ class StorageCenterApi(object):
'target_portals': portals,
'target_lun': luns[pdata['active']],
'target_luns': luns,
'access_mode': pdata['access_mode']
}
LOG.debug('find_iscsi_properties return: %s',
data)

View File

@ -66,7 +66,6 @@ class DellStorageCenterISCSIDriver(dell_storagecenter_common.DellCommonDriver,
# 'target_portals': all portals,
# 'target_lun': preferred lun,
# 'target_luns': all luns,
# 'access_mode': access_mode
# }
# We use id to name the volume name as it is a

View File

@ -145,7 +145,6 @@ class EMCCLIFCDriver(driver.FibreChannelDriver):
'target_discovered': True,
'target_lun': 1,
'target_wwn': '1234567890123',
'access_mode': 'rw'
'initiator_target_map': {
'1122334455667788': ['1234567890123']
}
@ -160,7 +159,6 @@ class EMCCLIFCDriver(driver.FibreChannelDriver):
'target_discovered': True,
'target_lun': 1,
'target_wwn': ['1234567890123', '0987654321321'],
'access_mode': 'rw'
'initiator_target_map': {
'1122334455667788': ['1234567890123',
'0987654321321']

View File

@ -136,7 +136,6 @@ class EMCCLIISCSIDriver(driver.ISCSIDriver):
'target_iqn': 'iqn.2010-10.org.openstack:volume-00000001',
'target_portal': '127.0.0.0.1:3260',
'target_lun': 1,
'access_mode': 'rw'
}
}
@ -150,7 +149,6 @@ class EMCCLIISCSIDriver(driver.ISCSIDriver):
'iqn.2010-10.org.openstack:volume-00002'],
'target_portals': ['127.0.0.1:3260', '127.0.1.1:3260'],
'target_luns': [1, 1],
'access_mode': 'rw'
}
}

View File

@ -847,8 +847,6 @@ class XtremIOISCSIDriver(XtremIOVolumeDriver, driver.ISCSIDriver):
the authentication details. Right now, either auth_method is not
present meaning no authentication, or auth_method == `CHAP`
meaning use CHAP with the specified credentials.
:access_mode: the volume access mode allow client used
('rw' or 'ro' currently supported)
multiple connection return
:target_iqns, :target_portals, :target_luns, which contain lists of
multiple values. The main portal information is also returned in

View File

@ -1346,7 +1346,6 @@ class InfortrendCommon(object):
'target_discovered': True,
'target_lun': int(lun_id),
'target_wwn': target_wwpns,
'access_mode': 'rw',
'initiator_target_map': initiator_target_map,
},
}

View File

@ -158,7 +158,6 @@ class InfortrendCLIFCDriver(driver.FibreChannelDriver):
'target_discovered': True,
'target_lun': 1,
'target_wwn': '1234567890123',
'access_mode': 'rw'
'initiator_target_map': {
'1122334455667788': ['1234567890123']
}
@ -173,7 +172,6 @@ class InfortrendCLIFCDriver(driver.FibreChannelDriver):
'target_discovered': True,
'target_lun': 1,
'target_wwn': ['1234567890123', '0987654321321'],
'access_mode': 'rw'
'initiator_target_map': {
'1122334455667788': ['1234567890123',
'0987654321321']

View File

@ -150,7 +150,6 @@ class InfortrendCLIISCSIDriver(driver.ISCSIDriver):
'target_iqn': 'iqn.2010-10.org.openstack:volume-00000001',
'target_portal': '127.0.0.0.1:3260',
'volume_id': 1,
'access_mode': 'rw'
}
}
"""

View File

@ -38,7 +38,6 @@ class DPLISCSIDriver(dplcommon.DPLCOMMONDriver,
properties['target_portal'] = ''
properties['target_iqn'] = None
properties['volume_id'] = volume['id']
properties['access_mode'] = 'rw'
dpl_server = self.configuration.san_ip
dpl_iscsi_port = self.configuration.iscsi_port

View File

@ -1500,7 +1500,6 @@ class PureISCSIDriver(PureBaseVolumeDriver, san.SanISCSIDriver):
"driver_volume_type": "iscsi",
"data": {
"target_discovered": False,
"access_mode": "rw",
"discard": True,
},
}
@ -1656,7 +1655,6 @@ class PureFCDriver(PureBaseVolumeDriver, driver.FibreChannelDriver):
'target_discovered': True,
"target_lun": connection["lun"],
"target_wwn": target_wwns,
'access_mode': 'rw',
'initiator_target_map': init_targ_map,
"discard": True,
}

View File

@ -528,7 +528,6 @@ class TegileISCSIDriver(TegileIntelliFlashVolumeDriver, san.SanISCSIDriver):
connection_data['target_lun'] = target_lun
connection_data['target_discovered'] = False,
connection_data['volume_id'] = volume['id'],
connection_data['access_mode'] = 'rw'
connection_data['discard'] = False
if getattr(self.configuration, 'use_chap_auth', False):
connection_data['auth_method'] = 'CHAP'
@ -622,7 +621,6 @@ class TegileFCDriver(TegileIntelliFlashVolumeDriver,
'data': {
'encrypted': False,
'target_discovered': False,
'access_mode': 'rw',
'target_lun': target_info['target_lun'],
'target_wwn': ast.literal_eval(target_info['target_wwn']),
'initiator_target_map': ast.literal_eval(initiator_target_map)

View File

@ -152,7 +152,6 @@ class V7000FCPDriver(driver.FibreChannelDriver):
properties['target_discovered'] = True
properties['target_wwn'] = target_wwns
properties['target_lun'] = lun_id
properties['access_mode'] = 'rw'
properties['initiator_target_map'] = init_targ_map
LOG.debug("Return FC data for zone addition: %(properties)s.",

View File

@ -1345,7 +1345,6 @@ class XIOISEDriver(object):
data = {}
data['target_lun'] = target_lun
data['volume_id'] = volume['id']
data['access_mode'] = 'rw'
return data
def ise_unpresent(self, volume, endpoints):

View File

@ -1412,14 +1412,13 @@ class VolumeManager(manager.SchedulerDependentManager):
# Add access_mode to connection info
volume_metadata = self.db.volume_admin_metadata_get(context.elevated(),
volume_id)
if conn_info['data'].get('access_mode') is None:
access_mode = volume_metadata.get('attached_mode')
if access_mode is None:
# NOTE(zhiyan): client didn't call 'os-attach' before
access_mode = ('ro'
if volume_metadata.get('readonly') == 'True'
else 'rw')
conn_info['data']['access_mode'] = access_mode
access_mode = volume_metadata.get('attached_mode')
if access_mode is None:
# NOTE(zhiyan): client didn't call 'os-attach' before
access_mode = ('ro'
if volume_metadata.get('readonly') == 'True'
else 'rw')
conn_info['data']['access_mode'] = access_mode
# Add encrypted flag to connection_info if not set in the driver.
if conn_info['data'].get('encrypted') is None:

View File

@ -67,9 +67,6 @@ class ISCSITarget(driver.Target):
present meaning no authentication, or auth_method == `CHAP`
meaning use CHAP with the specified credentials.
:access_mode: the volume access mode allow client used
('rw' or 'ro' currently supported)
:discard: boolean indicating if discard is supported
In some of drivers that support multiple connections (for multipath
@ -282,7 +279,6 @@ class ISCSITarget(driver.Target):
'target_iqn': 'iqn.2010-10.org.openstack:volume-00000001',
'target_portal': '127.0.0.0.1:3260',
'volume_id': '9a0d35d0-175a-11e4-8c21-0800200c9a66',
'access_mode': 'rw',
'discard': False,
}
}