Re: [libvirt] [PATCH v3 00/12] Add user namespace support for libvirt lxc

2013-06-11 Thread Richard Weinberger
Am 11.06.2013 05:12, schrieb Gao feng: On 06/11/2013 04:51 AM, Richard Weinberger wrote: Am 10.06.2013 21:53, schrieb Richard Weinberger: Am 10.06.2013 21:17, schrieb Richard Weinberger: Hi! Am 04.06.2013 13:03, schrieb Daniel P. Berrange: It's still under review. needs some ACK. If you can

Re: [libvirt] [PATCH v3 00/12] Add user namespace support for libvirt lxc

2013-06-11 Thread Gao feng
On 06/11/2013 02:02 PM, Richard Weinberger wrote: Am 11.06.2013 05:12, schrieb Gao feng: On 06/11/2013 04:51 AM, Richard Weinberger wrote: Am 10.06.2013 21:53, schrieb Richard Weinberger: Am 10.06.2013 21:17, schrieb Richard Weinberger: Hi! Am 04.06.2013 13:03, schrieb Daniel P. Berrange:

Re: [libvirt] [PATCH v3 00/12] Add user namespace support for libvirt lxc

2013-06-11 Thread Richard Weinberger
Am 11.06.2013 08:17, schrieb Gao feng: On 06/11/2013 02:02 PM, Richard Weinberger wrote: Am 11.06.2013 05:12, schrieb Gao feng: On 06/11/2013 04:51 AM, Richard Weinberger wrote: Am 10.06.2013 21:53, schrieb Richard Weinberger: Am 10.06.2013 21:17, schrieb Richard Weinberger: Hi! Am

[libvirt] [PATCH] LXC: Allow to input relative path of program for lxc-enter-namespace

2013-06-11 Thread Gao feng
User execvp to replace execv, so user can use relative path to exec programs in container. Such as lxc-enter-namespace domain ls And the user can still use absolute path to exec programs. Signed-off-by: Gao feng gaof...@cn.fujitsu.com --- tools/virsh-domain.c | 2 +- 1 file changed, 1

Re: [libvirt] [PATCH for v1.0.5-maint] Fix use of VIR_STRDUP vs strdup

2013-06-11 Thread Ján Tomko
On 06/10/2013 03:54 PM, Cole Robinson wrote: Whoops, I just pushed your patch to the maint branch before I saw your and Jan's responses. Thanks, now I don't feel so bad for failing to mention that the patch is missing virReportOOMError :) Jan -- libvir-list mailing list

Re: [libvirt] [PATCH v2 0/2] Resolve issues seen with schedinfo

2013-06-11 Thread Michal Privoznik
On 10.06.2013 18:06, John Ferlan wrote: These patches resolve an issue seen using 'virsh schedinfo domain' on a non running domain that have been present since 1.0.4 as a result of the cgroup infrastructure changes: https://www.redhat.com/archives/libvir-list/2013-April/msg00783.html The

Re: [libvirt] [PATCH 2/2] libxl: set bootloader for PV domains if not specified

2013-06-11 Thread Michal Privoznik
On 10.06.2013 22:21, Jim Fehlig wrote: The legacy xen toolstack will set pygrub as the bootloader if not specified. For compatibility, do the same in the libxl driver iff not using direct kernel boot. --- src/libxl/libxl_conf.c | 13 +++-- src/libxl/libxl_conf.h |1 + 2

Re: [libvirt] [PATCH 1/2] libxl: Report connect type as Xen

2013-06-11 Thread Michal Privoznik
On 10.06.2013 22:21, Jim Fehlig wrote: Currently, the libxl driver reports a connection type of xenlight. To be compatible with the legacy Xen driver, it should return Xen. Note: I noticed this while testing the libxl driver on OpenStack. After switching my Xen compute nodes to use the libxl

[libvirt] [PATCH 0/2] qemu: Cancel/forbid migration on I/O error

2013-06-11 Thread Peter Krempa
When qemu has paused itself due to a I/O error it's not safe to migrate it as it still might contain state in kernel buffers. These patches forbid and cancel ongoing migration if a I/O error happens. Peter Krempa (2): qemu: Cancel migration if guest encoutners I/O error while migrating qemu:

[libvirt] [PATCH 1/2] qemu: Cancel migration if guest encoutners I/O error while migrating

