Re: [libvirt] [PATCHv2 0/4] support element for interfaces and networks

2012-08-15 Thread Kyle Mestery (kmestery)
On Aug 15, 2012, at 12:47 PM, Laine Stump wrote: > On 08/14/2012 03:15 AM, Laine Stump wrote: >> The purpose of these patches is to define a data type that can >> properly describe an interface's vlan configuration (including >> multiple vlans for trunking), provide XML parser and formatter to >> t

Re: [libvirt] [PATCHv2 0/4] support element for interfaces and networks

2012-08-15 Thread Laine Stump
On 08/14/2012 03:15 AM, Laine Stump wrote: > The purpose of these patches is to define a data type that can > properly describe an interface's vlan configuration (including > multiple vlans for trunking), provide XML parser and formatter to > translate to and from the new data type, add the new el

Re: [libvirt] [PATCHv2 0/4] support element for interfaces and networks

2012-08-14 Thread Dennis Jenkins
On Tue, Aug 14, 2012 at 2:15 AM, Laine Stump wrote: > danpb raised a (valid) objection about the XML format in the first > version of these patches, so I've redone it to be like this: > > > > > > > The downside is that for a single vlan tag, it's more verbose: > > >

[libvirt] [PATCHv2 0/4] support element for interfaces and networks

2012-08-14 Thread Laine Stump
danpb raised a (valid) objection about the XML format in the first version of these patches, so I've redone it to be like this: The downside is that for a single vlan tag, it's more verbose: On the other hand, it's more consistent. The rest of the patches in