[libvirt] Assigning a static IP to the Linux Container through libvirt.

2010-03-04 Thread Kumar L Srikanth-B22348
Hi, I want to assign a static IP 172.16.1.20 to one of the containers created using libvirt. For that, I defined a network 'mynetwork' using net-define. The XML format for that network is shown below: mynetwork f76c44dd-bcef-6b45-4a50-00d575a369ad I started the network 'mynetwork'

Re: [libvirt] [PATCH] Change default for storage uid/gid from getuid()/getgid() to -1/-1

2010-03-04 Thread Laine Stump
On 03/04/2010 05:54 PM, Cole Robinson wrote: A few of us talked offline, and decided we should commit this patch as is for the release. Dan's proposed change is just a cleanup which can wait, and the patch fixes a regression when building pools with virt-manager, so I've just pushed it. Thanks,

Re: [libvirt] [PATCH] Remove hard dependency on DMI

2010-03-04 Thread Ed Swierk
On Thu, Mar 4, 2010 at 10:31 AM, Dave Allan wrote: > I reworked udevSetupSystemDev into this style (which also fixes the bug you > pointed out that it didn't properly free resources on error).  The patch > also makes failure to find DMI data non-fatal. Your patch works fine. Thanks. Acked-by: Ed

Re: [libvirt] [PATCH] Change default for storage uid/gid from getuid()/getgid() to -1/-1

2010-03-04 Thread Cole Robinson
On 03/02/2010 09:51 AM, Laine Stump wrote: > On 03/02/2010 05:48 AM, Daniel P. Berrange wrote: >> On Tue, Mar 02, 2010 at 02:17:59AM -0500, Laine Stump wrote: >> >>> +uid_t uid = (vol->target.perms.uid == -1) ? getuid() : >>> vol->target.perms.uid; >>> +gid_t gid = (vol->target.perms.gid

[libvirt] [PATCH 5/9] Define wire protocol format for vol wiping

2010-03-04 Thread David Allan
--- src/remote/remote_protocol.c | 11 +++ src/remote/remote_protocol.h |9 + src/remote/remote_protocol.x |8 +++- 3 files changed, 27 insertions(+), 1 deletions(-) diff --git a/src/remote/remote_protocol.c b/src/remote/remote_protocol.c index 701acab..6bd9019 10064

[libvirt] [PATCH 9/9] Virsh support for vol wiping

2010-03-04 Thread David Allan
--- tools/virsh.c | 42 ++ 1 files changed, 42 insertions(+), 0 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index 65487ed..00f412e 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -5221,6 +5221,47 @@ cmdVolDelete(vshControl *ctl, const vshCmd

[libvirt] [PATCH 7/9] Implement the remote dispatch bits of vol wiping

2010-03-04 Thread David Allan
* I had to remove daemon/remote_dispatch* and do a make remote.c in daemon in order to get the dispatch files regenerated properly. The make was failing before I did that. --- daemon/remote.c | 32 daemon/remote_dispatch_args.h |1

[libvirt] [PATCH 8/9] Simplified version of volume wiping based on feedback from the list.

2010-03-04 Thread David Allan
--- src/storage/storage_driver.c | 224 ++ 1 files changed, 224 insertions(+), 0 deletions(-) diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c index f3be6de..b0afb05 100644 --- a/src/storage/storage_driver.c +++ b/src/storage/storag

[libvirt] [PATCH 4/9] Implement the public API for vol wiping

2010-03-04 Thread David Allan
--- src/libvirt.c | 47 +++ 1 files changed, 47 insertions(+), 0 deletions(-) diff --git a/src/libvirt.c b/src/libvirt.c index d9242bc..6abf74f 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -8468,6 +8468,53 @@ error: /** + * virStorageVolWipe:

[libvirt] [PATCH 6/9] Implement RPC client for vol wiping

2010-03-04 Thread David Allan
--- src/remote/remote_driver.c | 27 +++ 1 files changed, 27 insertions(+), 0 deletions(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index b7b2e09..9d28c84 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -5538,6 +553

[libvirt] [PATCH 3/9] Add vol wiping to ESX storage driver struct

2010-03-04 Thread David Allan
--- src/esx/esx_storage_driver.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/esx/esx_storage_driver.c b/src/esx/esx_storage_driver.c index d09831a..3062473 100644 --- a/src/esx/esx_storage_driver.c +++ b/src/esx/esx_storage_driver.c @@ -101,6 +101,7 @@ static virSt

[libvirt] [PATCH 1/9] Add public API for volume wiping

2010-03-04 Thread David Allan
--- include/libvirt/libvirt.h.in |2 ++ src/libvirt_public.syms |1 + 2 files changed, 3 insertions(+), 0 deletions(-) diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in index 0d1b5b5..1430a95 100644 --- a/include/libvirt/libvirt.h.in +++ b/include/libvirt/libvi

[libvirt] [PATCH 2/9] Define the internal driver API for vol wiping

2010-03-04 Thread David Allan
--- src/driver.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/src/driver.h b/src/driver.h index 7b7dfea..6e1403a 100644 --- a/src/driver.h +++ b/src/driver.h @@ -722,6 +722,10 @@ typedef int unsigned int flags); typede

[libvirt] [PATCH 0/9] Storage volume wiping API

2010-03-04 Thread David Allan
Here's the revised volume wiping API patchset (thanks to Dan for the improved name). I'm sending a replacement patchset since the name change made for a very large and messy incremental. This set contains everybody's feedback, which was all very helpful. Many thanks. Dave David Allan (9):

Re: [libvirt] Libvirt Auth

2010-03-04 Thread Bryan Kearney
On 03/03/2010 12:11 PM, Marco Mornati wrote: Hello guys, I've created a plugin for Hudson (CI) to manage virtmachines using libvirt java (start vm for a build and stop just after). I've just a problem in authentication, in fact I cannot send uname and password using code, libvirt ask me for the

[libvirt] Libvirt segfault in qemuMonitorSend() with multi-threaded API use

2010-03-04 Thread Adam Litke
I have a multi-threaded Python program that shares a single libvirt connection object among several threads (one thread per active domain on the system plus a management thread). On a heavily loaded host with 8 running domains I am getting a consistent libvirtd segfault in the qemu monitor handlin

Re: [libvirt] [PATCH 2/2] Addition of XenAPI support to libvirt

2010-03-04 Thread Matthias Bolte
2010/2/26 Sharadha Prabhakar (3P) : > This is the last patch in this series. It contains changes made to the > following files. > ~/libvirt/configure.ac > ~/libvirt/src/Makefile.am > ~/libvirt/src/util/virterror.c > ~/libvirt/src/driver.h > ~/libvirt/include/libvirt/virterror.h > ~/libvirt/virterr

Re: [libvirt] [PATCH] Remove hard dependency on DMI

2010-03-04 Thread Dave Allan
On 03/03/2010 07:20 PM, Ed Swierk wrote: On Wed, Mar 3, 2010 at 2:57 PM, Dave Allan wrote: Although I use goto a lot, I generally try to avoid multiple labels within a function, just because I think it gets out of hand really quickly. Although it's a slightly more invasive patch, would you ref

Re: [libvirt] FW: Libvirt and openvz?

2010-03-04 Thread Thomas Graves
Ok, after recompiling libvirt 0.7.4 I am able to get it to connect and sort of works with openvz. I always get an error about "error: internal error Could not set UUID". It ends up defining the domain but uses the id of last id+1 instead of 600. If I dumpxml on the domain it created it looks di

Re: [libvirt] [PATCH] fix rawhide/F13 --no-add-needed/dlopen-related build failures

2010-03-04 Thread Daniel Veillard
On Thu, Mar 04, 2010 at 05:14:38PM +0100, Jim Meyering wrote: > Diego, > > I saw no follow-up here, > http://thread.gmane.org/gmane.comp.emulators.libvirt/21398/focus=21671 > so I've fixed your two patches to address Eric's review feedback, > and confirmed that they solve the problem on Rawhide.

[libvirt] [PATCH] fix rawhide/F13 --no-add-needed/dlopen-related build failures

2010-03-04 Thread Jim Meyering
Diego, I saw no follow-up here, http://thread.gmane.org/gmane.comp.emulators.libvirt/21398/focus=21671 so I've fixed your two patches to address Eric's review feedback, and confirmed that they solve the problem on Rawhide. >From 55c182d89cd6e5417eafe9f4bdf2e0ef7f8721f5 Mon Sep 17 00:00:00 2001

[libvirt] RFC: Add further domain event callbacks

2010-03-04 Thread Daniel P. Berrange
In the current domain APIs, we currently have support for getting notified of domain lifecycle transition events. THis is done using two methods int virConnectDomainEventRegister(virConnectPtr conn, virConnectDomainEventCallback cb,

Re: [libvirt] [PATCH] Change logrotate to be per-hypervisor logs

2010-03-04 Thread Eric Blake
According to Daniel Veillard on 3/4/2010 7:36 AM: > Original bug is: > > https://bugzilla.redhat.com/show_bug.cgi?id=547514 > > maybe this could be done in slightly different way, possibly more > generic, but I think doing a simple split is good enough for now. Seems easy enough. > +++ b/daem

[libvirt] [PATCH] Change logrotate to be per-hypervisor logs

2010-03-04 Thread Daniel Veillard
Original bug is: https://bugzilla.redhat.com/show_bug.cgi?id=547514 maybe this could be done in slightly different way, possibly more generic, but I think doing a simple split is good enough for now. Change logrotate to be per-hypervisor logs Having a single logrotate configuration file for

Re: [libvirt] FW: Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Daniel P. Berrange
On Thu, Mar 04, 2010 at 06:20:31PM +0530, Kumar L Srikanth-B22348 wrote: > Hi Daniel, > One more thing is that in my host machine vibr0 inteface is missing. > Because of the the 'default' network is in 'inactive' state and if I > tried to activate, it's giving the following error: > > error: Faile

[libvirt] FW: Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Kumar L Srikanth-B22348
Hi Daniel, One more thing is that in my host machine vibr0 inteface is missing. Because of the the 'default' network is in 'inactive' state and if I tried to activate, it's giving the following error: error: Failed to start network default error: internal error '/usr/sbin/dnsmasq --strict-order --

[libvirt] [PATCH] Fix USB passthrough based on product/vendor

2010-03-04 Thread Daniel P. Berrange
Changeset commit 5073aa994af460e775cb3e548528e28d7660fcc8 Author: Cole Robinson Date: Mon Jan 11 11:40:46 2010 -0500 Added support for product/vendor based passthrough, but it only worked at the security driver layer. The main guest XML config was not updated with the resolved bus/device

[libvirt] [PATCH 1/3] Avoid creating top level cgroups if just querying for existance

2010-03-04 Thread Daniel P. Berrange
When getting the driver/domain cgroup it is possible to specify whether it should be auto created. If auto-creation was turned off, libvirt still mistakenly created its own top level cgroup * src/util/cgroup.c: Honour autocreate flag for top level cgroup --- src/util/cgroup.c |7 --- 1 fi

[libvirt] [PATCH 2/3] Misc fixes for LXC cgroups setup

2010-03-04 Thread Daniel P. Berrange
When using the 'ns' cgroup controller, the moment a process calls 'unshare(CLONE_NEWNS)', it will be given a private cgroup tree under its current location. This really messages up the LXC controller process, because it ends up creating the containers' cgroup in the wrong place. The fix is fairly e

[libvirt] [PATCH 3/3] Tweak container initialization to make upstart/init happier

2010-03-04 Thread Daniel P. Berrange
Upstart crashes & burns in a heap if $TERM environment variable is missing. Presumably the kernel always sets this when booting init on a real machine, so libvirt should set it for containers too. To make a typical inittab / mingetty setup happier, we need to symlink the primary console /dev/pts/0

[libvirt] [PATCH 0/3] Misc fixes for LXC driver

2010-03-04 Thread Daniel P. Berrange
This series provides some small fixes to the LXC driver which bring it closer to being able to run a full Fedora OS with regular init (upstart) startup process -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Kumar L Srikanth-B22348
Hi, I want to create eth0 network interface in the container, which is linked to br0 bridge in the host machine. For that, I have created a bridge 'br0' with the IP '172.16.1.10'. Now, I want to create a network interface eth0 in the container with the IP '172.16.1.17'. I created the following netw

Re: [libvirt] [libvirt-users] Regarding SPICE

2010-03-04 Thread anuj rampal
Hi, Thanks a lot for the reply. Is it going to be there in the future? And i wanted to confirm one more thing... KVM on RHEL 5.4 supports SPICE through libvirt? Because there were some patches regarding " Implement RHEL-5.4 support for SPICE graphics" Regards Anuj On Thu, Mar 4, 2010 at 3:31

Re: [libvirt] [PATCH 8/9] Simplified version of volume zeroing based on feedback from the list.

2010-03-04 Thread Jim Meyering
David Allan wrote: > diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c ... Hi Dave, > +static int > +storageVolumeZeroSparseFile(virStorageVolDefPtr vol, > +struct stat *st, > +int fd) Since the only use of "st" is fo

Re: [libvirt] [PATCH] Add TCK script for verifying disk based storage pools

2010-03-04 Thread Daniel P. Berrange
On Wed, Mar 03, 2010 at 11:54:28AM -0700, Eric Blake wrote: > According to Daniel P. Berrange on 3/3/2010 11:46 AM: > > High-level review nits below. However, I still count myself as too > inexperienced to offer either an ACK or NAK on this. > > > This adds a reasonably comprehensive test script

Re: [libvirt] [PATCH] Add TCK script for verifying disk based storage pools

2010-03-04 Thread Jim Meyering
Eric Blake wrote: ... >> +# List of host block devices that the test suite can trash all >> +# data on, for purposes of testing. You have been warned ! > > Indeed! The intrepid souls who run these tests might appreciate a crosscheck. Let them specify more than just a block device name, where it's

Re: [libvirt] [PATCH 9/9] Virsh support for vol zeroing

2010-03-04 Thread Daniel P. Berrange
On Tue, Mar 02, 2010 at 05:13:34PM -0500, David Allan wrote: > --- > tools/virsh.c | 42 ++ > 1 files changed, 42 insertions(+), 0 deletions(-) > > diff --git a/tools/virsh.c b/tools/virsh.c > index a6a637d..e52b011 100644 > --- a/tools/virsh.c > +++ b/to

Re: [libvirt] [PATCH 8/9] Simplified version of volume zeroing based on feedback from the list.

2010-03-04 Thread Daniel P. Berrange
On Tue, Mar 02, 2010 at 05:13:33PM -0500, David Allan wrote: > @@ -1518,6 +1521,220 @@ cleanup: > return ret; > + > +static int > +storageZeroExtent(virStorageVolDefPtr vol, > + struct stat *st, > + int fd, > + size_t extent_start, > +

Re: [libvirt] [PATCH 4/9] Implement the public API for vol zeroing

2010-03-04 Thread Daniel P. Berrange
On Tue, Mar 02, 2010 at 04:31:19PM -0700, Eric Blake wrote: > According to David Allan on 3/2/2010 3:13 PM: > > + * @flags: future flags, use 0 for now > > + * > > + * Ensure that future reads from the storage volume return zeroes. > > + * > > + * Returns 0 on success, or -1 on error > > + */ > > +

Re: [libvirt] [PATCH 2/9] Add public API for volume zeroing

2010-03-04 Thread Daniel P. Berrange
On Tue, Mar 02, 2010 at 05:13:27PM -0500, David Allan wrote: > --- > include/libvirt/libvirt.h.in |2 ++ > src/libvirt_public.syms |1 + > 2 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in > index 0d1b5b5..6f

Re: [libvirt] Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Daniel P. Berrange
On Thu, Mar 04, 2010 at 03:38:43PM +0530, Kumar L Srikanth-B22348 wrote: > So, again I need to add one network interface in the libvirt XML?. It > will not take the interface which was created earlier while creating > container using lxc-tools?. > But, in the container which was created with lxc-to

Re: [libvirt] Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Kumar L Srikanth-B22348
So, again I need to add one network interface in the libvirt XML?. It will not take the interface which was created earlier while creating container using lxc-tools?. But, in the container which was created with lxc-tools(lxc-create, lxc-start ..etc), I am not getting any such issues. Why I am get

Re: [libvirt] [libvirt-users] Regarding SPICE

2010-03-04 Thread Daniel P. Berrange
On Thu, Mar 04, 2010 at 12:07:06PM +0530, anuj rampal wrote: > Hi, > > I have set up KVM on FC12. > I was wondering if its possible to Display the VM using spice + libvirt? KVM in Fedora 12 does not support SPICE, since it is not merged in the upstream repository. Daniel -- |: Red Hat, Engineer

Re: [libvirt] Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Daniel P. Berrange
On Thu, Mar 04, 2010 at 03:25:18PM +0530, Kumar L Srikanth-B22348 wrote: > Hi Daniel, > I am able to get the 'ifconfig' details in any one of the following 2 > cases: > 1. If I execute '/sbin/ifconfig' > 2. If I include '/sbin' in the PATH environment variable. > > But, the result of ifconfig is s

Re: [libvirt] Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Kumar L Srikanth-B22348
Hi Daniel, I am able to get the 'ifconfig' details in any one of the following 2 cases: 1. If I execute '/sbin/ifconfig' 2. If I include '/sbin' in the PATH environment variable. But, the result of ifconfig is showing all the IP's available in the host machine (The same result when I execute ifcon

Re: [libvirt] Live Migration with non-shared storage for kvm

2010-03-04 Thread Daniel Veillard
On Thu, Mar 04, 2010 at 10:04:30AM +0200, Kenneth Nagin wrote: > > >Daniel Veillard wrote on 03/03/2010 10:18:08: > > > Daniel Veillard > > 03/03/2010 10:18 > > > > Please respond to > > veill...@redhat.com > > > > To > > > > Kenneth Nagin/Haifa/i...@ibmil > > > > cc > > > > "Daniel P. Berrange

Re: [libvirt] Bringing up LXC container from virsh command prompt.

2010-03-04 Thread Daniel P. Berrange
On Thu, Mar 04, 2010 at 11:54:18AM +0530, Kumar L Srikanth-B22348 wrote: > Hi, > I am new to libvirt virsh console. > Currently I am working on Linux Containers(LXC). After lot of > exploration, I am able to create a Linux Container(fedora12) with the > help of > lxc-tools(lxc-create, lxc-start, .

Re: [libvirt] Live Migration with non-shared storage for kvm

2010-03-04 Thread Kenneth Nagin
>Daniel Veillard wrote on 03/03/2010 10:18:08: > Daniel Veillard > 03/03/2010 10:18 > > Please respond to > veill...@redhat.com > > To > > Kenneth Nagin/Haifa/i...@ibmil > > cc > > "Daniel P. Berrange" , libvir-list@redhat.com > > Subject > > Re: [libvirt] Live Migration with non-shared storage