From Dan Kenigsberg <[email protected]>: Dan Kenigsberg has submitted this change and it was merged. ( https://gerrit.ovirt.org/80015 )
Change subject: net: Centralize call to pre-config validation ...................................................................... net: Centralize call to pre-config validation Introduce network.validator module, which is responsible to call all pre-config validators. This patch creats a new validator module for the link package and moves the call to these validations (bond naming) to the new network.validator module. New pre-config validations should be called from this new module and existing validation calls should be migrated and called from the new module as well. Change-Id: Icd8fab6168816cd03545f268f8402f06e1fe84d6 Signed-off-by: Edward Haas <[email protected]> --- M lib/vdsm/network/api.py A lib/vdsm/network/link/validator.py M lib/vdsm/network/netswitch/configurator.py M lib/vdsm/network/netswitch/validator.py A lib/vdsm/network/validator.py A tests/network/link_validator_test.py M tests/network/netswitch_test.py 7 files changed, 138 insertions(+), 53 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Edward Haas: Verified -- To view, visit https://gerrit.ovirt.org/80015 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icd8fab6168816cd03545f268f8402f06e1fe84d6 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Edward Haas <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Leon Goldberg <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