2013-06-11 Thread Peter Krempa
During a live migration the guest may receive a disk access I/O error. In this state the guest is unable to continue running on a remote host after migration as some state may be present in the kernel and not migrated. With this patch, the migration is canceled in such case so it can either

Re: [libvirt] [PATCH] qemu_migrate: Dispose listen address if set from config

2013-06-11 Thread Peter Krempa
On 06/10/13 19:38, Michal Privoznik wrote: https://bugzilla.redhat.com/show_bug.cgi?id=971485 As of d7f9d827531bc843b7c5aa9d3e8c08738a1de248 we copy the listen address from the qemu.conf config file in case none has been provided via XML. But later, when migrating, we should not include such

Re: [libvirt] [PATCH v3] selinux: assume 's0' if the range is empty

2013-06-11 Thread Peter Krempa
On 06/10/13 14:43, Ján Tomko wrote: This fixes a crash: https://bugzilla.redhat.com/show_bug.cgi?id=969878 --- v3: rewrote the patch to assume 's0' if the range is empty, instead of reporting an error v2: https://www.redhat.com/archives/libvir-list/2013-June/msg00082.html

Re: [libvirt] [PATCH v3] selinux: assume 's0' if the range is empty

2013-06-11 Thread Ján Tomko
On 06/11/2013 12:23 PM, Peter Krempa wrote: On 06/10/13 14:43, Ján Tomko wrote: This fixes a crash: https://bugzilla.redhat.com/show_bug.cgi?id=969878 --- v3: rewrote the patch to assume 's0' if the range is empty, instead of reporting an error v2:

Re: [libvirt] [PATCH] qemu: Fix memory leak in Prepare phase

2013-06-11 Thread Jiri Denemark
On Mon, Jun 10, 2013 at 15:29:03 +0200, Jano Tomko wrote: On 06/07/2013 02:14 PM, Jiri Denemark wrote: Avoid leaking virDomainDef if Prepare phase fails before it gets to qemuMigrationPrepareAny. --- src/qemu/qemu_driver.c| 12 src/qemu/qemu_migration.c | 29

[libvirt] AttributeError in virConnect.__del__ at program exit

2013-06-11 Thread Sandro Bonazzola
Hi, using vdsm python code, I've the following error at program exit that seems to be related to libvirt python code, something wrong in a destructor: Exception AttributeError: AttributeError(virConnect instance has no attribute 'domainEventCallbacks',) in bound method virConnect.__del__ of

Re: [libvirt] [PATCH 1/2] qemu: Cancel migration if guest encoutners I/O error while migrating

2013-06-11 Thread Michal Privoznik
On 11.06.2013 11:49, Peter Krempa wrote: During a live migration the guest may receive a disk access I/O error. I'd reword this: ... may receive an I/O error while accessing a disk. In this state the guest is unable to continue running on a remote host after migration as some state may be

Re: [libvirt] [PATCH 2/2] qemu: Forbid migration of machines with I/O errors

2013-06-11 Thread Michal Privoznik
On 11.06.2013 11:49, Peter Krempa wrote: Such machine can't be successuflly migrated unles the I/O error has recovered and might lead to data corruption. Forbid this kind of migration. --- src/qemu/qemu_migration.c | 10 ++ 1 file changed, 10 insertions(+) diff --git

Re: [libvirt] [PATCH 2/2] qemu: Forbid migration of machines with I/O errors

2013-06-11 Thread Peter Krempa
On 06/11/13 14:05, Michal Privoznik wrote: On 11.06.2013 11:49, Peter Krempa wrote: Such machine can't be successuflly migrated unles the I/O error has recovered and might lead to data corruption. Forbid this kind of migration. --- src/qemu/qemu_migration.c | 10 ++ 1 file changed,

Re: [libvirt] [PATCH 2/2] qemu: Forbid migration of machines with I/O errors

2013-06-11 Thread Michal Privoznik
On 11.06.2013 14:39, Peter Krempa wrote: On 06/11/13 14:05, Michal Privoznik wrote: On 11.06.2013 11:49, Peter Krempa wrote: Such machine can't be successuflly migrated unles the I/O error has recovered and might lead to data corruption. Forbid this kind of migration. ---

[libvirt] [PATCH] doc: Explicitly document migration limitations on errors

2013-06-11 Thread Peter Krempa
Migration of guests that are in error state for example because of I/O errors may not be possible and encountering of such errors may lead to cancellation of such migration. Document this behavior explicitly. --- src/libvirt.c | 20 1 file changed, 16 insertions(+), 4

