[libvirt] Error

2014-08-05 Thread ahmad jan
I want to configure Libvirt-1.2.7 on ubuntu and found the following Error "Could not find linxml2 anywhere" guide please-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] valgrind report on active commit

2014-08-05 Thread Eric Blake
I'm still investigating, but posting this trace now in case someone else spots the bug. I created a chain three deep (base.img <- mid.img <- top.img), then ran blockcopy --finish, blockcommit --active --shallow --pivot, and another blockcommit --active --shallow --pivot. The last attempt to pivot

Re: [libvirt] [PATCH v2 0/2] add support for --config in setmaxmem command

2014-08-05 Thread chenhanx...@cn.fujitsu.com
ping > -Original Message- > From: libvir-list-boun...@redhat.com [mailto:libvir-list-boun...@redhat.com] > On Behalf Of Chen Hanxiao > Sent: Thursday, July 31, 2014 11:41 AM > To: libvir-list@redhat.com > Subject: [libvirt] [PATCH v2 0/2] add support for --config in setmaxmem > command >

Re: [libvirt] [python PATCH] Bump version to 1.2.8 for new dev cycle

2014-08-05 Thread Daniel Veillard
On Tue, Aug 05, 2014 at 09:03:03AM -0600, Eric Blake wrote: > On 08/05/2014 12:26 AM, Jiri Denemark wrote: > > On Mon, Aug 04, 2014 at 11:07:00 -0600, Eric Blake wrote: > >> Signed-off-by: Eric Blake > >> --- > >> > >> Pushing under the trivial rule. > >> > >> setup.py | 2 +- > >> 1 file changed

[libvirt] [PATCH v4 2/3] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread John Ferlan
Introduce a new structure to handle an iSCSI host device based on the existing virDomainHostdevSubsysSCSI by adding a "protocol='iscsi'" to the element. The hostdev structure mimics the existing element for an iSCSI device (network) device. New XML is:

Re: [libvirt] libvirt external disk-only snapshot will pause the VM?

2014-08-05 Thread Eric Blake
On 08/05/2014 03:54 PM, Yuanzhen Gu wrote: > Hello folks, > > If I give command to take external disk-only live snapshot, such as > # virsh snapshot-create-as vm1 snapshot_vm1 --disk-only > > will the guest OS be paused or suspend while the snapshot being taken? The guest will temporarily pause,

[libvirt] libvirt external disk-only snapshot will pause the VM?

2014-08-05 Thread Yuanzhen Gu
Hello folks, If I give command to take external disk-only live snapshot, such as # virsh snapshot-create-as vm1 snapshot_vm1 --disk-only will the guest OS be paused or suspend while the snapshot being taken? Thanks very much! Best, Yuanzhen -- libvir-list mailing list libvir-list@redhat.com ht

Re: [libvirt] [PATCH v2 8/8] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread John Ferlan
On 08/05/2014 04:24 PM, Eric Blake wrote: > On 08/05/2014 01:59 PM, John Ferlan wrote: > >> Both are 'scsi' adapters - existing is 'scsi_host' and the new one is >> 'iscsi'. >> I'm unsure about putting the 'optional' attribute name really does. Not to >> confuse things further, but the next ste

Re: [libvirt] [PATCH v2 8/8] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread Eric Blake
On 08/05/2014 01:59 PM, John Ferlan wrote: > Both are 'scsi' adapters - existing is 'scsi_host' and the new one is 'iscsi'. > I'm unsure about putting the 'optional' attribute name really does. Not to > confuse things further, but the next step in the evolution of this is to add > a fiber channel

[libvirt] [PATCH v3 2/3] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread John Ferlan
Introduce a new structure to handle an iSCSI host device based on the existing virDomainHostdevSubsysSCSI by adding a "protocol='iscsi'" to the element. The hostdev structure mimics the existing element for an iSCSI device (network) device. New XML is:

[libvirt] [PATCH v3 1/3] domain_conf: Common routine to handle network storage host xml def

