[libvirt] [libvirt-test-api][PATCH] Add a new test case for getMemoryParameters

2015-05-04 Thread Luyao Huang
Signed-off-by: Luyao Huang lhu...@redhat.com --- cases/test_connection.conf | 4 ++ repos/virconn/connection_getMemoryParameters.py | 61 + 2 files changed, 65 insertions(+) create mode 100644 repos/virconn/connection_getMemoryParameters.py diff

[libvirt] [PATCH] qemu: fix the vm deadlock when delete an unexist iothread

2015-05-04 Thread Luyao Huang
https://bugzilla.redhat.com/show_bug.cgi?id=1218145 Jump to endjob instead of cleanup to fix this deadlock. Signed-off-by: Luyao Huang lhu...@redhat.com --- src/qemu/qemu_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c

Re: [libvirt] [PATCH] qemu: fix the vm deadlock when delete an unexist iothread

2015-05-04 Thread Ján Tomko
s/delete an unexist/delete a nonexistent/ in the summary On Mon, May 04, 2015 at 05:36:16PM +0800, Luyao Huang wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1218145 Jump to endjob instead of cleanup to fix this deadlock. Signed-off-by: Luyao Huang lhu...@redhat.com ---

[libvirt] [PATCH] qemu: improve error message for domain stats API

2015-05-04 Thread Pavel Hrdina
Filtering specific list of domains is not supported yet. Print a better error message. old error message: error: unsupported flags (0x1) in function qemuConnectGetAllDomainStats new error message: error: using filters with list of domains is not implemented yet Resolves:

[libvirt] [PATCH] Bump version to 1.2.16 for new dev cycle

2015-05-04 Thread Jiri Denemark
Signed-off-by: Jiri Denemark jdene...@redhat.com --- Pushed as trivial. configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index d5fb71d..6122fa5 100644 --- a/configure.ac +++ b/configure.ac @@ -16,7 +16,7 @@ dnl You should have received

Re: [libvirt] [PATCH v5 0/9] qemu: Add quorum support to libvirt

2015-05-04 Thread Matthias Gatto
On Thu, Apr 23, 2015 at 2:41 PM, Matthias Gatto matthias.ga...@outscale.com wrote: The purpose of these patches is to introduce quorum for libvirt I've try to follow this proposal: http://www.redhat.com/archives/libvir-list/2014-May/msg00533.html This feature ask for 6 task: 1) Allow a

Re: [libvirt] [PATCH 5/6] conf: Add helper to convert list of virDomains to a list of virDomainObjs

2015-05-04 Thread Michal Privoznik
On 30.04.2015 14:44, Peter Krempa wrote: Add virDomainObjListConvert that will take a list of virDomains, apply filters and return a list of virDomainObjs. --- src/conf/domain_conf.c | 54 src/conf/domain_conf.h | 9

Re: [libvirt] [PATCH 0/6] Refactor critical section in virDomainListAllDomains and fix qemuConnectGetAllDomainStats

2015-05-04 Thread Michal Privoznik
On 30.04.2015 14:44, Peter Krempa wrote: Peter Krempa (6): util: Make the virDomainListFree helper more universal conf: Extract code to filter domain list into a separate function conf: Rename virDomainObjListFilter type to virDomainObjListACLFilter conf: Refactor domain list

Re: [libvirt] [PATCH] qemu: improve error message for domain stats API

2015-05-04 Thread Michal Privoznik
On 04.05.2015 12:44, Pavel Hrdina wrote: Filtering specific list of domains is not supported yet. Print a better error message. old error message: error: unsupported flags (0x1) in function qemuConnectGetAllDomainStats new error message: error: using filters with list of domains

Re: [libvirt] [PATCH] qemu: improve error message for domain stats API

2015-05-04 Thread Pavel Hrdina
On Mon, May 04, 2015 at 01:29:03PM +0200, Michal Privoznik wrote: On 04.05.2015 12:44, Pavel Hrdina wrote: Filtering specific list of domains is not supported yet. Print a better error message. old error message: error: unsupported flags (0x1) in function

[libvirt] [PATCH v2 0/2] Introduce GIC for aarch64

