[libvirt] virStorageVolCreateXMLFrom

2009-06-22 Thread Kenneth Nagin
I'd like to get a better understanding of the virStorageVolCreateXMLFrom api Its name and signature imply that it is a cloning operation that copies a volume from one location to another. Is this correct? Assuming that the answer is yes I have few other questions: Does it support remote copy of

[libvirt] [PATCH] Cleanup when Storage Pool Create fails.

2009-06-22 Thread Henrik Persson E
When libvirt fails to create a storage pool (e.g. a disk pool does not have a partition table), the failed storage pool is not cleaned up correctly and is not possible to re-create the pool again. This attached patch addresses this problem. /Henrik pool_create_fixup.patch Description:

Re: [libvirt] netbsd5.0

2009-06-22 Thread Daniel P. Berrange
On Sun, Jun 21, 2009 at 10:56:48AM +0200, Fabien Georget wrote: Hi I want to run netbsd 5.0 with kvm in virt-manager. It works with default option but any network devices is recognized by the netbsd kernel. To enable network with qemu-kvm, we had to disable the acpi and add other

Re: [libvirt] virStorageVolCreateXMLFrom

2009-06-22 Thread Daniel P. Berrange
On Mon, Jun 22, 2009 at 09:59:00AM +0300, Kenneth Nagin wrote: I'd like to get a better understanding of the virStorageVolCreateXMLFrom api Its name and signature imply that it is a cloning operation that copies a volume from one location to another. Is this correct? Yes this method does a

Re: [libvirt] [PATCH] Cleanup when Storage Pool Create fails.

2009-06-22 Thread Daniel P. Berrange
On Mon, Jun 22, 2009 at 09:27:32AM +0200, Henrik Persson E wrote: When libvirt fails to create a storage pool (e.g. a disk pool does not have a partition table), the failed storage pool is not cleaned up correctly and is not possible to re-create the pool again. This attached patch addresses

Re: [libvirt] Can't scale XP on remote

2009-06-22 Thread Daniel P. Berrange
On Sat, Jun 20, 2009 at 04:02:05PM -0400, sean darcy wrote: sean darcy wrote: Just starting out with an XP guest on Fedora 11 kvm. I'm using virt-manager to connect from a laptop. Server: kernel-2.6.29.4-167.fc11.x86_64 qemu-*-0.10.50-6.kvm86.fc11.x86_64 libvirt-0.6.4-2.fc11.x86_64 On

Re: [netcf-devel] [libvirt] [RFC] Reporting host interface status/statistics via netcf/libvirt, and listing active vs. inactive interfaces

2009-06-22 Thread Jonas Eriksson
On Thu, Jun 18, 2009 at 05:14:44PM + David Lutterkort wrote: [..] There's a few more options we need to add for completeness, at least PERSISTENT_DHCLIENT, DHCPRELEASE, and DHCLIENT_IGNORE_GATEWAY are supported by initscripts. This raises a question - how should the features of some

Re: [netcf-devel] [libvirt] [RFC] Reporting host interface status/statistics via netcf/libvirt, and listing active vs. inactive interfaces

2009-06-22 Thread Daniel P. Berrange
On Mon, Jun 22, 2009 at 10:58:49AM +0200, Jonas Eriksson wrote: On Thu, Jun 18, 2009 at 05:14:44PM + David Lutterkort wrote: [..] There's a few more options we need to add for completeness, at least PERSISTENT_DHCLIENT, DHCPRELEASE, and DHCLIENT_IGNORE_GATEWAY are supported by

Re: [netcf-devel] [libvirt] [RFC] Reporting host interface status/statistics via netcf/libvirt, and listing active vs. inactive interfaces

2009-06-22 Thread Jonas Eriksson
On Thu, Jun 18, 2009 at 11:02:03PM + David Lutterkort wrote: On Thu, 2009-06-18 at 09:15 +0200, Jonas Eriksson wrote: I am a bit critical to the policy restrictions of the current incarnation of the netcf API. Currently, a interface (or connection) has to have an IP address and a bridge

