Re: [PATCH 1/3] network: new network forward mode 'open'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12.08.2016 04:41, Laine Stump wrote:
> The new forward mode 'open' is just like mode='route', except that no
> firewall rules are added to assure that any traffic does or doesn't
> pass. It is assumed that either they aren't necessary, or they will be
> setup outside the scope of libvirt.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=846810
> ---
>  docs/formatnetwork.html.in                         | 22 ++++++++++++
>  docs/schemas/network.rng                           |  1 +
>  src/conf/network_conf.c                            | 25 +++++++++++--
>  src/conf/network_conf.h                            |  1 +
>  src/network/bridge_driver.c                        | 41 +++++++++++++++-------
>  tests/networkxml2confdata/open-network.conf        | 11 ++++++
>  tests/networkxml2confdata/open-network.xml         |  9 +++++
>  tests/networkxml2conftest.c                        |  1 +
>  .../open-network-with-forward-dev.xml              |  9 +++++
>  tests/networkxml2xmlin/open-network.xml            |  9 +++++
>  tests/networkxml2xmlout/open-network.xml           |  9 +++++
>  tests/networkxml2xmltest.c                         |  2 ++
>  12 files changed, 125 insertions(+), 15 deletions(-)
>  create mode 100644 tests/networkxml2confdata/open-network.conf
>  create mode 100644 tests/networkxml2confdata/open-network.xml
>  create mode 100644 tests/networkxml2xmlin/open-network-with-forward-dev.xml
>  create mode 100644 tests/networkxml2xmlin/open-network.xml
>  create mode 100644 tests/networkxml2xmlout/open-network.xml

Okay, looks like we have an agreement on the XML then. The code is
solid. ACK

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]