2015-05-04 Thread Michal Privoznik
diff to v1: - John's review worked in (hopefully) Michal Privoznik (2): Introduce GIC feature qemu: Implement GIC docs/formatdomain.html.in | 10 ++ docs/schemas/domaincommon.rng | 11 ++- src/conf/domain_conf.c

[libvirt] [PATCH v2 1/2] Introduce GIC feature

2015-05-04 Thread Michal Privoznik
Some platforms, like aarch64, don't have APIC but GIC. So there's no reason to have apic/ feature turned on. However, we are still missing gic/ feature. This commit introduces the feature to XML parser and formatter, adds documentation and updates RNG schema. Signed-off-by: Michal Privoznik

[libvirt] [PATCH 2/2] qemu: Forbid unsupported parameters for tunnelled migration

2015-05-04 Thread Jiri Denemark
Neither migrate URI nor lister address make any sense for tunnelled migration. https://bugzilla.redhat.com/show_bug.cgi?id=1066375 Signed-off-by: Jiri Denemark jdene...@redhat.com --- src/qemu/qemu_migration.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git

[libvirt] [PATCH v2 2/2] qemu: Implement GIC

2015-05-04 Thread Michal Privoznik
The only version that's supported in QEMU is version 2, currently. Fortunately, it is enabled by aarch64 automatically, so there's nothing for us that needs to be put onto command line. Signed-off-by: Michal Privoznik mpriv...@redhat.com --- src/qemu/qemu_command.c|

[libvirt] [PATCH 1/2] virsh: Don't check migrate parameters

2015-05-04 Thread Jiri Denemark
Just pass anything a user specified to the appropriate API. It's the API or libvirtd that should be responsible for checking its parameters. https://bugzilla.redhat.com/show_bug.cgi?id=1066375 Signed-off-by: Jiri Denemark jdene...@redhat.com --- tools/virsh-domain.c | 13 + 1 file

[libvirt] [PATCH 0/2] Forbid unsupported parameters for tunnelled migration

2015-05-04 Thread Jiri Denemark
Jiri Denemark (2): virsh: Don't check migrate parameters qemu: Forbid unsupported parameters for tunnelled migration src/qemu/qemu_migration.c | 16 +++- tools/virsh-domain.c | 13 + 2 files changed, 16 insertions(+), 13 deletions(-) -- 2.4.0 -- libvir-list

Re: [libvirt] [PATCH] test: remove s390 tests used only for testing usb and ide controllers

2015-05-04 Thread Boris Fiuczynski
Hi Laine, thanks for cleaning this up. The patch looks good to me. Reviewed-by: Boris Fiuczynski fiu...@linux.vnet.ibm.com On 04/30/2015 10:40 PM, Laine Stump wrote: Back in 2013, commit 877bc089 added in some tests that made sure no error was generated on a domain definition that had an

Re: [libvirt] [PATCH 0/2] Forbid unsupported parameters for tunnelled migration

2015-05-04 Thread Michal Privoznik
On 04.05.2015 14:10, Jiri Denemark wrote: Jiri Denemark (2): virsh: Don't check migrate parameters qemu: Forbid unsupported parameters for tunnelled migration src/qemu/qemu_migration.c | 16 +++- tools/virsh-domain.c | 13 + 2 files changed, 16

Re: [libvirt] [PATCH] test: remove s390 tests used only for testing usb and ide controllers

2015-05-04 Thread Michal Privoznik
On 30.04.2015 22:40, Laine Stump wrote: Back in 2013, commit 877bc089 added in some tests that made sure no error was generated on a domain definition that had an automatically added usb controller if that domain didn't have a PCI bus to attach the usb controller to. In particular, two

Re: [libvirt] [PATCH] test: remove s390 tests used only for testing usb and ide controllers

2015-05-04 Thread Ján Tomko
On Thu, Apr 30, 2015 at 04:40:46PM -0400, Laine Stump wrote: Back in 2013, commit 877bc089 added in some tests that made sure no error was generated on a domain definition that had an automatically added usb controller if that domain didn't have a PCI bus to attach the usb controller to. In

