openstack
/
os-brick
Code Issues Proposed changes

Merge "Improve error handling on target query"

This commit is contained in:
Zuul 2021-01-08 20:12:30 +00:00 committed by Gerrit Code Review
parent 82909f4824 4fabe1b33d
commit ec70b4092f
2 changed files with 19 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
os_brick/initiator/connectors/iscsi.py
View File

@ -156,7 +156,10 @@ class ISCSIConnector(base.BaseLinuxConnector, base_iscsi.BaseISCSIConnector):
for line in out.splitlines():
if line:
info = line.split()
lines.append((info[0].split(',')[0], info[1]))
try:
lines.append((info[0].split(',')[0], info[1]))
except IndexError:
pass
return lines
def _get_iscsi_sessions(self):

15
os_brick/tests/initiator/connectors/test_iscsi.py
View File

@ -119,6 +119,21 @@ class ISCSIConnectorTestCase(test_connector.ConnectorTestCase):
res = self.connector._get_iscsi_nodes()
self.assertEqual([], res)
@mock.patch.object(iscsi.ISCSIConnector, '_execute')
def test_get_iscsi_nodes_corrupt(self, exec_mock):
iscsiadm_result = ('ip1:port1,-1 tgt1\n'
'ip2:port2,-1 tgt2\n'
'[]:port3,-1\n'
'ip4:port4,-1 tgt4\n')
exec_mock.return_value = (iscsiadm_result, '')
res = self.connector._get_iscsi_nodes()
expected = [('ip1:port1', 'tgt1'), ('ip2:port2', 'tgt2'),
('ip4:port4', 'tgt4')]
self.assertListEqual(expected, res)
exec_mock.assert_called_once_with(
'iscsiadm', '-m', 'node', run_as_root=True,
root_helper=self.connector._root_helper, check_exit_code=False)
@mock.patch.object(iscsi.ISCSIConnector, '_get_ips_iqns_luns')
@mock.patch('glob.glob')
@mock.patch.object(iscsi.ISCSIConnector, '_get_iscsi_sessions_full')