Re: [libvirt] Switch from CVS to GIT is done

2009-07-07 Thread Jim Meyering
Daniel P. Berrange wrote: > On Mon, Jul 06, 2009 at 02:42:59PM +0200, Daniel Veillard wrote: ... >> The new repo is at: >> http://libvirt.org/git/?p=libvirt.git;a=summary > > FYI, since the main repository is now using GIT, there is little benefit > in having the other related repositories un

Re: [libvirt] Switch from CVS to GIT is done

2009-07-07 Thread Daniel Veillard
On Tue, Jul 07, 2009 at 09:29:17AM +0200, Jim Meyering wrote: > Daniel P. Berrange wrote: > > On Mon, Jul 06, 2009 at 02:42:59PM +0200, Daniel Veillard wrote: > ... > >> The new repo is at: > >> http://libvirt.org/git/?p=libvirt.git;a=summary > > > > FYI, since the main repository is now usin

[libvirt] make distcheck failure

2009-07-07 Thread Guido Günther
Hi, in current git make distcheck currently fails on me with: make[5]: Entering directory `/var/scratch/debian/libvirt/cvs/libvirt/libvirt-0.6.5/_build/gnulib/lib' /bin/sh ../../../mylibtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I../../../gnulib/lib -I../.. -I../../intl -g -O2 -M

Re: [libvirt] make distcheck failure

2009-07-07 Thread Daniel Veillard
On Tue, Jul 07, 2009 at 11:45:52AM +0200, Guido Günther wrote: > Hi, > in current git make distcheck currently fails on me with: > > make[5]: Entering directory > `/var/scratch/debian/libvirt/cvs/libvirt/libvirt-0.6.5/_build/gnulib/lib' > /bin/sh ../../../mylibtool --tag=CC --mode=compile gcc -

[libvirt] [PATCH] allow to set path to xen userspace tools

2009-07-07 Thread Guido Günther
Hi, attached patch makes the path to the xen userspace tools configurable. Debian keeps this under /usr/lib/xen-default/ instead of /usr/lib/xen/. We don't have the amd64 libs in /usr/lib64/xen either so we can use: ./configure --with-xen-tools=/usr/lib/xen-defaults --with-xen-tools64=/usr/lib/xe

Re: [libvirt] [PATCH] allow to set path to xen userspace tools

2009-07-07 Thread Daniel Veillard
On Tue, Jul 07, 2009 at 01:16:02PM +0200, Guido Günther wrote: > Hi, > attached patch makes the path to the xen userspace tools configurable. > Debian keeps this under /usr/lib/xen-default/ instead of /usr/lib/xen/. > We don't have the amd64 libs in /usr/lib64/xen either so we can use: > ./configu

[libvirt] Can't boot guest after adding an IDE storage

