Re: [PATCHv5 13/18] Domain network devices can now have a <route> element

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

 



On Tue, Jan 06, 2015 at 01:41:34PM -0500, Laine Stump wrote:
> On 12/30/2014 05:27 AM, Cédric Bosdonnat wrote:
> > Network interfaces devices and host devices with net capabilities can
> > now have IPv4 and/or an IPv6 routes configured.
> > ---
> >  docs/formatdomain.html.in            |  19 ++++-
> >  docs/schemas/domaincommon.rng        |  31 ++++++++
> >  src/conf/domain_conf.c               | 135 ++++++++++++++++++++++++++++++++++-
> >  src/conf/domain_conf.h               |  12 ++++
> >  src/util/virnetdev.c                 |  31 +++++++-
> >  src/util/virnetdev.h                 |   2 +-
> >  src/util/virsocketaddr.h             |   2 +
> >  tests/lxcxml2xmldata/lxc-hostdev.xml |   2 +
> >  tests/lxcxml2xmldata/lxc-idmap.xml   |   2 +
> >  9 files changed, 230 insertions(+), 6 deletions(-)
> >
> > diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
> > index 3f203a5..499879e 100644
> > --- a/docs/formatdomain.html.in
> > +++ b/docs/formatdomain.html.in
> > @@ -4328,14 +4328,18 @@ qemu-kvm -net nic,model=? /dev/null
> >      &lt;interface type='network'&gt;
> >        &lt;source network='default'/&gt;
> >        &lt;target dev='vnet0'/&gt;
> > -      <b>&lt;ip family='ipv4' address='192.168.122.5' prefix='24'/&gt;</b>
> > +      <b>&lt;ip address='192.168.122.5' prefix='24'/&gt;</b>
> > +      <b>&lt;route family='ipv4' address='192.168.122.0' prefix='24' via='192.168.122.1'/&gt;</b>
> > +      <b>&lt;route family='ipv4' via='192.168.122.1'/&gt;</b>
> 
> Sorry I didn't see this before - we already have an XML specification
> for a <route> element in libvirt networks, and had decided that the
> gateway should be called, well, "gateway" instead of "via". It also
> includes a "metric" attribute. I think these two <route> elements should
> share their syntax, RNG, and parse/format functions. Fortunately we
> haven't yet done a release with this new functionality, so such a change
> is still possible.
> 
> Shall I send a patch for this?

Sure, sounds like a reasonable idea to keep things consistent.


Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

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