Re: [libvirt] [PATCH]: Add MIGRATE_LIVE definition to ruby-libvirt bindings

2008-07-18 Thread Chris Lalancette
David Lutterkort wrote: ACK .. Committed. (1) Do you need a new ruby-libvirt release for this ? No, it's fine for now; I'm just hard-coding the 1 in oVirt for now. At some future point when you do a new release, though, I'll convert over to this flag. (2) How far back has 'enum

Re: [libvirt] [PATCH] cpu usage in OpenVZ

2008-07-18 Thread Evgeniy Sokolov
On Thu, Jul 17, 2008 at 03:04:46PM +0400, Evgeniy Sokolov wrote: OpenVZ calculate statistics and allow to get them . Added function for getting cpu usage of container. Modular some minor comments, ACK for this patch. +if (!openvzIsActiveVM(vm)) { +info-cpuTime = 0; +} else {

Re: [libvirt] [PATCH] repeat lookup by name in LookupByID

2008-07-18 Thread Jim Meyering
Daniel P. Berrange [EMAIL PROTECTED] wrote: On Thu, Jul 17, 2008 at 01:20:59PM +0400, Evgeniy Sokolov wrote: ... Index: virsh.c === RCS file: /data/cvs/libvirt/src/virsh.c,v retrieving revision 1.155 diff -u -p -r1.155 virsh.c

[libvirt] virStorageVolCreateXML vs virStorageVolCloneXML

2008-07-18 Thread Stefan de Konink
Hi, There is currently no implementation in the api to clone snapshots or images. I wonder if we could add an XML node to specify a backed device. Or add a new function that allows to clone. Like the create is not supported by all pools, cloning should also be based on best effort, falling back

[libvirt] Libvirtd default.xml (in qemu/networks/autostart) breaks XEN

2008-07-18 Thread Stefan de Konink
The desire to automatically install the autostarted network configuration of libvirt broke my (and some other users on xen-users) setup. I suggest to *remove* this network configuration as default and *not* put it into xenstore as a stateful config. *It does not work by default* Worse is that it

[libvirt] [PATCH] enable format-safety checks for virDomainReportError

2008-07-18 Thread Jim Meyering
This enables format-safety checks for virDomainReportError, so that if you try to print e.g., an int via %s, gcc will detect it. The Makefile.maint check ensures that all string arguments to virDomainReportError are marked for translation with _(...). From

Re: [libvirt] [PATCH] enable format-safety checks for virDomainReportError

2008-07-18 Thread Daniel Veillard
On Fri, Jul 18, 2008 at 12:49:01PM +0200, Jim Meyering wrote: This enables format-safety checks for virDomainReportError, so that if you try to print e.g., an int via %s, gcc will detect it. The Makefile.maint check ensures that all string arguments to virDomainReportError are marked for

Re: [libvirt] [PATCH] enable format-safety checks for virDomainReportError

2008-07-18 Thread Jim Meyering
Daniel Veillard [EMAIL PROTECTED] wrote: On Fri, Jul 18, 2008 at 12:49:01PM +0200, Jim Meyering wrote: This enables format-safety checks for virDomainReportError, so that if you try to print e.g., an int via %s, gcc will detect it. The Makefile.maint check ensures that all string arguments

[libvirt] [PATCH]: Add floppy support to xm_internal

2008-07-18 Thread Chris Lalancette
Yes, you read the subject right; add floppy support to xm internal. Let's just say I didn't do this by choice. In any case, there was a cryptic comment in xenXMParseXMLDisks() that said: /* Xend (all versions) put the floppy device config * under the hvm (image (os)) block */ What this

Re: [libvirt] [PATCH]: Add floppy support to xm_internal

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 02:05:06PM +0200, Chris Lalancette wrote: Yes, you read the subject right; add floppy support to xm internal. Let's just say I didn't do this by choice. In any case, there was a cryptic comment in xenXMParseXMLDisks() that said: /* Xend (all versions) put the

Re: [libvirt] Libvirtd default.xml (in qemu/networks/autostart) breaks XEN

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 12:33:22PM +0200, Stefan de Konink wrote: The desire to automatically install the autostarted network configuration of libvirt broke my (and some other users on xen-users) setup. I suggest to *remove* this network configuration as default and *not* put it into xenstore

Re: [libvirt] virStorageVolCreateXML vs virStorageVolCloneXML

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 12:27:49PM +0200, Stefan de Konink wrote: Hi, There is currently no implementation in the api to clone snapshots or images. I wonder if we could add an XML node to specify a backed device. Or add a new function that allows to clone. I think this wants to be done with

Re: [libvirt] attach device on a defined (non-active) vm

2008-07-18 Thread Daniel P. Berrange
On Thu, Jul 17, 2008 at 03:49:01AM +0200, Stefan de Konink wrote: It seems that the attach device function really 'attaches' a (live) device to a non-active domain. Is this by design? Otherwise I would not have a clue how to interpreted this message: libvir: Xen Daemon error : POST

Re: [libvirt] [PATCH]: Add floppy support to xm_internal

2008-07-18 Thread Chris Lalancette
Daniel P. Berrange wrote: Sorry, this patch is no use - there is a stack of patches pending review which re-write the XM and XenD driver's XML handling from scratch to use the new generic domain XML APIs. The floppy stuff will have to stack on top of that: That's OK; the same general idea

[libvirt] Java bindings

2008-07-18 Thread Alejandro Berna Juan
Hi all, I'm Alejandro Berna from i2CAT (a non-profit foundation in Barcelona, Spain, www.i2cat.net). I'm collaborating in a Europena project called Federica ( www.fp7-*federica*.eu ). One of the branch of this project is to permit virtualization of different hosts in the Federica test-bed. We are

Re: [libvirt] Java bindings

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 02:55:39PM +0200, Alejandro Berna Juan wrote: Hi all, I'm Alejandro Berna from i2CAT (a non-profit foundation in Barcelona, Spain, www.i2cat.net). I'm collaborating in a Europena project called Federica ( www.fp7-*federica*.eu ). One of the branch of this project is to

[libvirt] [PATCH] parse container id in separate function

2008-07-18 Thread Evgeniy Sokolov
There were several places with parsing container id from code. Separate function is used now. Index: src/openvz_conf.c === RCS file: /data/cvs/libvirt/src/openvz_conf.c,v retrieving revision 1.29 diff -u -p -r1.29 openvz_conf.c ---

Re: [libvirt] Java bindings

2008-07-18 Thread Daniel Veillard
On Fri, Jul 18, 2008 at 02:00:53PM +0100, Daniel P. Berrange wrote: On Fri, Jul 18, 2008 at 02:55:39PM +0200, Alejandro Berna Juan wrote: Hi all, I'm Alejandro Berna from i2CAT (a non-profit foundation in Barcelona, Spain, www.i2cat.net). I'm collaborating in a Europena project called

Re: [libvirt] RFC? finding potential storage pool resources

2008-07-18 Thread David Lively
On Thu, 2008-07-17 at 22:42 +0100, Daniel P. Berrange wrote: You're not missing anything - this is a TODO item. When I wrote the original storage APIs, I had a prototype http://www.redhat.com/archives/libvir-list/2008-February/msg00107.html

[libvirt] [PATCH] fix lookup

2008-07-18 Thread Evgeniy Sokolov
I am often catching error related with that stoped VM has ID = -1. 1. If I create/define VM with existing ID of stoped VM. Creation failed, but error message is wrong. Detection is fixed. 2. Dominfo command may show info about some another VM. And report error that can't get autostart state.

[libvirt] Release of libvirt-java-0.2.0

2008-07-18 Thread Daniel Veillard
Okay I made a new release with the API renaming changes we discussed previously. It is available at the usual place: ftp://libvirt.org/libvirt/java/ I also built it for Fedora-9, it should be available for testing there soon too. There is still a few issues, for example I get an out of bound

Re: [libvirt] PATCH: 3/14: Generic domain XML parser/formatter

2008-07-18 Thread Jim Meyering
... +static virDomainChrDefPtr +virDomainChrDefParseXML(virConnectPtr conn, +xmlNodePtr node) { +xmlNodePtr cur; +char *type = NULL; +char *bindHost = NULL; +char *bindService = NULL; +char *connectHost = NULL; +char *connectService = NULL;

Re: [libvirt] [PATCH]: Add floppy support to xm_internal

2008-07-18 Thread John Levon
On Fri, Jul 18, 2008 at 02:05:06PM +0200, Chris Lalancette wrote: What this actually means is that we shouldn't parse the floppy stuff to put it in the disks = section of the /etc/xen configuration file, since it doesn't have meaning there. Instead, floppy disks go at the top-level of a Xen

Re: [libvirt] [PATCH]: Add floppy support to xm_internal

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 05:31:27PM +0100, John Levon wrote: On Fri, Jul 18, 2008 at 02:05:06PM +0200, Chris Lalancette wrote: What this actually means is that we shouldn't parse the floppy stuff to put it in the disks = section of the /etc/xen configuration file, since it doesn't

Re: [libvirt] Libvirtd default.xml (in qemu/networks/autostart) breaks XEN

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 01:13:30PM +0100, Daniel P. Berrange wrote: On Fri, Jul 18, 2008 at 12:33:22PM +0200, Stefan de Konink wrote: The desire to automatically install the autostarted network configuration of libvirt broke my (and some other users on xen-users) setup. I suggest to

Re: [libvirt] [PATCH] parse container id in separate function

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 06:02:00PM +0400, Evgeniy Sokolov wrote: There were several places with parsing container id from code. Separate function is used now. Index: src/openvz_conf.c === RCS file:

Re: [libvirt] [PATCH] fix lookup

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 07:01:33PM +0400, Evgeniy Sokolov wrote: I am often catching error related with that stoped VM has ID = -1. 1. If I create/define VM with existing ID of stoped VM. Creation failed, but error message is wrong. Detection is fixed. 2. Dominfo command may show info about

Re: [libvirt] virStorageVolCreateXML vs virStorageVolCloneXML

2008-07-18 Thread Stefan de Konink
Daniel P. Berrange schreef: On Fri, Jul 18, 2008 at 12:27:49PM +0200, Stefan de Konink wrote: Hi, There is currently no implementation in the api to clone snapshots or images. I wonder if we could add an XML node to specify a backed device. Or add a new function that allows to clone. I think

Re: [libvirt] Libvirtd default.xml (in qemu/networks/autostart) breaks XEN

2008-07-18 Thread Daniel P. Berrange
On Fri, Jul 18, 2008 at 08:15:20PM +0200, Stefan de Konink wrote: Daniel P. Berrange schreef: On Fri, Jul 18, 2008 at 12:33:22PM +0200, Stefan de Konink wrote: The desire to automatically install the autostarted network configuration of libvirt broke my (and some other users on xen-users)

[libvirt] [PATCH] configure.in: make --with-xen-distdir work for 64bit xen too

2008-07-18 Thread David Lively
Currently running autogen.sh with --with-xen-distdir=something fails to find libxenstore if there's only a 64-bit version, and subsequently fails to enable xen support (i.e., ends up with WITH_XEN=0). This one-line patch fixes that by telling it to search both lib and lib64. I guess it would be

[libvirt] ruby-libvirt on debian based systems

2008-07-18 Thread Tom Verhaeghe
Hello I don't know if ruby-libvirt can work in a debian-based environment. I installed kvm, libvirt, RoR, etc on Ubuntu and it all works fine. I pasted the output when installing the gem below. I think the problem could be that libvirt is called differently in Ubuntu or stored on a different