Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: Introduce network.ip package
......................................................................


net: Introduce network.ip package

vdsm.network.ip.address is added
IPv4 and IPv6 classes are moved to it (from models).

Basic unit test has been created, proper coverage should be added in
future patches.

Change-Id: I89ecb8fc5083f2d645887502a86e84ceddadd939
Signed-off-by: Edward Haas <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/55043
Continuous-Integration: Jenkins CI
Reviewed-by: Petr Horáček <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
Reviewed-by: Ondřej Svoboda <[email protected]>
---
M configure.ac
M lib/vdsm/network/Makefile.am
M lib/vdsm/network/configurators/ifcfg.py
A lib/vdsm/network/ip/Makefile.am
A lib/vdsm/network/ip/__init__.py
A lib/vdsm/network/ip/address.py
M lib/vdsm/network/legacy_switch.py
M lib/vdsm/network/models.py
A tests/network/ip_test.py
M vdsm.spec.in
M vdsm_hooks/ovs/ovs_before_network_setup_ip.py
11 files changed, 246 insertions(+), 129 deletions(-)

Approvals:
  Ondřej Svoboda: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Petr Horáček: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Edward Haas: Verified



-- 
To view, visit https://gerrit.ovirt.org/55043
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I89ecb8fc5083f2d645887502a86e84ceddadd939
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: Ondřej Svoboda <[email protected]>
Gerrit-Reviewer: Petr Horáček <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to