Re: [libvirt] netbsd5.0

2009-06-22 Thread Fabien Georget
Le lundi 22 juin 2009 11:04:17, Daniel P. Berrange a écrit : To enable network with qemu-kvm, we had to disable the acpi and add other options like -tdf, -localtime ... Hmm, I would not expect BSD to need 'localtime' -... yes, sorry, locatime is not what I meant. The other mandatory option

Re: [libvirt] [PATCH] Cleanup when Storage Pool Create fails.

2009-06-22 Thread Daniel Veillard
On Mon, Jun 22, 2009 at 09:27:32AM +0200, Henrik Persson E wrote: When libvirt fails to create a storage pool (e.g. a disk pool does not have a partition table), the failed storage pool is not cleaned up correctly and is not possible to re-create the pool again. This attached patch addresses

[libvirt] virsh -no-kvm problem on debian install

2009-06-22 Thread ChaosMedia WebDev
Hi, i've installed lastest kvm-86 and libvirt-0.6.4 tarballs on my debian stable and have a problem using virsh No matter what i do in the guest domain config or with /usr/bin/ symlinks, libvirt keeps using the -no-kvm option when starting guests when my cpu has kvm-amd support, modules are

Re: [libvirt] Extend virConfParser to allow parsing VMware VMX config files

2009-06-22 Thread Daniel Veillard
On Fri, Jun 19, 2009 at 05:54:47PM +0200, Matthias Bolte wrote: 2009/6/19 Daniel P. Berrange berra...@redhat.com: On Fri, Jun 19, 2009 at 10:01:20AM +0200, Matthias Bolte wrote: Hi, As part of the VMware ESX driver I need to parse VMware VMX config files. This files have basically the

Re: [libvirt] [PATCH 1/4] Define Public API for new virInterface Functions.

2009-06-22 Thread Daniel Veillard
On Thu, Jun 18, 2009 at 02:44:27PM -0400, Laine Stump wrote: This adds virConnectListDefinedInterfaces() and virConnectNumOfDefinedInterfaces(). Looks fine. None of the other ListDefined function carries a flag so I think we can do without here. ACK Daniel -- Daniel Veillard |

Re: [libvirt] [PATCH 2/4] Define internal driver API for new virInterface functions

2009-06-22 Thread Daniel Veillard
On Thu, Jun 18, 2009 at 02:44:28PM -0400, Laine Stump wrote: --- src/driver.h |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/src/driver.h b/src/driver.h index ca759ff..2502c63 100644 --- a/src/driver.h +++ b/src/driver.h @@ -508,6 +508,12 @@ typedef int

Re: [libvirt] [PATCH 3/4] Implement Public API of new virInterface functions

2009-06-22 Thread Daniel Veillard
On Thu, Jun 18, 2009 at 02:44:29PM -0400, Laine Stump wrote: --- src/libvirt.c | 89 +++-- 1 files changed, 86 insertions(+), 3 deletions(-) Okay, ACK, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit

Re: [libvirt] [PATCH 4/4] Implement client and server side of RPC for new virInterface functions

2009-06-22 Thread Daniel Veillard
On Thu, Jun 18, 2009 at 02:44:30PM -0400, Laine Stump wrote: --- qemud/remote.c | 51 qemud/remote_dispatch_args.h |1 + qemud/remote_dispatch_prototypes.h | 14 +++ qemud/remote_dispatch_ret.h|2 +

[libvirt] Storage volume issues when creating an LVM or RAID partition

2009-06-22 Thread Henrik Persson E
Hi everyone, I have currently some patches coming up for storage pools and volumes based on physical disks. However I have an issue that I would like to hear your opinion about. When it comes to storage pools and volumes based on physical disks, the name given does not really have any real