2009-07-07 Thread M. Mohan Kumar
Hi, We installed Fedora 11 guest in an emulated SCSI disk using virt-install. After the installation the guest was shutdown and additional IDE virtual storage was added to the guest. When we try to boot the guest from virt-manager, it tries to boot from the newly added IDE storage (instead of ins

[libvirt] [PATCH 2/2] RPM spec file updated with glusterfs dependency

2009-07-07 Thread Harshavardhana
Add new dependency for glusterfs rpm. --- libvirt.spec.in |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 1e8a8ef..ccef3a5 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -86,6 +86,8 @@ BuildRequires: util-linux # For showmou

[libvirt] [PATCH 1/2] Added new filesystem glusterfs

2009-07-07 Thread Harshavardhana
Added new enum for glusterfs, a new netfs based filesystem --- src/storage_conf.c |2 +- src/storage_conf.h |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/storage_conf.c b/src/storage_conf.c index 5f724dc..16ed906 100644 --- a/src/storage_conf.c +++ b/src/storage

[libvirt] [PATCH 1/2] Added new filesystem glusterfs

2009-07-07 Thread Harshavardhana
Added new enum for glusterfs, another netfs based filesystem. --- src/storage_conf.c |2 +- src/storage_conf.h |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/storage_conf.c b/src/storage_conf.c index 5f724dc..16ed906 100644 --- a/src/storage_conf.c +++ b/src/st

[libvirt] Re: Can't boot guest after adding an IDE storage

2009-07-07 Thread Daniel Veillard
On Tue, Jul 07, 2009 at 05:58:37PM +0530, M. Mohan Kumar wrote: > Hi, > > We installed Fedora 11 guest in an emulated SCSI disk using virt-install. > After the installation the guest was shutdown and additional IDE virtual > storage was added to the guest. > > When we try to boot the guest from v

[libvirt] Re: kernel summit topic - 'containers end-game'

2009-07-07 Thread Oren Laadan
Serge E. Hallyn wrote: > Quoting Oren Laadan (or...@cs.columbia.edu): >> >> Serge E. Hallyn wrote: >>> A topic on ksummit agenda is 'containers end-game and how do we >>> get there'. >>> >>> So for starters, looking just at application (and system) containers, what >>> do >>> the libvirt and lib

Re: [libvirt] Switch from CVS to GIT is done

2009-07-07 Thread Atsushi SAKAI
Hi, Daniel I am happy to hear this, since I am primaly tracking libvirt via git. >From seeing http://libvirt.org/git/, only 4 packages are controlled under git. (libvirt, libvirt-glib, libvirt-perl, libvirt-tck) Will other libvirt related tools goes to the directory in future? Thanks Atsushi SAK

Re: [libvirt] Switch from CVS to GIT is done

2009-07-07 Thread Daniel Veillard
On Tue, Jul 07, 2009 at 10:08:12PM +0900, Atsushi SAKAI wrote: > Hi, Daniel > > I am happy to hear this, since I am primaly tracking libvirt via git. > > >From seeing http://libvirt.org/git/, > only 4 packages are controlled under git. > (libvirt, libvirt-glib, libvirt-perl, libvirt-tck) > Will o

[libvirt] [PATCH] don't raise a missing network as an internal error

2009-07-07 Thread Daniel Veillard
https://bugzilla.redhat.com/show_bug.cgi?id=509189 When defining a domain referencing a missing network the erro should not be raised as an internal error. Patch by Paolo Bonzini looks fine to me, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ dan...@veill

Re: [libvirt] [PATCH] don't raise a missing network as an internal error

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 04:06:03PM +0200, Daniel Veillard wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=509189 > > When defining a domain referencing a missing network the erro should > not be raised as an internal error. Patch by Paolo Bonzini looks fine to > me, ACK, though I get the f

[libvirt] Cannot get network to work with LXC

2009-07-07 Thread James Yu
Hi all, I tried to use the first xml config in http://libvirt.org/drvlxc.html to setup my first light weight container. However, I was unable to get the container's network to work. When I tried to give the interface an IP, I got the following error message: sh-4.0# /sbin/ifconfig veth1 192.168.0

Re: [libvirt] [PATCH] don't raise a missing network as an internal error

2009-07-07 Thread Daniel Veillard
On Tue, Jul 07, 2009 at 03:16:43PM +0100, Daniel P. Berrange wrote: > On Tue, Jul 07, 2009 at 04:06:03PM +0200, Daniel Veillard wrote: > > https://bugzilla.redhat.com/show_bug.cgi?id=509189 > > > > When defining a domain referencing a missing network the erro should > > not be raised as an inter

Re: [libvirt] Cannot get network to work with LXC

2009-07-07 Thread Serge E. Hallyn
Quoting James Yu (cyu...@gmail.com): > Hi all, > > I tried to use the first xml config in http://libvirt.org/drvlxc.html to Which version of libvirt were you using? Was it the one which came with your distro, and, if so, which distro? You might try installing the version from git clone

[libvirt] [PATCH] provide the object name on lookup error

2009-07-07 Thread Daniel Veillard
There were a number of places where the object name was not not reported, based on the use I don't think the string could be NULL in any of those. I also fixed a few POOL error which were really VOLume ones. Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ da

Re: [libvirt] Re: Can't boot guest after adding an IDE storage

2009-07-07 Thread Gene Czarcinski
On Tuesday 07 July 2009 09:03:57 Daniel Veillard wrote: > On Tue, Jul 07, 2009 at 05:58:37PM +0530, M. Mohan Kumar wrote: > > Hi, > > > > We installed Fedora 11 guest in an emulated SCSI disk using virt-install. > > After the installation the guest was shutdown and additional IDE virtual > > storag

[libvirt] PATCH: Allow libvirtd autostart to be disabled

2009-07-07 Thread Daniel P. Berrange
When connecting to a non-root libvirt driver, the remote libvirt will attempt to perform autostart of the libvirtd daemon. This is not always desirable, particularly in test scripts where you don't want the daemon autostarted every time it crashes - you want a clear failure Daniel src/remote_int

[libvirt] PATCH: Set stdout/err to /dev/null for spawned children

2009-07-07 Thread Daniel P. Berrange
If passing a 'NULL' to virExec() for the stdout/err file descriptor it is intended that the child process get connected to /dev/null. This behaviour was previously broken when built with DEBUG enabled, and with Amy's recent logging change, it is now permanently broken. The problem this causes, is t

Re: [libvirt] [PATCH] provide the object name on lookup error

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 05:52:03PM +0200, Daniel Veillard wrote: > There were a number of places where the object name was not not > reported, based on the use I don't think the string could be NULL > in any of those. I also fixed a few POOL error which were really > VOLume ones. There's a few p

Re: [libvirt] [PATCH 1/2] Added new filesystem glusterfs

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 05:46:19AM -0700, Harshavardhana wrote: > Added new enum for glusterfs, another netfs based filesystem. > > --- > src/storage_conf.c |2 +- > src/storage_conf.h |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) ACK > > diff --git a/src/storage_conf.c b/