Re: [libvirt] [PATCH] tests: Fix grammar in comments.

2015-05-04 Thread Michal Privoznik
On 28.04.2015 17:43, Andrea Bolognani wrote: Replace all occurrences of stream write to differences to with stream to write differences to. --- tests/testutils.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/testutils.c b/tests/testutils.c index

Re: [libvirt] [PATCH 0/2] Forbid unsupported parameters for tunnelled migration

2015-05-04 Thread Jiri Denemark
On Mon, May 04, 2015 at 14:40:04 +0200, Michal Privoznik wrote: On 04.05.2015 14:10, Jiri Denemark wrote: Jiri Denemark (2): virsh: Don't check migrate parameters qemu: Forbid unsupported parameters for tunnelled migration src/qemu/qemu_migration.c | 16 +++-

Re: [libvirt] [PATCH 7/7] storage: Check for duplicate host for incoming pool def

2015-05-04 Thread Ján Tomko
On Thu, Apr 30, 2015 at 10:16:55AM -0400, John Ferlan wrote: On 04/30/2015 05:59 AM, Ján Tomko wrote: On Wed, Apr 29, 2015 at 12:37:38PM -0400, John Ferlan wrote: virStoragePoolSourceFindDuplicate for VIR_STORAGE_POOL_ISCSI calls virStoragePoolSourceFindDuplicateDevices which does the dual

Re: [libvirt] [PATCH] tests: Fix grammar in comments.

2015-05-04 Thread Andrea Bolognani
On Monday, May 4, 2015 at 3:04:24 PM, Michal Privoznik wrote: ACKed pushed. Thanks! -- Andrea Bolognani abolo...@redhat.com -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] test: remove s390 tests used only for testing usb and ide controllers

2015-05-04 Thread Stefan Zimmermann
Fine with me. Reviewed-by: Stefan Zimmermann s...@linux.vnet.ibm.com On Thu, 2015-04-30 at 16:40 -0400, Laine Stump wrote: Back in 2013, commit 877bc089 added in some tests that made sure no error was generated on a domain definition that had an automatically added usb controller if that

Re: [libvirt] [PATCH 2/2] qemu: Forbid unsupported parameters for tunnelled migration

2015-05-04 Thread Eric Blake
On 05/04/2015 06:10 AM, Jiri Denemark wrote: Neither migrate URI nor lister address make any sense for tunnelled s/lister/listener/ ? migration. https://bugzilla.redhat.com/show_bug.cgi?id=1066375 Signed-off-by: Jiri Denemark jdene...@redhat.com --- src/qemu/qemu_migration.c | 16

Re: [libvirt] [PATCH v2 0/2] Introduce GIC for aarch64

2015-05-04 Thread John Ferlan
On 05/04/2015 08:09 AM, Michal Privoznik wrote: diff to v1: - John's review worked in (hopefully) Michal Privoznik (2): Introduce GIC feature qemu: Implement GIC docs/formatdomain.html.in | 10 ++ docs/schemas/domaincommon.rng |

Re: [libvirt] [PATCH v3] polkit: Allow password-less access for 'libvirt' group

2015-05-04 Thread Cole Robinson
On 04/30/2015 09:14 AM, Cole Robinson wrote: Many users, who admin their own machines, want to be able to access system libvirtd via tools like virt-manager without having to enter a root password. Just google 'virt-manager without password' and you'll find many hits. I've read at least 5 blog

Re: [libvirt] [PATCH 6/6] storage: fs: Only force directory permissions if required

2015-05-04 Thread Cole Robinson
On 04/28/2015 09:54 PM, Cole Robinson wrote: On 04/28/2015 09:36 AM, Peter Krempa wrote: On Mon, Apr 27, 2015 at 16:48:44 -0400, Cole Robinson wrote: Only set directory permissions at pool build time, if: - User explicitly requested a mode via the XML - The directory needs to be created -

Re: [libvirt] [PATCH] test: remove s390 tests used only for testing usb and ide controllers