[libvirt] [PATCH] qemu: allow restore with non-migratable XML input

2013-06-11 Thread Ján Tomko
Convert input XML to migratable before using it in qemuDomainSaveImageOpen. XML in the save image is migratable, i.e. doesn't contain implicit controllers. If these controllers were in a non-default order in the input XML, the ABI check would fail. Removing and re-adding these controllers fixes

Re: [libvirt] [PATCH 1/2] libxl: Report connect type as Xen

2013-06-11 Thread Jim Fehlig
Michal Privoznik wrote: On 10.06.2013 22:21, Jim Fehlig wrote: Currently, the libxl driver reports a connection type of xenlight. To be compatible with the legacy Xen driver, it should return Xen. Note: I noticed this while testing the libxl driver on OpenStack. After switching my Xen

Re: [libvirt] [PATCH] Fix F_DUPFD_CLOEXEC operation args

2013-06-11 Thread Paolo Bonzini
Il 03/05/2013 06:12, Daniel P. Berrange ha scritto: From: Daniel P. Berrange berra...@redhat.com The F_DUPFD_CLOEXEC operation with fcntl() expects a single int argument, specifying the minimum FD number for the newly dup'd file descriptor. We were not specifying that causing random stack

Re: [libvirt] [PATCH 0/2] qemu: Cancel/forbid migration on I/O error

2013-06-11 Thread Paolo Bonzini
Il 11/06/2013 05:49, Peter Krempa ha scritto: When qemu has paused itself due to a I/O error it's not safe to migrate it as it still might contain state in kernel buffers. These patches forbid and cancel ongoing migration if a I/O error happens. Peter Krempa (2): qemu: Cancel migration if

[libvirt] libvirtd 1.0.6 refuses to start

2013-06-11 Thread Franky Van Liedekerke
I just downloaded and compiled libvirtd on the same server as I did 1.0.4. Upon updating, libvirtd refuses to start, with this in the logfile (obfuscated the hostname by xxx's): 2013-06-11 13:43:49.154+: 3336: info : libvirt version: 1.0.6, package: 1.el6 (Unknown, 2013-06-06-14:57:07,

Re: [libvirt] [PATCH] Fix F_DUPFD_CLOEXEC operation args

2013-06-11 Thread Cole Robinson
On 06/11/2013 10:51 AM, Paolo Bonzini wrote: Il 03/05/2013 06:12, Daniel P. Berrange ha scritto: From: Daniel P. Berrange berra...@redhat.com The F_DUPFD_CLOEXEC operation with fcntl() expects a single int argument, specifying the minimum FD number for the newly dup'd file descriptor. We

Re: [libvirt] [PATCH] doc: Explicitly document migration limitations on errors

2013-06-11 Thread Michal Privoznik
On 11.06.2013 15:10, Peter Krempa wrote: Migration of guests that are in error state for example because of I/O errors may not be possible and encountering of such errors may lead to cancellation of such migration. Document this behavior explicitly. --- src/libvirt.c | 20

Re: [libvirt] [PATCH 1/2] libxl: Report connect type as Xen

2013-06-11 Thread Michal Privoznik
On 11.06.2013 16:12, Jim Fehlig wrote: Michal Privoznik wrote: On 10.06.2013 22:21, Jim Fehlig wrote: Currently, the libxl driver reports a connection type of xenlight. To be compatible with the legacy Xen driver, it should return Xen. Note: I noticed this while testing the libxl driver

Re: [libvirt] AttributeError in virConnect.__del__ at program exit

2013-06-11 Thread Cole Robinson
On 06/11/2013 07:58 AM, Sandro Bonazzola wrote: Hi, using vdsm python code, I've the following error at program exit that seems to be related to libvirt python code, something wrong in a destructor: Exception AttributeError: AttributeError(virConnect instance has no attribute

Re: [libvirt] [PATCH 1/2] libxl: Report connect type as Xen

2013-06-11 Thread Jim Fehlig
Michal Privoznik wrote: On 11.06.2013 16:12, Jim Fehlig wrote: Michal Privoznik wrote: On 10.06.2013 22:21, Jim Fehlig wrote: Currently, the libxl driver reports a connection type of xenlight. To be compatible with the legacy Xen driver, it should return Xen. Note: I

[libvirt] VirtualBox 4.2 Support

2013-06-11 Thread Dead Horse
Is there any active development to support VirtualBox 4.2 under libvirt? - DHC -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] libxl: calculating free pages in libvirt

