On 05/16/2016 12:21 PM, Laine Stump wrote:
This is identical to type='bridge', but without the
"connect to a
bridge" part, so it can be handled by using the same functions (and
often even the same cases in switch statements), after renaming
virLXCProcessSetupInterfaceBridged() to virLXCProcessInterfaceTap()
and enhancing it to skip bridge-related items when brname == NULL.
To be truly useful, we need to support setting the ip address on the
host side veth as well as guest side veth (already supported for
type='bridge'), as well as setting the peer address for both.
The <script> element isn't supported in this patch because I have no
need for it. I'd rather add it after determining it's needed rather
than adding it for no reason and than being required to support it
forever.
I suggest explicitly rejecting the <script> option then, since at least the
formatdomain docs make it sound like it's the only valid type=ethernet config.
virLXCProcessSetupInterfaceTap seems like the only shared place for it though
Also, this is
https://bugzilla.redhat.com/show_bug.cgi?id=1325687
Patch looks fine otherwise, ACK if it's simple to reject the script bit, up to you
- Cole