Re: [libvirt] virsh -no-kvm problem on debian install

2009-06-22 Thread Daniel P. Berrange
On Mon, Jun 22, 2009 at 12:18:44PM +0200, ChaosMedia WebDev wrote: Hi, i've installed lastest kvm-86 and libvirt-0.6.4 tarballs on my debian stable and have a problem using virsh No matter what i do in the guest domain config or with /usr/bin/ symlinks, libvirt keeps using the -no-kvm

[libvirt] [PATCH] Do a correct format mapping of partitions

2009-06-22 Thread Henrik Persson E
This patch reads the partition type and sets the correct target format of the storage volume when based on physical disk. --- libvirt-0.6.4.org/src/parthelper.c 2008-09-02 11:24:21.0 +0200 +++ libvirt-0.6.4/src/parthelper.c 2009-06-22 16:29:49.108681000 +0200 @@ -67,6 +67,7 @@

Re: [libvirt] Re: [PATCH] Fix raw storage volume creation for allocation capacity.

2009-06-22 Thread Cole Robinson
Daniel Veillard wrote: On Thu, Jun 18, 2009 at 03:13:06PM -0400, Cole Robinson wrote: Cole Robinson wrote: CreateXMLFrom changes accidentally caused all raw volume creation to be fully allocated (as though allocation == capacity). Fix this. Also force CreateXMLFrom to maintain the previous

Re: [libvirt] [PATCH] Refactor storage XML parsing to be consistent with domain/network conf.

2009-06-22 Thread Cole Robinson
Cole Robinson wrote: On 06/19/2009 01:09 PM, Daniel P. Berrange wrote: On Fri, Jun 19, 2009 at 12:37:11PM -0400, Cole Robinson wrote: The storage driver arranges its parsing routines in a way that make them difficult to use in the test driver for non-default file parsing. This refactoring

[libvirt] [PATCH 0/3] Control Linux capabilities in libvirt

2009-06-22 Thread Daniel P. Berrange
The libvirtd system instance runs privileged (as root), and likewise so do VMs run from it. This is undesirable for obvious security reasons, even if you do have SELinux available. Linux has a concept of capabilities which are actually what gives 'root' its power. If you take away all capabilities

Re: [libvirt] [PATCH 1/3] Probe for libcap-ng

2009-06-22 Thread Daniel P. Berrange
Probe for capng in configure, and set some RPM spec rules. Trivial boring stuff. Daniel diff -r 57a8eb45975e configure.in --- a/configure.in Mon Jun 22 11:54:49 2009 + +++ b/configure.in Mon Jun 22 19:00:54 2009 +0100 @@ -749,6 +749,49 @@ AM_CONDITIONAL([HAVE_NUMACTL], [test $w

Re: [libvirt] [PATCH 2/3] Set basic capabilities needed for libvirtd

2009-06-22 Thread Daniel P. Berrange
This sets up some basic support in libvirtd for dropping privileges by removing capabilities, or changing uid/gid of the process. It needed a little movement of existing code to allow us to drop privileges in between initializing the daemon and initializing the drivers. As I mentioned in the

Re: [libvirt] [PATCH 3/3] Run all VMs without capabilities

2009-06-22 Thread Daniel P. Berrange
This patch adds a new flag to virExec() called VIR_EXEC_CLEAR_CAPS. If you set this flag than all capabilities are removed inbetween the fork() and exec() pair. It also updates QEMU and UML driver to run their VMs without any privileges. A mild security benefit for most distros today, but if

Re: [libvirt] problems with remote authentication with policykit

2009-06-22 Thread Guido Günther
On Thu, Jun 18, 2009 at 12:20:40PM -0400, Jim Paris wrote: Daniel P. Berrange wrote: We close the socket to the 'nc' process here so in theory it should be getting a HUP event from poll or EOF from read, etc and then exiting. Ominously though I see several patches to Fedora's 'nc' RPM at