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