Yeela Kaplan has posted comments on this change.

Change subject: iscsiadm: Filter out IPv6 addresses when discovering targets
......................................................................


Patch Set 2:

(1 comment)

....................................................
Commit Message
Line 3: AuthorDate: 2013-09-22 13:16:41 +0200
Line 4: Commit:     Yeela Kaplan <ykap...@redhat.com>
Line 5: CommitDate: 2013-09-24 13:15:19 +0200
Line 6: 
Line 7: iscsiadm: Filter out IPv6 addresses when discovering targets
This patch is not diminishing any capabilities we have, since we currently 
don't support ipV6 targets in vdsm.

When discovering targets on Netapp storage we get a return value that includes 
both IPv4 and IPv6 addresses.

The only thing I've changed is preventing vdsm from failing discovery because 
of getting an unexpected response from iscsiadm (our parser is very hardcoded).

If we decide to support ipV6 addresses for discovery, login and managing the 
connection then it needs to be addressed separately.
Line 8: 
Line 9: Change-Id: I2f6d60e780e9ac50041cdcaa0c45394e392b52f4


-- 
To view, visit http://gerrit.ovirt.org/19431
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2f6d60e780e9ac50041cdcaa0c45394e392b52f4
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykap...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Better Saggi <bettersa...@gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Livnat Peer <lp...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to