2013-06-11 Thread Dario Faggioli
On gio, 2013-06-06 at 15:50 -0600, Jim Fehlig wrote: Dario Faggioli wrote: On gio, 2013-06-06 at 14:59 +0100, Ian Campbell wrote: This only really matters if libvirt wants to coexist on a host with toolstacks which use claim, otherwise outstanding pages will never be non-zero. I don't

Re: [libvirt] libxl: calculating free pages in libvirt

2013-06-11 Thread Konrad Rzeszutek Wilk
On Thu, Jun 06, 2013 at 03:53:13PM +0200, Dario Faggioli wrote: Hi Jim, As I told you in Dublin, I'm looking into libvirt a bit, with the main purpose of implementing the NUMA interface for the libxl driver. While at it I noticed that libxlNodeGetFreeMemory() uses the value contained in

[libvirt] Disabling bootmenu causes boot to skip over cdrom

2013-06-11 Thread Keno Fischer
Hello, I have domain with the following setup: - 1 VirtIO Disk - 1 VirtIO CDROM - 2 IDE CDROMs Now, depending on whether or not I have the bootmenu flag enabled, the boot sequence skips over booting from CDROM entirely (it does work if the bootmenu is enabled). This started happening after I

Re: [libvirt] Disabling bootmenu causes boot to skip over cdrom

2013-06-11 Thread Jiri Denemark
On Sun, Jun 09, 2013 at 16:20:17 -0400, Keno Fischer wrote: Hello, I have domain with the following setup: - 1 VirtIO Disk - 1 VirtIO CDROM - 2 IDE CDROMs Now, depending on whether or not I have the bootmenu flag enabled, the boot sequence skips over booting from CDROM entirely (it does

[libvirt] Can we allow users in 'qemu' group to skip polkit auth?

2013-06-11 Thread Cole Robinson
There's a bug report filed against Fedora libvirt requesting a polkit rule be installed that grants read/write libvirt access to all users in the 'qemu' group: https://bugzilla.redhat.com/show_bug.cgi?id=957300 I'm inclined to agree with the reporter, and time has shown that many users install

Re: [libvirt] Can we allow users in 'qemu' group to skip polkit auth?

2013-06-11 Thread Jiri Denemark
On Tue, Jun 11, 2013 at 17:11:12 -0400, Cole Robinson wrote: There's a bug report filed against Fedora libvirt requesting a polkit rule be installed that grants read/write libvirt access to all users in the 'qemu' group: https://bugzilla.redhat.com/show_bug.cgi?id=957300 I'm inclined to

[libvirt] [PATCH] configure: Remove unused brctl check

2013-06-11 Thread Cole Robinson
brctl isn't used anywhere AFAICT --- configure.ac | 2 -- 1 file changed, 2 deletions(-) diff --git a/configure.ac b/configure.ac index 5d1bc6b..59a3d6d 100644 --- a/configure.ac +++ b/configure.ac @@ -318,8 +318,6 @@ AC_PATH_PROG([DNSMASQ], [dnsmasq], [dnsmasq],

[libvirt] libvirt-php: 0.4.8 tag is missing

2013-06-11 Thread Doug Goldstein
The libvirt-php 0.4.8 tar is missing from the git repo [1]. Hopefully someone who created the tarball can create the tag as well and push it. [1] http://libvirt.org/git/?p=libvirt-php.git Thanks. -- Doug Goldstein -- libvir-list mailing list libvir-list@redhat.com

Re: [libvirt] [PATCH] configure: Remove unused brctl check

2013-06-11 Thread Laine Stump
On 06/11/2013 06:07 PM, Cole Robinson wrote: brctl isn't used anywhere AFAICT Too true. We switched to ioctls a long time ago. ACK. --- configure.ac | 2 -- 1 file changed, 2 deletions(-) diff --git a/configure.ac b/configure.ac index 5d1bc6b..59a3d6d 100644 --- a/configure.ac +++

Re: [libvirt] VirtualBox 4.2 Support

2013-06-11 Thread Dead Horse
Last activity I could find on this subject was here: https://www.redhat.com/archives/libvir-list/2013-May/msg01610.html Further activity here as well: https://www.redhat.com/archives/libvir-list/2013-June/msg00379.html I am willing to volunteer systems here to test libvirt+virtualbox 4.2