2014-08-05 Thread John Ferlan
In preparation for hostdev support for iSCSI and a virStorageNetHostDefPtr, split out the network disk storage parsing of the 'host' element into a separate routine. Signed-off-by: John Ferlan --- src/conf/domain_conf.c | 137 +++-- 1 file changed, 76

[libvirt] [PATCH v3 3/3] qemu: Remove extraneous space in function prototypes

2014-08-05 Thread John Ferlan
During review of the iSCSI hostdev series, eblake noted that the prototypes shouldn't have the extranenous space between the "*" and the function name: http://www.redhat.com/archives/libvir-list/2014-July/msg01227.html Since it was more invasive than 1 or 2 lines - I said I'd send a patch coverin

[libvirt] [PATCH v3 0/3] Add iSCSI hostdev pass-through device

2014-08-05 Thread John Ferlan
Patches 1 & 2 are related to: Update to http://www.redhat.com/archives/libvir-list/2014-July/msg01098.html Patch 1/3 is already ACK'd, but felt it was easier to push together. Patch 2/3 change since last: - Fix typos as called in review of 8/8 - Adjust the rng file to - Change the order o

Re: [libvirt] [PATCH v2 8/8] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread John Ferlan
On 08/05/2014 11:20 AM, Eric Blake wrote: > On 08/05/2014 06:00 AM, John Ferlan wrote: > >> >> Maybe perhaps sleep helped me figure this out... Is the following what >> you had in mind? (and it passes make check too) >> >> >> >> scsi >> >> >> >> >>

Re: [libvirt] [PATCH RFC] LXC: add HOME environment variable

2014-08-05 Thread Richard Weinberger
On Fri, Jul 25, 2014 at 8:39 AM, Chen Hanxiao wrote: > We lacked of HOME environment variable, > set 'HOME=/' as default. > > Signed-off-by: Chen Hanxiao > --- > src/lxc/lxc_container.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/lxc/lxc_container.c b/src/lxc/lxc_container.c > i

Re: [libvirt] [PATCH] LXC: Fix virLXCControllerSetupDevPTS() wrt user namespaces

2014-08-05 Thread Richard Weinberger
Am 29.07.2014 05:45, schrieb chenhanx...@cn.fujitsu.com: > > >> -Original Message- >> From: libvir-list-boun...@redhat.com [mailto:libvir-list-boun...@redhat.com] >> On Behalf Of Richard Weinberger >> Sent: Tuesday, July 29, 2014 4:59 AM >> To: libvir-list@redhat.com >> Cc: Richard Weinbe

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Wei Liu
On Tue, Aug 05, 2014 at 07:45:48PM +0100, Ian Campbell wrote: [...] > > > > Sure. I think having an API that emulates a power reset button would be > > > > a nice addition to libxl's domain operations. The destroy/start > > > > approach incurs a small bit of overhead, which would be avoided with

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Wei Liu
On Tue, Aug 05, 2014 at 04:30:47PM +0100, Ian Campbell wrote: > On Tue, 2014-08-05 at 09:10 -0600, Jim Fehlig wrote: > > Ian Campbell wrote: > > > On Tue, 2014-08-05 at 08:06 -0600, Jim Fehlig wrote: > > > > > >> Ian Campbell wrote: > > >> > > >>> On Tue, 2014-08-05 at 10:55 +0200, Olaf Her

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Ian Campbell
On Tue, 2014-08-05 at 17:12 +0100, Wei Liu wrote: > On Tue, Aug 05, 2014 at 04:30:47PM +0100, Ian Campbell wrote: > > On Tue, 2014-08-05 at 09:10 -0600, Jim Fehlig wrote: > > > Ian Campbell wrote: > > > > On Tue, 2014-08-05 at 08:06 -0600, Jim Fehlig wrote: > > > > > > > >> Ian Campbell wrote: >

[libvirt] [PATCH] lxc: Report container init process failures.