2015-05-04 Thread Laine Stump
On 05/04/2015 08:59 AM, Ján Tomko wrote: On Thu, Apr 30, 2015 at 04:40:46PM -0400, Laine Stump wrote: Back in 2013, commit 877bc089 added in some tests that made sure no error was generated on a domain definition that had an automatically added usb controller if that domain didn't have a PCI

Re: [libvirt] [PATCH v5 4/9] virstoragefile: Always use virStorageSourceSetBackingStore to set backing store

2015-05-04 Thread John Ferlan
On 04/23/2015 08:41 AM, Matthias Gatto wrote: Replace the parts of the code where a backing store is set manually with virStorageSourceSetBackingStore Signed-off-by: Matthias Gatto matthias.ga...@outscale.com Signed-off-by: John Ferlan jfer...@redhat.com --- src/conf/domain_conf.c

[libvirt] ANNOUNCE: virt-manager 1.2.0 released

2015-05-04 Thread Cole Robinson
I'm happy to announce the release of virt-manager 1.2.0! virt-manager is a desktop application for managing KVM, Xen, and LXC virtualization via libvirt. The release can be downloaded from: http://virt-manager.org/download/ The direct download links are:

[libvirt] [PATCH 0/2] Storage : Fixes for cloning raw volumes

2015-05-04 Thread Prerna Saxena
From: Prerna Saxena pre...@linux.vnet.ibm.com Date: Mon, 4 May 2015 12:00:46 -0700 This series has some long-overdue fixes for copying of raw storage volumes with libvirt. Prerna Saxena (2): Storage : Suppress metadata refresh for volumes being built. Storage : Fix cloning of raw, sparse

[libvirt] [PATCH 1/2] Storage: Suppress metadata refresh for volumes being built.

2015-05-04 Thread Prerna Saxena
Libvirt periodically calls 'stat' on all volumes in a storage pool, to update fields such as 'target.allocation'. The operation doesnt make sense for a volume which is curently being allocated. Also, the 'target.allocation' sub-field is taken into account while copying a raw image. To suppress

[libvirt] [PATCH 2/2] Storage : Fix cloning of raw, sparse volumes.

2015-05-04 Thread Prerna Saxena
When virsh vol-clone is attempted on a raw file where capacity allocation, the resulting cloned volume has a size that matches the virtual-size of the parent; in place of matching its actual, disk size. This patch fixes the cloned disk to have same _allocated_size_ as the parent file from which

[libvirt] add a rbd disk by virsh attach-device , the operation fails

2015-05-04 Thread 饶俊明
Hello: add a rbd disk by virsh attach-device ,the operation fails; libvirt version: libvirt-0.10.2-46.el6.x86_64 operated virsh pool-info, pool-edit, attach-device domain_id /tmp/rbddisk.xml command: virsh # pool-info 0a081d22-070b-3f58-a883-78e29012b052 Name:

[libvirt] [PATCH] qemu: Properly rename persistent def after migration

2015-05-04 Thread Jiri Denemark
When migrating a domain while changing its name and using VIR_MIGRATE_PERSIST_DEST flag, libvirt would fail to properly change the name in the persistent definition. The inconsistency results in weird behavior when dumping domain XML, destroying the domain, restarting libvirtd and likely in

Re: [libvirt] [PATCH 7/7] storage: Check for duplicate host for incoming pool def

2015-05-04 Thread John Ferlan
On 05/04/2015 09:44 AM, Ján Tomko wrote: On Thu, Apr 30, 2015 at 10:16:55AM -0400, John Ferlan wrote: On 04/30/2015 05:59 AM, Ján Tomko wrote: On Wed, Apr 29, 2015 at 12:37:38PM -0400, John Ferlan wrote: virStoragePoolSourceFindDuplicate for VIR_STORAGE_POOL_ISCSI calls

[libvirt] [PATCH] libxl: support soundhw for hvm domains

2015-05-04 Thread Jim Fehlig
The xend driver and the parsing/formating code in src/xenconfig have long supported soundhw. Add support in the libxl driver too. Signed-off-by: Jim Fehlig jfeh...@suse.com --- This patch continues the tradition of silently ignoring unsupported config in the libxl driver. But it is an