Dan Kenigsberg has submitted this change and it was merged. Change subject: Simplify ip address validation leveraging socket. ......................................................................
Simplify ip address validation leveraging socket. Currently we have a ad-hoc validation of the validity of ip addresses. Since the standard socket module has a built-in validator inside inet_pton, we could leverage it. Change-Id: I6f43556f092d60160860e813b22fcc8660d82a80 Signed-off-by: Antoni S. Puimedon <[email protected]> --- M vdsm/configNetwork.py 1 file changed, 3 insertions(+), 7 deletions(-) Approvals: Antoni Segura Puimedon: Verified Mark Wu: Looks good to me, but someone else must approve Dan Kenigsberg: Igor Lvovsky: Looks good to me, approved Zhou Zheng Sheng: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/11334 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6f43556f092d60160860e813b22fcc8660d82a80 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Livnat Peer <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