2014-08-05 Thread Jarek Dziedzic
This change improves the reporting of container stop reason. Before this change if the init process of lxc container has crashed or returned non zero status, the domain state listeners would be notified that a domain has stopped due to normal shutdown. This maps to VIR_DOMAIN_EVENT_STOPPED_SHUTDOW

Re: [libvirt] qemu driver fails to load on Xen after be0782e1

2014-08-05 Thread Jim Fehlig
Jim Fehlig wrote: > Until recently, the qemu driver has always succeeded in loading even on > a system booted to Xen. Commit be0782e1 broke this, hence libvirtd no > longer loads > > xendom0 # /usr/sbin/libvirtd -l > 2014-08-04 22:27:31.586+: 35859: info : libvirt version: 1.2.8 > 2014-08-04 2

Re: [libvirt] handle different version of QEMU

2014-08-05 Thread Eric Blake
On 08/05/2014 10:14 AM, Matthias Gatto wrote: > Hello, > > I'm implementing the "new" QEMU option of > block_set_io_throttle(@bps_max, @bps_rd_max...), but i have a problem: > The old version of QEMU(before 1.7) does not handle it, so I'd like to > know what function should i use to know the versi

Re: [libvirt] [PATCH 1/2] domain_conf: refactor virDomainDeviceAddressIsValid to boolean

2014-08-05 Thread Eric Blake
On 08/05/2014 10:18 AM, Pavel Hrdina wrote: > Signed-off-by: Pavel Hrdina > --- > src/conf/device_conf.c | 4 ++-- > src/conf/device_conf.h | 2 +- > src/conf/domain_conf.c | 16 > src/conf/domain_conf.h | 4 ++-- > 4 files changed, 13 insertions(+), 13 deletions(-) ACK. --

[libvirt] [RFC PATCH 1/5] doc: schema: Add basic documentation for the ivshmem support

2014-08-05 Thread Maxime Leroy
This patch documents XML elements used for support of ivshmem devices. About ivshmem, please see the following documentation: http://git.qemu.org/?p=qemu.git;a=blob;f=docs/specs/ivshmem_device_spec.txt (Ivshmem documentation for qemu will be updated soon: https://lists.nongnu.org/archive/html/qemu

[libvirt] [RFC PATCH 5/5] tests: Add tests for ivshmem device handling

2014-08-05 Thread Maxime Leroy
Add XML parsing and qemu command line tests for the ivshmem device support. Signed-off-by: Maxime Leroy --- tests/qemuxml2argvdata/qemuxml2argv-ivshmem.args | 10 +++ tests/qemuxml2argvdata/qemuxml2argv-ivshmem.xml | 36 tests/qemuxml2argvtest.c

[libvirt] [RFC PATCH 3/5] qemu: Add cap flag QEMU_CAPS_IVSHMEM

2014-08-05 Thread Maxime Leroy
Ivshmem is supported by QEMU since 0.13 release. Signed-off-by: Maxime Leroy --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilities.h | 1 + tests/qemucapabilitiesdata/caps_1.2.2-1.caps | 1 + tests/qemucapabilitiesdata/caps_1.3.1-1.caps | 1 +

[libvirt] [RFC PATCH 4/5] qemu: Build command line for ivshmem device

2014-08-05 Thread Maxime Leroy
This patch implements support for the ivshmem device in QEMU. Example from this xml: 32 The following QEMU line is built: -device ivshmem,size=32m,vectors=32,chardev=charivshmem0,msi=on, ioeventfd=on,role=master -chardev socket,path=/tmp/socket-ivshme

[libvirt] [RFC PATCH 2/5] conf: Parse and format ivshmem device XML

2014-08-05 Thread Maxime Leroy
This patch adds configuration support for the ivshmem device as described in the schema in the previous patch. Signed-off-by: Maxime Leroy --- src/conf/domain_conf.c | 234 ++- src/conf/domain_conf.h | 40 src/libvirt_private.syms | 4 +

[libvirt] [RFC PATCH 0/5] ivshmem support