Re: [libvirt] [PATCH] provide the object name on lookup error

2009-07-07 Thread Cole Robinson
Daniel P. Berrange wrote: > On Tue, Jul 07, 2009 at 05:52:03PM +0200, Daniel Veillard wrote: >> There were a number of places where the object name was not not >> reported, based on the use I don't think the string could be NULL >> in any of those. I also fixed a few POOL error which were really

Re: [libvirt] [PATCH 2/2] RPM spec file updated with glusterfs dependency

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 05:47:56AM -0700, Harshavardhana wrote: > Add new dependency for glusterfs rpm. > --- > libvirt.spec.in |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/libvirt.spec.in b/libvirt.spec.in > index 1e8a8ef..ccef3a5 100644 > --- a/libvirt.spec.in

[libvirt] [PATCH] use gnulib via a submodule

2009-07-07 Thread Jim Meyering
Please NOTE: do not apply the changes below. They are incomplete because I've removed all the boring, repetitive parts. If you want to apply something, use this: http://meyering.net/code/tmp/libvirt-submodule.patch.bz2 For an overall summary of the process of hooking up gnulib as a submodule, s

Re: [libvirt] [PATCH] allow to set path to xen userspace tools

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 01:16:02PM +0200, Guido G?nther wrote: > Hi, > attached patch makes the path to the xen userspace tools configurable. > Debian keeps this under /usr/lib/xen-default/ instead of /usr/lib/xen/. > We don't have the amd64 libs in /usr/lib64/xen either so we can use: > ./configu

Re: [libvirt] PATCH: 0/3: Run QEMU guests within a CGroup

2009-07-07 Thread Daniel P. Berrange
On Mon, Jul 06, 2009 at 11:03:10PM +0200, Nikola Ciprich wrote: > Hello Daniel, > I'd like to ask, what is the progres of qemu cgroups support? > are there any problems preventing it from getting into git? It should be in this next release we do. Daniel > On Thu, Feb 26, 2009 at 04:36:10PM +

[libvirt] PATCH Fix QEMU monitor prompt confusion

2009-07-07 Thread Daniel P. Berrange
When you first connect to the QEMU monitor after a VM starts, it will print out QEMU 0.10.5 monitor - type 'help' for more information We already have code to detect & discard this. Unfortunately it seems that with QEMU >= 0.10.0 it will also print out this prompt when you disconnect and recon

[libvirt] PATCH: Handle changed monitor syntax for PCI hotplug/unplug

