On Fri, Jun 19, 2009 at 10:57:59AM -0400, Laine Stump wrote:
This is a resubmission of the virsh commands to expose virInterface*
functionality. I've made the following changes from the original:
1) command names changed from if-* to iface-*
2) "if-create" is now "iface-start"
3) iface-edit implementation is manually included directly in virsh.c
rather than being generated with a makefile rule.
4) iface-list now has options --inactive and --all, similar to net-list.
(4) depends on new functions contained in the patch I submitted yesterday.
All of this builds and appears to be correct, but is non-functional
until either the netcf backend or test driver backend is
committed. The netcf backend is waiting on something in netcf to
provide the functionality required for
virConnectListDefinedInterfaces(), and the test driver is waiting on
XML parsing/formatting functions (and needs a small amount of rework
to support inactive interfaces).
Looks fine to me, though IMHO we really need to clear up the XML
format description before netcf can really be plugged. I have some
versions of interface_conf.[ch] too but with an intermediate XML format
so not too useful right now,
ACK,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit
http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine
http://rpmfind.net/
http://veillard.com/ | virtualization library
http://libvirt.org/