openstack/cinder
Code Issues Proposed changes

Merge "cinder-rtstool: should use acl.node_wwn"

Browse Source
This commit is contained in:
Jenkins 2015-01-08 18:41:15 +00:00 committed by Gerrit Code Review
commit 9742b7d361
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/cmd/rtstool.py
View File

@ -100,7 +100,7 @@ def add_initiator(target_iqn, initiator_iqn, userid, password):
tpg = target.tpgs.next() # get the first one
for acl in tpg.node_acls:
# See if this ACL configuration already exists
if acl['node_wwn'] == initiator_iqn:
if acl.node_wwn == initiator_iqn:
# No further action required
return

2
cinder/tests/test_cmd.py
View File

@ -844,7 +844,7 @@ class TestCinderRtstoolCmd(test.TestCase):
target_iqn = mock.MagicMock()
target_iqn.tpgs.return_value = \
[{'node_acls': mock.sentinel.initiator_iqn}]
acl = {'node_wwn': mock.sentinel.initiator_iqn}
acl = mock.MagicMock(node_wwn=mock.sentinel.initiator_iqn)
tpg = mock.MagicMock(node_acls=[acl])
tpgs = mock.MagicMock()
tpgs.next.return_value = tpg