[libvirt] libvirt will create two guest with the same name, but with different UUIDs

2009-04-29 Thread Kaitlin Rupert
Hi, libvirt allows you to create two guests with same name, but with different UUIDs. This causes some problems in the reporting though. virsh will report that a guest is running (when it isn't). This also causes some confusion for the dominfo call - virsh will only return info for one of th

[libvirt] VM image running on 2 Intels

2009-04-29 Thread Zvi Dubitzky
I have 2 types of hosts as below ( each is 64 bits processor . number of processor is not equal . First has 1G memory . the second 3 G memor): IBM blade HS21 with Intel Xeon L5320 (1,86Ghz) . 8 processors (blade) IBM desktop with Intel(R) Core(TM)2 Duo CPU E6550 @ 2.33GHz I have generated

Re: [libvirt] PATCH: Add a new error code for invalid operations

2009-04-29 Thread Daniel Veillard
On Wed, Apr 29, 2009 at 03:32:02PM +0100, Daniel P. Berrange wrote: > A number of operations are only valid when the corresponding object is in > a particular state. eg, you can only suspend a domain, if it is currently > running. We don't have a very consistent way of reporting this problem > ac

Re: [libvirt] [RFC] Power Hypervisor Libvirt support

2009-04-29 Thread Daniel Veillard
On Thu, Apr 23, 2009 at 06:44:13PM -0300, Eduardo Otubo wrote: > I very am glad with all the feedback I got. The following attachtment is > the result of the work using all the suggestions the list made. I also > have a few comments over this new RFC. Here is a little Changelog: > [...] > --- a/in

[libvirt] PATCH: Add a new error code for invalid operations

2009-04-29 Thread Daniel P. Berrange
A number of operations are only valid when the corresponding object is in a particular state. eg, you can only suspend a domain, if it is currently running. We don't have a very consistent way of reporting this problem across drivers, so this patch introduces a new error code for this situation c

Re: [libvirt] [PATCH] Remove unused file volume format 'dir'

2009-04-29 Thread Cole Robinson
Daniel P. Berrange wrote: > On Tue, Apr 28, 2009 at 12:33:24PM -0400, Cole Robinson wrote: >> It isn't documented, detected, and barely has any code accomodating it. >> Appears to be left over from an earlier iteration of the storage APIs. >> >> commit 2aec7c2f843d30e5b6882b8a335494a2de7bc156 >> Au

Re: [libvirt] [PATCH] Refresh QEMU driver caps in getCapabilities

2009-04-29 Thread Cole Robinson
Daniel P. Berrange wrote: > On Tue, Apr 28, 2009 at 12:25:41PM -0400, Cole Robinson wrote: >> Hi all, >> >> The attached patch fixes QEMU getCapabilities to refresh caps before >> returning them to the user. Currently the capabilities are only >> refreshed once (at daemon startup), which means libv

Re: [libvirt] [PATCH] Fix a comment typo in storage_backend_fs.c

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 12:33:44PM -0400, Cole Robinson wrote: > This comment hid from my attempt to 'grep FILE_BOCHS', which is the > actual name of the constant. ACK > commit 62861e902fc3c272e46cdf9948079f12564ca119 > Author: Cole Robinson > Date: Wed Apr 22 14:51:54 2009 -0400 > > Upda

Re: [libvirt] [PATCH] Remove unused file volume format 'dir'

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 12:33:24PM -0400, Cole Robinson wrote: > It isn't documented, detected, and barely has any code accomodating it. > Appears to be left over from an earlier iteration of the storage APIs. > > commit 2aec7c2f843d30e5b6882b8a335494a2de7bc156 > Author: Cole Robinson > Date: We

Re: [libvirt] [PATCH] Refresh QEMU driver caps in getCapabilities

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 12:25:41PM -0400, Cole Robinson wrote: > Hi all, > > The attached patch fixes QEMU getCapabilities to refresh caps before > returning them to the user. Currently the capabilities are only > refreshed once (at daemon startup), which means libvirtd needs to be > restarted to

Re: [libvirt] [PATCH 7/7] Don't throw away QEMU startup errors when migrating.

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:57AM -0400, Cole Robinson wrote: > --- > src/qemu_driver.c | 10 +++--- > 1 files changed, 7 insertions(+), 3 deletions(-) > > diff --git a/src/qemu_driver.c b/src/qemu_driver.c > index 04df8ae..b293f72 100644 > --- a/src/qemu_driver.c > +++ b/src/qemu_driver.c

Re: [libvirt] [PATCH 6/7] Check that QEMU is still alive while reading startup output.

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:56AM -0400, Cole Robinson wrote: > By checking the pid every retry period, we can quickly determine if > the process crashed at startup, rather than make the user wait for > the entire timeout (3 seconds). ACK, this looks good. > --- > src/qemu_driver.c | 33 +

Re: [libvirt] [PATCH 5/7] Add pidfile argument to __virExec

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:55AM -0400, Cole Robinson wrote: > virExec will write out the pid of the daemonized process only. Use this > in the QEMU driver, rather than QEMU's pidfile, so we can errors we might > miss if the emulator bails early. > diff --git a/src/qemu_driver.c b/src/qemu_drive

Re: [libvirt] [PATCH 4/7] Report qemu log data if we fail to daemonize the process.

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:54AM -0400, Cole Robinson wrote: > This ensures the user will actually see 'hook' function error output. > --- > src/qemu_driver.c | 13 +++-- > 1 files changed, 11 insertions(+), 2 deletions(-) > > diff --git a/src/qemu_driver.c b/src/qemu_driver.c > index

Re: [libvirt] [PATCH 3/7] Add helper function virExecDaemonize

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:53AM -0400, Cole Robinson wrote: > Wraps __virExec with the VIR_EXEC_DAEMON flag. Waits on the intermediate > process to ensure we don't end up with any zombies, and differentiates between > original process errors and intermediate process errors. > +int virExecDaemoni

Re: [libvirt] [PATCH 2/7] virExec: Delay daemonizing as long as possible.

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:52AM -0400, Cole Robinson wrote: > This way the caller can catch more errors (e.g. from a hook callback) from > the intermediate process. > --- > src/util.c | 56 > 1 files changed, 28 insertions(+), 28 deleti

Re: [libvirt] [PATCH 1/7] Add documentation for __virExec and virExec

2009-04-29 Thread Daniel P. Berrange
On Tue, Apr 28, 2009 at 11:31:51AM -0400, Cole Robinson wrote: > --- > src/util.c | 27 +++ > 1 files changed, 27 insertions(+), 0 deletions(-) > > diff --git a/src/util.c b/src/util.c > index b7e0362..87e215d 100644 > --- a/src/util.c > +++ b/src/util.c > @@ -266,6 +266

[libvirt] [PATCH 0/1] Added Networking API to change/create/etc. hostonly/internal network in VirtualBox

2009-04-29 Thread Pritesh Kothari
Hi All: [PATCH 0/1]: Contains sample xml file showing features supported so far. [PATCH 1/1]: Contains the patch for adding Networking API to change/create/etc. hostonly/internal network in VirtualBox Regards, Pritesh vboxnet0 786f6276-656e-4074-8000-0a002700 -- Lib

[libvirt] [PATCH 1/1] Added Networking API to change/create/etc. hostonly/internal network in VirtualBox

2009-04-29 Thread Pritesh Kothari
Hi All: [PATCH 0/1]: Contains sample xml file showing features supported so far. [PATCH 1/1]: Contains the patch for adding Networking API to change/create/etc. hostonly/internal network in VirtualBox Regards, Pritesh diff --git a/src/network_conf.c b/src/network_conf.c index b4da3fb..b4e0b39 10

[libvirt] [PATCH 1/2] Support for 'hostonly' and 'internal' network in VirtualBox

2009-04-29 Thread Pritesh Kothari
Hi All, I have added the support for hostonly and internal network as suggested on irc and list and am attaching the patch for same. PATCH 1/2: contains changes in the xml parsing/formatting on libvirt side namely adding a mode attribute to tag in domain/interface xml PATCH 2/2: contains chang

[libvirt] [PATCH 2/2] Support for 'hostonly' and 'internal' network in VirtualBox

2009-04-29 Thread Pritesh Kothari
Hi All, I have added the support for hostonly and internal network as suggested on irc and list and am attaching the patch for same. PATCH 1/2: contains changes in the xml parsing/formatting on libvirt side namely adding a mode attribute to tag in domain/interface xml PATCH 2/2: contains chang

[libvirt] [PATCH 1/1] Making path detection more robust in VirtualBox

2009-04-29 Thread Pritesh Kothari
Hi All, PATCH 1/1: contains changes in the glue code for making path detection more robust in VirtualBox Regards, Pritesh commit 0f31b4cce710ab6efea352174049e305b29a7a2f Author: Pritesh Kothari Date: Wed Apr 29 10:52:37 2009 +0200 made the path detection more robust in the glue library

[libvirt] [PATCH 1/1] Changes in version detection and moving a global variable in the global struct

2009-04-29 Thread Pritesh Kothari
Hi All, PATCH 1/1: contains changes in version detection and moving a global variable in the global struct, so that no more global vairables exists. Regards, Pritesh commit 4a76967975b311151454af60d03c1e84ad0f6a11 Author: Pritesh Kothari Date: Wed Apr 29 11:16:59 2009 +0200 making versio