From Dan Kenigsberg <dan...@redhat.com>: Dan Kenigsberg has submitted this change and it was merged. ( https://gerrit.ovirt.org/79951 )
Change subject: net: Introduce ipv4 config validator ...................................................................... net: Introduce ipv4 config validator ipv4 configuration validity is shared between ovs and legacy switch and as such should be validated prior to ovs/legacy split. As there were some inconsistencies in the raised error codes of invalid ip configurations, all related errors now raise ERR_BAD_ADDR. Change-Id: Ibd2e3116249a30c917a945117022934ea87e0d29 Signed-off-by: Leon Goldberg <lgold...@redhat.com> --- M lib/vdsm/network/ip/address/__init__.py M lib/vdsm/network/ip/validator.py M tests/network/config_network_test.py M tests/network/ip_validator_test.py 4 files changed, 72 insertions(+), 33 deletions(-) Approvals: Jenkins CI: Passed CI tests Leon Goldberg: Verified Edward Haas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/79951 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd2e3116249a30c917a945117022934ea87e0d29 Gerrit-PatchSet: 31 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Leon Goldberg <lgold...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Edward Haas <edwa...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Leon Goldberg <lgold...@redhat.com> Gerrit-Reviewer: Petr Horáček <phora...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org