From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: Introduce add/remove sourceroute api
......................................................................


net: Introduce add/remove sourceroute api

sourceroutes have been configured by the existing configurators, using
the sourceroute module to generate the correct configuration details.

There are users of sourceroute that do not really need the
configurators (dhclient_monitor), but had to use it as there was no
other option to add or remove a sourceroute.

This patch introduces the ability to add sourceroutes and remove them
through the sourceroute module, without the need to pass through the
configurators.
This step will allow supporting sourceroute with OVS and follows the
'driver' pattern, where each component has its own drivers.

The add/remove functionality is using the ip.rule and ip.route modules.

Change-Id: I70f1755734e3de1d7cdc26cc1c7bf755efcad1f2
Signed-off-by: Edward Haas <edwa...@redhat.com>
---
M lib/vdsm/network/dhclient_monitor.py
M lib/vdsm/network/sourceroute.py
M tests/Makefile.am
A tests/network/sourceroute_test.py
D tests/network/sourcerouting_test.py
5 files changed, 180 insertions(+), 57 deletions(-)

Approvals:
  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/69914
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I70f1755734e3de1d7cdc26cc1c7bf755efcad1f2
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Jenkins CI
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

Reply via email to