[Libvir] [RFC] Linux-VServer support

[Daniel Veillard]

On Wed, Oct 31, 2007 at 04:09:54AM +0000, Daniel P. Berrange wrote:
> On Tue, Oct 30, 2007 at 04:28:59PM +0100, Daniel Hokka Zakrisson wrote:
> > This is an initial stab at adding Linux-VServer support to libvirt. 
> > There are still a couple of things missing, like scheduler support in 
> > the XML parsing, and proper network support.
> 
> Great to see interest in adding more drivers ! I've not had time to look

  Definitely, welcome aboard ! I'm just a bit worried that it's Yet
Another Daniel in the project, confusion guaranteed :-)

> at your patch in great detail yet, but I'll give some more detailed
> feedback in the next day or so. My first comment though - why the 
> <xid>123</xid> field in the XML - the unique integer ID for a domain
> should really be in the 'id' attribute  <domain id='123'>. There are a
> couple of other small XML format consistency issues like that to check
> up on.

  I looked at the code, that seems clean but I have a concern about the
overall XML format. Could you paste a couple of examples. Also I think
Linux-VServer and OpenVZ kind of configuration may end up with the same
kind of limitations or differences, so I would like to try to harmonize
both format when possible.

> > I've got a few questions though. virsh's schedinfo hardcodes the 
> > available options, should I be adding new ones there? Would better 
> > introspection from getSchedulerType make this a non-issue? How do the 
> > network domains interact and associate with the regular domains?
> 
> Yes, the virsh schedinfo command is broken in this respect. Rather
> than hardcoding params, it should simply allow
> 
>    virsh schedinfo --set foo=bar --set wizz=123 
> 
> To determine the data types for each param, it can simply query the existing
> values with getSchedularInfo and then update them accordingly.

  the scheduling side of the API is probably the part where it's the
harder to keep a coherent access between hypervisors, that's the reason
why Rich designed a very flexible mechanism but drivers should use some
introspection by looking at parameters names to process them, and
provide good error reporting.
  virsh command can certainly be improved as Dan suggested, yes.

  thanks for the patch, but let's discuss and fix the XML format before
trying to finish and push that first version :-)

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard at redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/