[PATCH v6 0/5] Add TXT record and hosts support for DNS service and dnsmasq command-line regression testing

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

 



Hi,
this is the 6th (and hopefully the last) version of my patch to
introduce the TXT record and DNS hosts support for the DNS service
on the virtual network. This can be defined using the txt
subelement in the dns element of the network XML description.
The definition of TXT record names containing spaces is rejected
with the error message that TXT record names in DNS doesn't
support spaces.

Also, regression testing for the dnsmasq command line has been
added to test whether the dnsmasq command-line is correct or not.

The new XML definition syntax is:

          <dns>
            <txt name="example" value="example value" />
            <host ip='192.168.122.1'>
              <hostname>gateway</hostname>
              <hostname>host</hostname>
            </host>
          </dns>

Where multiple host elements can be defined to put the aliases.
The <dns> element have to be present on the <ip> level, i.e.
one level upper than it was in version 2 of the patch since
the definition affects all the IP addresses on the network.

The patch series has been tested for the configuration and it
was working fine and also RelaxNG schema with the tests have
been both altered to add test cases to test those patches.

This is the new version for latest libvirt codebase so please
review the patchset.

Some fixes and code optimizations were done by Laine Stump,
so I'm adding him into the SOB clause as well ;-)

Please review.

Thanks,
Michal

Signed-off-by: Michal Novotny <minovotn@xxxxxxxxxx>
Signed-off-by: Laine Stump <lstump@xxxxxxxxxx>

Michal Novotny (5):
  Add TXT record support for virtual DNS service
  Network: Add regression tests for the command-line arguments
  Network: Move dnsmasqContext creation to
    networkSaveDnsmasqHostsfile()
  Network: Add additional hosts internal infrastructure
  Network: Add support for DNS hosts definition to the network XML

 docs/formatnetwork.html.in                         |   29 ++-
 docs/schemas/network.rng                           |   28 ++
 src/Makefile.am                                    |    7 +-
 src/conf/network_conf.c                            |  206 ++++++++++++++
 src/conf/network_conf.h                            |   26 ++
 src/libvirt_network.syms                           |    6 +
 src/libvirt_private.syms                           |    1 +
 src/network/bridge_driver.c                        |  123 ++++++---
 src/network/bridge_driver.h                        |    3 +
 src/util/dnsmasq.c                                 |  285 ++++++++++++++++++--
 src/util/dnsmasq.h                                 |   22 ++-
 tests/Makefile.am                                  |   10 +
 tests/networkxml2argvdata/isolated-network.argv    |    1 +
 tests/networkxml2argvdata/isolated-network.xml     |   11 +
 .../networkxml2argvdata/nat-network-dns-hosts.argv |    1 +
 .../networkxml2argvdata/nat-network-dns-hosts.xml  |   14 +
 .../nat-network-dns-txt-record.argv                |    1 +
 .../nat-network-dns-txt-record.xml                 |   24 ++
 tests/networkxml2argvdata/nat-network.argv         |    1 +
 tests/networkxml2argvdata/nat-network.xml          |   21 ++
 tests/networkxml2argvdata/netboot-network.argv     |    1 +
 tests/networkxml2argvdata/netboot-network.xml      |   14 +
 .../networkxml2argvdata/netboot-proxy-network.argv |    1 +
 .../networkxml2argvdata/netboot-proxy-network.xml  |   13 +
 tests/networkxml2argvdata/routed-network.argv      |    1 +
 tests/networkxml2argvdata/routed-network.xml       |    9 +
 tests/networkxml2argvtest.c                        |  109 ++++++++
 tests/networkxml2xmlin/nat-network-dns-hosts.xml   |   14 +
 .../nat-network-dns-txt-record.xml                 |   24 ++
 tests/networkxml2xmlout/nat-network-dns-hosts.xml  |   14 +
 .../nat-network-dns-txt-record.xml                 |   24 ++
 tests/networkxml2xmltest.c                         |    2 +
 32 files changed, 990 insertions(+), 56 deletions(-)
 create mode 100644 src/libvirt_network.syms
 create mode 100644 tests/networkxml2argvdata/isolated-network.argv
 create mode 100644 tests/networkxml2argvdata/isolated-network.xml
 create mode 100644 tests/networkxml2argvdata/nat-network-dns-hosts.argv
 create mode 100644 tests/networkxml2argvdata/nat-network-dns-hosts.xml
 create mode 100644 tests/networkxml2argvdata/nat-network-dns-txt-record.argv
 create mode 100644 tests/networkxml2argvdata/nat-network-dns-txt-record.xml
 create mode 100644 tests/networkxml2argvdata/nat-network.argv
 create mode 100644 tests/networkxml2argvdata/nat-network.xml
 create mode 100644 tests/networkxml2argvdata/netboot-network.argv
 create mode 100644 tests/networkxml2argvdata/netboot-network.xml
 create mode 100644 tests/networkxml2argvdata/netboot-proxy-network.argv
 create mode 100644 tests/networkxml2argvdata/netboot-proxy-network.xml
 create mode 100644 tests/networkxml2argvdata/routed-network.argv
 create mode 100644 tests/networkxml2argvdata/routed-network.xml
 create mode 100644 tests/networkxml2argvtest.c
 create mode 100644 tests/networkxml2xmlin/nat-network-dns-hosts.xml
 create mode 100644 tests/networkxml2xmlin/nat-network-dns-txt-record.xml
 create mode 100644 tests/networkxml2xmlout/nat-network-dns-hosts.xml
 create mode 100644 tests/networkxml2xmlout/nat-network-dns-txt-record.xml

-- 
1.7.3.2

--
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]