2009-07-07 Thread Daniel P. Berrange
Annoyingly when PCI hotplug support was merged from KVM into QEMU the monitor syntax changed :-( This patch tries to deal with this by detecting the error message, and then re-trying with the alternative syntax. It also ensures the monitor error mesages is include with any libvirt error raised

[libvirt] PATCH: Relabel disk images *before* hotplugging

2009-07-07 Thread Daniel P. Berrange
For some bizarre reason the original code we added for sVirt disk labelling with hotplug did not relabel the disks until after telling QEMU to hotplug. Clearly this is going to fail 99% of the time. It also forgot todo labelling when invoking a CDROM media change src/qemu_driver.c |6 --

Re: [libvirt] [PATCH] provide the object name on lookup error

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 12:52:48PM -0400, Cole Robinson wrote: > Kind of a tangent, but... > > Can we go back to using macros for all this duplicate validation? For > example, the idiom: > > if (!vm) { > char uuidstr[VIR_UUID_STRING_BUFLEN]; > virUUIDFormat(dom->uuid, uuidstr)

Re: [libvirt] [PATCH] Power Hypervisor Support for libvirt - minimum set of features

2009-07-07 Thread Daniel P. Berrange
On Mon, Jul 06, 2009 at 04:40:58PM -0300, Eduardo Otubo wrote: > Hello all, > > This is the last version of this enormous PATCH. All minor bugs are > fixed, and the minimal set of features are implemented. Any comments are > alwayes welcome. This looks like it has addressed all the issues I rai

Re: [libvirt] [PATCH] allow to set path to xen userspace tools

2009-07-07 Thread Guido Günther
On Tue, Jul 07, 2009 at 06:02:20PM +0100, Daniel P. Berrange wrote: > On Tue, Jul 07, 2009 at 01:16:02PM +0200, Guido G?nther wrote: > > Hi, > > attached patch makes the path to the xen userspace tools configurable. > > Debian keeps this under /usr/lib/xen-default/ instead of /usr/lib/xen/. > > We

[libvirt] PATCH: Video device config

2009-07-07 Thread Tom Hughes
I recently posted a patch on the RedHat bugzilla (bug #467038) to add a model attribute to the graphics element so that the different graphics cards supported by QEMU could be selected. Apparently that approach has been rejected in the past however, and the preferred approach is to add a separ

Re: [libvirt] PATCH: Video device config

2009-07-07 Thread Daniel P. Berrange
On Tue, Jul 07, 2009 at 10:17:18PM +0100, Tom Hughes wrote: > I recently posted a patch on the RedHat bugzilla (bug #467038) to add a > model attribute to the graphics element so that the different graphics > cards supported by QEMU could be selected. > > Apparently that approach has been reject

[libvirt] PATCH 1/2: Support element in domain config

2009-07-07 Thread Daniel P. Berrange
This patch implements the basic domain config support for a element as per http://www.redhat.com/archives/libvir-list/2009-May/msg00424.html Since last time this - Centralized the logic for default video model & default RAM - Added a type=vbox for their custom graphics card Daniel From: D

[libvirt] PATCH 2/2: Support element in QEMU driver

2009-07-07 Thread Daniel P. Berrange
This patch extends the QEMU driver to support the element for configuring the type of video adapter exposed to the guest. It implements the XML -> ARGV and ARGV -> XML conversions, so both import & export work. Some of the existing tests are updated to use the element to get test coverage. The Q

Re: [libvirt] [PATCH] use gnulib via a submodule

2009-07-07 Thread Jim Meyering
Jim Meyering wrote: > Please NOTE: do not apply the changes below. > They are incomplete because I've removed all the boring, repetitive parts. > If you want to apply something, use this: > > http://meyering.net/code/tmp/libvirt-submodule.patch.bz2 By the way, for those new to git, in order to a

Re: [libvirt] [PATCH 2/2] RPM spec file updated with glusterfs dependency

2009-07-07 Thread Harshavardhana
Daniel, Yes indeed but as in libvirt is a standalone project it should have its own dependencies. Right now this only effects the ovirt-node-image part. Was thinking of patching virt-manager also?. I have other doubts are you all planning for a generic function for mounting filesystems agnost