Dan Kenigsberg has posted comments on this change. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses ......................................................................
Patch Set 14: (1 comment) pep8 is unhappy: 20:01:55 ./vdsm/storage/hsm.py:2943:80: E501 line too long (82 > 79 characters) 20:01:55 iscsi.getTargetString(target.portal, target.tpgt, target.iqn)) https://gerrit.ovirt.org/#/c/65707/14/tests/iscsiTests.py File tests/iscsiTests.py: Line 65: Line 66: def test_discoverydb_discover(self): Line 67: def _fakeRunCmd(*ignored): Line 68: rc = 0 Line 69: out = ['[2001:db8:ca2:2:1::79]:3260,1 iqn.2014-06.com.example:t1', > I do not think that IPv6 testing is relevant to iscsi. This function parses the output of discovery. The RFC gives examples of the outputs of discovery. I see value in top-level test of the top level function, regardless of the hosttail_* implementation. But it's not a hard requirement from me. Line 70: '192.168.122.81:3261,2 iqn.2016-08.com.example:t2'] Line 71: error = '' Line 72: return rc, out, error Line 73: -- To view, visit https://gerrit.ovirt.org/65707 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0e9fa8d8f5ac34753ad6d40625e64dfeee1a3c02 Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dominik Holler <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dominik Holler <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Kaul <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