2014-08-05 Thread Maxime Leroy
The following patches are an implementation proposal for the ivshmem device support in libvirt. Any feedback is welcome. Note: SELinux is not supported (need to be done for the next patchset version) Maxime Leroy (5): doc: schema: Add basic documentation for the ivshmem support conf: P

Re: [libvirt] [PATCH 2/2] device_conf: PCI address pci_0000_00_00_0 is also valid

2014-08-05 Thread Eric Blake
On 08/05/2014 10:18 AM, Pavel Hrdina wrote: > All the variables used for pci addr in virDevicePCIAddress are > unsigned and therefore in this validation function they never cannot > be less than zero. This check will fail only for the address above, > but that address is valid so we can safely retu

Re: [libvirt] [PATCH] blockjob: correctly report active commit for job info

2014-08-05 Thread Eric Blake
On 08/05/2014 08:57 AM, Adam Litke wrote: > On 05/08/14 08:51 -0600, Eric Blake wrote: >> Commit 232a31b munged job info to report 'active commit' instead of >> 'commit' when generating events, but forgot to also munge the polling >> variant of the command. >> >> * src/qemu/qemu_driver.c (qemuDomai

Re: [libvirt] [PATCH v2 8/8] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread Eric Blake
On 08/05/2014 06:00 AM, John Ferlan wrote: > > Maybe perhaps sleep helped me figure this out... Is the following what > you had in mind? (and it passes make check too) > > > > scsi > > > > > filtered > unfiltered > > >

Re: [libvirt] [PATCH] qemu: reword caps-related error

2014-08-05 Thread Eric Blake
On 08/05/2014 03:03 AM, Martin Kletzander wrote: > Signed-off-by: Martin Kletzander > --- > src/qemu/qemu_driver.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > ACK > diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c > index 8eba8e8..a4a96af 100644 > --- a/src/qemu/qe

Re: [libvirt] [RFC][scale] new API for querying domains stats

2014-08-05 Thread Eric Blake
On 08/04/2014 11:46 PM, Li Wei wrote: >> >> How about the following API: >> >> int virConnectGetAllBlockStats(virConnectPtr conn, >> virDomainPtr domain, >> virDomainBlockBulkStatsPtr *stats, >> unsigned int fla

Re: [libvirt] [python PATCH] Bump version to 1.2.8 for new dev cycle

2014-08-05 Thread Eric Blake
On 08/05/2014 12:26 AM, Jiri Denemark wrote: > On Mon, Aug 04, 2014 at 11:07:00 -0600, Eric Blake wrote: >> Signed-off-by: Eric Blake >> --- >> >> Pushing under the trivial rule. >> >> setup.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/setup.py b/setup.py >> inde

[libvirt] [PATCH 1/2] domain_conf: refactor virDomainDeviceAddressIsValid to boolean

2014-08-05 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina --- src/conf/device_conf.c | 4 ++-- src/conf/device_conf.h | 2 +- src/conf/domain_conf.c | 16 src/conf/domain_conf.h | 4 ++-- 4 files changed, 13 insertions(+), 13 deletions(-) diff --git a/src/conf/device_conf.c b/src/conf/device_conf.c index

[libvirt] [PATCH 0/2] Fix PCI address validation

2014-08-05 Thread Pavel Hrdina
Pavel Hrdina (2): domain_conf: refactor virDomainDeviceAddressIsValid to boolean device_conf: PCI address pci__00_00_0 is also valid src/conf/device_conf.c | 7 --- src/conf/device_conf.h | 2 +- src/conf/domain_conf.c | 16 src/conf/domain_conf.h | 4 ++-- 4 files

[libvirt] [PATCH 2/2] device_conf: PCI address pci_0000_00_00_0 is also valid

2014-08-05 Thread Pavel Hrdina
All the variables used for pci addr in virDevicePCIAddress are unsigned and therefore in this validation function they never cannot be less than zero. This check will fail only for the address above, but that address is valid so we can safely return true instead. Resolves: https://bugzilla.redhat.

[libvirt] handle different version of QEMU

2014-08-05 Thread Matthias Gatto
Hello, I'm implementing the "new" QEMU option of block_set_io_throttle(@bps_max, @bps_rd_max...), but i have a problem: The old version of QEMU(before 1.7) does not handle it, so I'd like to know what function should i use to know the version of QEMU, and what's the proper way to handle this probl

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Ian Campbell
On Tue, 2014-08-05 at 09:10 -0600, Jim Fehlig wrote: > Ian Campbell wrote: > > On Tue, 2014-08-05 at 08:06 -0600, Jim Fehlig wrote: > > > >> Ian Campbell wrote: > >> > >>> On Tue, 2014-08-05 at 10:55 +0200, Olaf Hering wrote: > >>> > >>> > The original report was that 'Reset'

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Jim Fehlig
Ian Campbell wrote: > On Tue, 2014-08-05 at 08:06 -0600, Jim Fehlig wrote: > >> Ian Campbell wrote: >> >>> On Tue, 2014-08-05 at 10:55 +0200, Olaf Hering wrote: >>> >>> The original report was that 'Reset' does not work from GUI, like virt-manager or virsh. I think the e

Re: [libvirt] [PATCH] blockjob: correctly report active commit for job info

2014-08-05 Thread Adam Litke
On 05/08/14 08:51 -0600, Eric Blake wrote: Commit 232a31b munged job info to report 'active commit' instead of 'commit' when generating events, but forgot to also munge the polling variant of the command. * src/qemu/qemu_driver.c (qemuDomainBlockJobImpl): Adjust type as needed. Signed-off-by: E

[libvirt] [PATCH] blockjob: correctly report active commit for job info

2014-08-05 Thread Eric Blake
Commit 232a31b munged job info to report 'active commit' instead of 'commit' when generating events, but forgot to also munge the polling variant of the command. * src/qemu/qemu_driver.c (qemuDomainBlockJobImpl): Adjust type as needed. Signed-off-by: Eric Blake --- src/qemu/qemu_driver.c | 3 ++

[libvirt] [PATCH 0/4] Wrong allocation size displayed for NFS volumes

2014-08-05 Thread John Ferlan
https://bugzilla.redhat.com/show_bug.cgi?id=1077068 The virsh vol-info (and virsh domblkinfo) commands return an incorrect size if the target device is an NFS volume. These patches will add a check for the target being NFS and rather than calculate the size based on "st_blocks" and "DEV_BSIZE" (5

[libvirt] [PATCH 3/4] vol-info: Check for NFS FS type

2014-08-05 Thread John Ferlan
https://bugzilla.redhat.com/show_bug.cgi?id=1077068 Check for the NFS FS type being true for a "local" stat of the file to force usage of the 'st_size' value rather than calculating the size using the 'st_blocks' and 'st_blksize'. As described in the stat(2) man page: "Use of the st_blocks and s

[libvirt] [PATCH 2/4] virfile: Introduce virFileIsNFSFSType

2014-08-05 Thread John Ferlan
Use the virFileGetFSFtype() in order to compare the returned f_type for the NFS super magic number Signed-off-by: John Ferlan --- src/libvirt_private.syms | 1 + src/util/virfile.c | 19 +++ src/util/virfile.h | 1 + 3 files changed, 21 insertions(+) diff --git a/s

[libvirt] [PATCH 4/4] domblkinfo: Check for NFS FS Type

2014-08-05 Thread John Ferlan
Similar to the vol-info code, check if the target device for the blkinfo fetch is an NFS volume and return the st_size value rather than calculating the size based on st_blksize and DEV_BSIZE. Prior to the change a 40M target device would display: Capacity: 41943040 Allocation: 126976 P

[libvirt] [PATCH 1/4] virfile: Split virFileGetFSFtype

2014-08-05 Thread John Ferlan
Split out the portion of virFileGetFSFtype that makes the statfs call in order to return the "f_type" field Signed-off-by: John Ferlan --- src/util/virfile.c | 34 -- 1 file changed, 24 insertions(+), 10 deletions(-) diff --git a/src/util/virfile.c b/src/util/vir

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Ian Campbell
On Tue, 2014-08-05 at 08:06 -0600, Jim Fehlig wrote: > Ian Campbell wrote: > > On Tue, 2014-08-05 at 10:55 +0200, Olaf Hering wrote: > > > >> The original report was that 'Reset' does not work from GUI, like > >> virt-manager or virsh. I think the expected outcome is like pushing the > >> reset

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Jim Fehlig
Ian Campbell wrote: > On Tue, 2014-08-05 at 10:55 +0200, Olaf Hering wrote: > >> The original report was that 'Reset' does not work from GUI, like >> virt-manager or virsh. I think the expected outcome is like pushing the >> reset button on a physical board. Xen doesnt do it that way, no idea >>

[libvirt] [PATCH] lxc: Report container init process failures.

2014-08-05 Thread Jarek Dziedzic
This change improves the reporting of container stop reason. Before this change if the init process of lxc container has crashed or returned non zero status, the domain state listeners would be notified that a domain has stopped due to normal shutdown. This maps to VIR_DOMAIN_EVENT_STOPPED_SHUTDOW

Re: [libvirt] [libvirt-glib] [PATCH 3/4] GVirDomainSnapshot: Add _set_config

2014-08-05 Thread Timm Bäder
On 05.08, Christophe Fergeau wrote: > On Sat, Aug 02, 2014 at 10:41:48AM +0200, m...@baedert.org wrote: > > From: Timm Bäder > > > > ... which is basically analogous to gvir_domain_set_config > > --- > > libvirt-gobject/libvirt-gobject-domain-snapshot.c | 58 > > +++ > > lib

Re: [libvirt] help

2014-08-05 Thread Michal Privoznik
[Dropping libvirt-announce list] On 05.08.2014 11:47, Himanshu Sharma wrote: Dear Team, Greetings!! I'm testing libvirt with VMware ESXi. I'm able to connect to VMware ESXi with libvirt driver but not able to connect it through "Virtual Machine manager" GUI also not able to run virt-clone com

Re: [libvirt] [libvirt-glib] [PATCH 3/4] GVirDomainSnapshot: Add _set_config

2014-08-05 Thread Christophe Fergeau
On Sat, Aug 02, 2014 at 10:41:48AM +0200, m...@baedert.org wrote: > From: Timm Bäder > > ... which is basically analogous to gvir_domain_set_config > --- > libvirt-gobject/libvirt-gobject-domain-snapshot.c | 58 > +++ > libvirt-gobject/libvirt-gobject-domain-snapshot.h | 5

Re: [libvirt] [PATCH v2 8/8] hostdev: Add iSCSI hostdev XML

2014-08-05 Thread John Ferlan
On 08/04/2014 02:21 PM, Eric Blake wrote: >> +++ b/docs/schemas/domaincommon.rng >> @@ -3563,13 +3563,47 @@ >> >> >> >> + >> + >> + > > Does diskAuth work for all configurations, or only for the new iscsi > configuration? By putting it here, you are allow

Re: [libvirt] [libvirt-glib] [PATCH 0/4] Add more snapshot API

2014-08-05 Thread Christophe Fergeau
On Sat, Aug 02, 2014 at 10:41:45AM +0200, m...@baedert.org wrote: > From: Timm Bäder > The implementation of gvir_domain_snapshot_set_config does not allow > renaming of the snapshot as that will just create a new snapshot in > any case (should that by documented?). But it looks like the same > is

Re: [libvirt] [libvirt-glib] [PATCH 4/4] GVirDomain: Add _has_current_snapshot

2014-08-05 Thread Christophe Fergeau
On Sat, Aug 02, 2014 at 10:41:49AM +0200, m...@baedert.org wrote: > From: Timm Bäder > > ... which uses virDomainHasCurrentSnapshot to determine if the given > domain has a current snapshot or not. > --- > libvirt-gobject/libvirt-gobject-domain.c | 30 ++ > libvirt-go

Re: [libvirt] [libvirt-glib] [PATCH 2/4] GVirDomainSnapshot: Add _revert_to

2014-08-05 Thread Christophe Fergeau
ACK. On Sat, Aug 02, 2014 at 10:41:47AM +0200, m...@baedert.org wrote: > From: Timm Bäder > > Add a way to revert a domain to one of its snapshots. > --- > libvirt-gobject/libvirt-gobject-domain-snapshot.c | 33 > +++ > libvirt-gobject/libvirt-gobject-domain-snapshot.h | 18

Re: [libvirt] [libvirt-glib] [PATCH 0/4] Add more snapshot API

2014-08-05 Thread Christophe Fergeau
Hey, On Sat, Aug 02, 2014 at 10:41:45AM +0200, m...@baedert.org wrote: > From: Timm Bäder > > The following 4 patches add some more snapshot API to libvirt-glib. > > I wasn't sure where to put _revert_to really, the libvirt API sounds > like it belongs into GVirDomain (i.e. gvir_domain_revert_t

Re: [libvirt] [libvirt-glib] [PATCH 1/4] GVirDomainSnapshot: Add _is_current

2014-08-05 Thread Christophe Fergeau
Hey, On Sat, Aug 02, 2014 at 10:41:46AM +0200, m...@baedert.org wrote: > From: Timm Bäder > > Add a way to determine if the given GVirDomainSnapshot is the current > snapshot of its GVirDomain. > --- > libvirt-gobject/libvirt-gobject-domain-snapshot.c | 29 > +++ > libvirt-

[libvirt] help

2014-08-05 Thread Himanshu Sharma
Dear Team, Greetings!! I'm testing libvirt with VMware ESXi. I'm able to connect to VMware ESXi with libvirt driver but not able to connect it through "Virtual Machine manager" GUI also not able to run virt-clone command inturn to clone ESXi VM. Can you please guide me how to do so? -Origi

[libvirt] help

2014-08-05 Thread Himanshu Sharma
Dear Team, Greetings!! I'm testing libvirt with VMware ESXi. I'm able to connect to VMware ESXi with libvirt driver but not able to connect it through "Virtual Machine manager" GUI also not able to run virt-clone command inturn to clone ESXi VM. Can you please guide me how to do so? -Origi

Re: [libvirt] Welcome to the "libvir-list" mailing list

2014-08-05 Thread Himanshu Sharma
Dear Team, Greetings!! I'm testing libvirt with VMware ESXi. I'm able to connect to VMware ESXi with libvirt driver but not able to connect it through "Virtual Machine manager" GUI also not able to run virt-clone command inturn to clone ESXi VM. Can you please guide me how to do so? -Origi

Re: [libvirt] [Discussion] How do we think about time out mechanism?

2014-08-05 Thread James
On 2014/8/5 17:13, Martin Kletzander wrote: > On Tue, Aug 05, 2014 at 03:15:18PM +0800, James wrote: >> In fact, to deal with this kind of situation, we add some timeout codes in >> libvirtd, during remote_dispatch process. >> The mechanism is like this: >> 1. when we call an API, we start a thre

Re: [libvirt] [Discussion] How do we think about time out mechanism?

2014-08-05 Thread Martin Kletzander
On Tue, Aug 05, 2014 at 03:15:18PM +0800, James wrote: In fact, to deal with this kind of situation, we add some timeout codes in libvirtd, during remote_dispatch process. The mechanism is like this: 1. when we call an API, we start a thread to do the timer, when time out, the timer set a timeo

[libvirt] [PATCH] qemu: reword caps-related error

2014-08-05 Thread Martin Kletzander
Signed-off-by: Martin Kletzander --- src/qemu/qemu_driver.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 8eba8e8..a4a96af 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3456,7 +3456,9 @@ static i

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Ian Campbell
On Tue, 2014-08-05 at 10:55 +0200, Olaf Hering wrote: > On Tue, Aug 05, Ian Campbell wrote: > > > On Tue, 2014-08-05 at 10:41 +0200, Olaf Hering wrote: > > > On Tue, Aug 05, Ian Campbell wrote: > > > > > > > case LIBXL_TRIGGER_RESET: > > > > rc = xc_domain_send_trigger(ctx->xch, domid

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Olaf Hering
On Tue, Aug 05, Ian Campbell wrote: > On Tue, 2014-08-05 at 10:41 +0200, Olaf Hering wrote: > > On Tue, Aug 05, Ian Campbell wrote: > > > > > case LIBXL_TRIGGER_RESET: > > > rc = xc_domain_send_trigger(ctx->xch, domid, > > > XEN_DOMCTL_SENDTRIGGER_R

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Ian Campbell
On Tue, 2014-08-05 at 10:41 +0200, Olaf Hering wrote: > On Tue, Aug 05, Ian Campbell wrote: > > > case LIBXL_TRIGGER_RESET: > > rc = xc_domain_send_trigger(ctx->xch, domid, > > XEN_DOMCTL_SENDTRIGGER_RESET, vcpu > > break; > > > > Do you mea

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Olaf Hering
On Tue, Aug 05, Ian Campbell wrote: > case LIBXL_TRIGGER_RESET: > rc = xc_domain_send_trigger(ctx->xch, domid, > XEN_DOMCTL_SENDTRIGGER_RESET, vcpu > break; > > Do you mean to say that it is broken or perhaps that it doesn't match > the requ

Re: [libvirt] [Xen-devel] [PATCH] libxl: support domainReset

2014-08-05 Thread Ian Campbell
On Mon, 2014-08-04 at 20:09 -0600, Jim Fehlig wrote: > Currently, libxl_send_trigger() does not implement the LIBXL_TRIGGER_RESET > option, There's a case in the switch within libxl_send_trigger which at least purports to do so: int libxl_send_trigger(libxl_ctx *ctx, uint32_t domid,

Re: [libvirt] [PATCH] Fix typo error attribute name vlan-id should be vlanid in docs

2014-08-05 Thread Ján Tomko
On 08/05/2014 07:25 AM, Jianwei Hu wrote: > [root@localhost ~]# virsh nwfilter-dumpxml myself > > 7192ef51-cd50-4f14-ad7b-fa5c69ea19e3 > > > > > > [root@localhost ~]# ebtables -t nat -L > Bridge table: nat > > ... > -p 802_1Q -d 0:11:22:33:44:55 --vlan-id 44 -j ACCEPT > --- > docs

Re: [libvirt] [PATCH v2] storage: ZFS support

2014-08-05 Thread Roman Bogorodskiy
Roman Bogorodskiy wrote: > Changes from the initial version: > - Update docs/schemas and docs/storage.html.in with ZFS backend >information > - Drop StartPool and StopPool that does nothing and therefore >not needed > - Fix memory leak in 'tokens' variable > - Fill volume key before

Re: [libvirt] [Discussion] How do we think about time out mechanism?

2014-08-05 Thread James
On 2014/8/4 19:59, Martin Kletzander wrote: > On Sat, Jul 26, 2014 at 03:47:09PM +0800, James wrote: >> On 2014/7/25 18:07, Martin Kletzander wrote: >> >>> On Fri, Jul 25, 2014 at 04:45:55PM +0800, James wrote: There's a kind of situation that when libvirtd's under a lot of pressure, ju

Re: [libvirt] [python PATCH] maint: document use of libvirt's run script

2014-08-05 Thread Martin Kletzander
On Mon, Aug 04, 2014 at 11:10:45AM -0600, Eric Blake wrote: Ever since libvirt commit 78c09530, it's easier to just use the run script. * README: Mention run script. Signed-off-by: Eric Blake --- README | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/README b/README i