Re: [libvirt] [PATCH 1/1] qemu: add support for multiple gluster hosts

2015-10-06 Thread Deepak C Shetty
On 10/06/2015 06:12 PM, Peter Krempa wrote: On Tue, Oct 06, 2015 at 15:35:05 +0530, Deepak C Shetty wrote: On 10/06/2015 02:53 PM, Peter Krempa wrote: On Tue, Oct 06, 2015 at 10:38:23 +0530, Deepak C Shetty wrote: On 10/05/2015 07:33 PM, Peter Krempa wrote: On Mon, Oct 05, 2015 at 18:59:54

Re: [libvirt] [PATCH 1/1] qemu: add support for multiple gluster hosts

2015-10-06 Thread Deepak C Shetty
On 10/06/2015 02:53 PM, Peter Krempa wrote: On Tue, Oct 06, 2015 at 10:38:23 +0530, Deepak C Shetty wrote: On 10/05/2015 07:33 PM, Peter Krempa wrote: On Mon, Oct 05, 2015 at 18:59:54 +0530, Deepak C Shetty wrote: [Note: Formatting of this mail is broken. Please configure your mail client to

Re: [libvirt] [PATCH 1/1] qemu: add support for multiple gluster hosts

2015-10-05 Thread Deepak C Shetty
On 10/05/2015 07:33 PM, Peter Krempa wrote: On Mon, Oct 05, 2015 at 18:59:54 +0530, Deepak C Shetty wrote: [Note: Formatting of this mail is broken. Please configure your mail client to properly flow text in the plaintext mode.] Sorry about that. I configured plain text format for libvir

Re: [libvirt] [PATCH 1/1] qemu: add support for multiple gluster hosts

2015-10-05 Thread Deepak C Shetty
ot;qcow2" } ,if=none,id=drive-virtio-disk0,cache=none if the number of hosts supplied is only one, then we use existing URI format for backward compatibility and if number of hosts is greater than one we switch to the new JSON format this patch requires qemu latest patch https://lists.gnu

Re: [libvirt] virDomainAttachDevice error during disk hotplug

2013-08-19 Thread Deepak C Shetty
On 08/19/2013 03:13 PM, Daniel P. Berrange wrote: On Fri, Aug 16, 2013 at 03:48:25PM -0500, Doug Goldstein wrote: On Fri, Aug 16, 2013 at 8:30 AM, Daniel P. Berrange wrote: On Fri, Aug 16, 2013 at 06:51:21PM +0530, Deepak C Shetty wrote: On 08/10/2013 06:20 PM, Deepak C Shetty wrote: 2013

Re: [libvirt] [SOLVED] virDomainAttachDevice error during disk hotplug

2013-08-17 Thread Deepak C Shetty
On 08/17/2013 02:18 AM, Doug Goldstein wrote: On Fri, Aug 16, 2013 at 8:30 AM, Daniel P. Berrange mailto:berra...@redhat.com>> wrote: On Fri, Aug 16, 2013 at 06:51:21PM +0530, Deepak C Shetty wrote: > On 08/10/2013 06:20 PM, Deepak C Shetty wrote: > >2013-08-10 11

Re: [libvirt] virDomainAttachDevice error during disk hotplug

2013-08-16 Thread Deepak C Shetty
On 08/16/2013 07:51 PM, Daniel P. Berrange wrote: On Sat, Aug 10, 2013 at 05:38:20PM +0530, Deepak C Shetty wrote: Hi All, I am trying to do a hotplug of a disk from VDSM (which uses libvirt to get things done). I hit the below error "operation failed: open disk image file failed&qu

Re: [libvirt] virDomainAttachDevice error during disk hotplug

2013-08-16 Thread Deepak C Shetty
On 08/16/2013 06:59 PM, Daniel P. Berrange wrote: On Sat, Aug 10, 2013 at 05:38:20PM +0530, Deepak C Shetty wrote: Hi All, I am trying to do a hotplug of a disk from VDSM (which uses libvirt to get things done). I hit the below error "operation failed: open disk image file failed&qu

Re: [libvirt] virDomainAttachDevice error during disk hotplug

2013-08-16 Thread Deepak C Shetty
On 08/10/2013 06:20 PM, Deepak C Shetty wrote: 2013-08-10 11:19:41.772+: 1091: debug : qemuMonitorIOProcess:345 : QEMU_MONITOR_IO_PROCESS: mon=0x7f92f0007830 buf={"return": "could not open disk image /var/run/vdsm/017d-1278-4bfb-8129-62bded257399: Invalid argument\r\n&q

Re: [libvirt] virDomainAttachDevice error during disk hotplug

2013-08-10 Thread Deepak C Shetty
On 08/10/2013 05:38 PM, Deepak C Shetty wrote: Hi All, I am trying to do a hotplug of a disk from VDSM (which uses libvirt to get things done). I hit the below error "operation failed: open disk image file failed" comign from virDomainAttachDevice. Some background: 1) Th

[libvirt] virDomainAttachDevice error during disk hotplug

2013-08-10 Thread Deepak C Shetty
Hi All, I am trying to do a hotplug of a disk from VDSM (which uses libvirt to get things done). I hit the below error "operation failed: open disk image file failed" comign from virDomainAttachDevice. Some background: 1) The qemu-img create cmd I use (inside vdsm) to create the file bei

Re: [libvirt] Issues with qemu-nbd over AF_UNIX and virDomainAttachDevice

2013-05-27 Thread Deepak C Shetty
On 05/27/2013 04:41 PM, Paolo Bonzini wrote: Il 27/05/2013 12:46, Deepak C Shetty ha scritto: On 05/27/2013 02:00 PM, Paolo Bonzini wrote: Il 26/05/2013 15:54, Deepak C Shetty ha scritto: 3) In nbdxml, i tried removing port="".. it didn't give any error with regard

Re: [libvirt] Issues with qemu-nbd over AF_UNIX and virDomainAttachDevice

2013-05-27 Thread Deepak C Shetty
On 05/27/2013 02:00 PM, Paolo Bonzini wrote: Il 26/05/2013 15:54, Deepak C Shetty ha scritto: 3) In nbdxml, i tried removing port="".. it didn't give any error with regards to port beign mandatory.. which si good.. but the attach device still gives the same error as above Is

Re: [libvirt] Issues with qemu-nbd over AF_UNIX and virDomainAttachDevice

2013-05-26 Thread Deepak C Shetty
On 05/25/2013 02:04 AM, Paolo Bonzini wrote: Il 24/05/2013 10:01, Deepak C Shetty ha scritto: Il 24/05/2013 06:59, Deepak C Shetty ha scritto: What version of libvirt is this? It should work starting from 1.0.4. Ah, thats what I seem to have missed. I was using the distro provided libvirt

Re: [libvirt] Issues with qemu-nbd over AF_UNIX and virDomainAttachDevice

2013-05-24 Thread Deepak C Shetty
On 05/24/2013 12:38 PM, Paolo Bonzini wrote: Il 24/05/2013 06:59, Deepak C Shetty ha scritto: What version of libvirt is this? It should work starting from 1.0.4. Ah, thats what I seem to have missed. I was using the distro provided libvirt.. which seems to be old (0.9)! I will compile

[libvirt] Issues with qemu-nbd over AF_UNIX and virDomainAttachDevice

2013-05-23 Thread Deepak C Shetty
Hi List, Facing issues trying to use virDomainAttachDevice API of libvirt from python. Here is what I am trying to do 1) Setup qemu-nbd to export a qcow2 disk image over unix socket on my localhost qemu-nbd -t -k /tmp/mysock2 /home/dpkshetty/work/img/iscsi_disk The above blocks. and i a

[libvirt] RFE: Ability to freeze/thaw guest using libvirt/virsh APIs

2013-04-30 Thread Deepak C Shetty
I am sending this mail based on the below IRC discussion. Apr 23 18:55:02 Hi all, is there a way to just do a fsfreeze (and un-freeze) of VM usign virsh. I don't see any option today Apr 23 18:55:52 there is option to take snapshot (which in turn would do freeze/un-freeze), bu

Re: [libvirt] [PATCH v2 1/2] Qemu/Gluster: Add Gluster protocol as supported network disk formats.

2012-10-04 Thread Deepak C Shetty
I chose to check for only ':' to decide if its a IPv6 addr because it doesnt make sense to be partial towards '.' What if someone specifies a host name like 12:12;12,12 or 23:23,23,23 ? A '.' in an IPv6 addr is as bad as any other invalid char. '.' is valid in IPv6 addr. But yes, ':

Re: [libvirt] [PATCH v2 1/2] Qemu/Gluster: Add Gluster protocol as supported network disk formats.

2012-10-04 Thread Deepak C Shetty
+/* if IPv6 addr, use square brackets to enclose it */ +virBufferAsprintf(opt, "[%s]:%s", disk->hosts->name, disk->hosts->port); +} else { +virBufferAsprintf(opt, "%s:%s", disk->hosts->name, disk->hosts->port); +} +

Re: [libvirt] [PATCH v2 1/2] Qemu/Gluster: Add Gluster protocol as supported network disk formats.

2012-10-04 Thread Deepak C Shetty
On 10/04/2012 07:01 PM, Harsh Prateek Bora wrote: Qemu accepts gluster protocol as supported storage backend beside others. This patch allows users to specify disks on gluster backends like this: Note: In the element above, transport is an

Re: [libvirt] [Gluster-devel] [RFC PATCH v1 0/2] Qemu/Gluster support in Libvirt

2012-08-29 Thread Deepak C Shetty
On 08/30/2012 08:27 AM, Yin Yin wrote: > Hi, Harsh: > I make some break in glusterd , and can gdb the qemu-kvm forked from > libvirtd. > > break in glusterd: > > (gdb) i b > Num Type Disp Enb Address What > 1 breakpoint keep y 0x7f903ef1a0a0 in server_getspec at > glusterd-handshake.c:122 > 2 b

Re: [libvirt] [RFC PATCH v1 0/2] Qemu/Gluster support in Libvirt

2012-08-24 Thread Deepak C Shetty
On 08/24/2012 12:22 PM, Harsh Bora wrote: On 08/24/2012 12:05 PM, Daniel Veillard wrote: On Thu, Aug 23, 2012 at 04:31:50PM +0530, Harsh Prateek Bora wrote: This patchset provides support for Gluster protocol based network disks. It is based on the proposed gluster support in Qemu on qemu-deve

Re: [libvirt] Linker error (virStorageFileResize) while compiling latest libvirt git source

2012-02-01 Thread Deepak C Shetty
Thanks Laine, that worked. On 02/02/2012 02:46 AM, Laine Stump wrote: On 02/01/2012 12:00 PM, Deepak C Shetty wrote: On 02/01/2012 10:05 PM, Laine Stump wrote: On 02/01/2012 11:01 AM, Deepak C Shetty wrote: Requesting zheeshan to look, as this API was added recently by him, as i see on the

Re: [libvirt] Linker error (virStorageFileResize) while compiling latest libvirt git source

2012-02-01 Thread Deepak C Shetty
On 02/01/2012 10:05 PM, Laine Stump wrote: On 02/01/2012 11:01 AM, Deepak C Shetty wrote: Requesting zheeshan to look, as this API was added recently by him, as i see on the list... jdenemar just pushed a fix. Do you mean the 'bogus warning about the uninit variable' fix, it doe

Re: [libvirt] Linker error (virStorageFileResize) while compiling latest libvirt git source

2012-02-01 Thread Deepak C Shetty
Requesting zheeshan to look, as this API was added recently by him, as i see on the list... On 02/01/2012 08:25 PM, Deepak C Shetty wrote: make all-am make[3]: Entering directory `/home/dpkshetty/work/libvirt/libvirt/daemon' GENqemu_dispatch.h GENremote_dispatch.h

[libvirt] Linker error while compiling latest libvirt git source

2012-02-01 Thread Deepak C Shetty
make all-am make[3]: Entering directory `/home/dpkshetty/work/libvirt/libvirt/daemon' GENqemu_dispatch.h GENremote_dispatch.h CC libvirtd-remote_protocol.o CC libvirtd-qemu_protocol.o CC libvirtd-remote.o CC libvirtd-libvirtd.o CC libvirtd-stream.o In fil

Re: [libvirt] Unable to compile libvirt git src... python-config error

2012-02-01 Thread Deepak C Shetty
On 02/01/2012 07:56 PM, Jiri Denemark wrote: On Wed, Feb 01, 2012 at 19:49:43 +0530, Deepak C Shetty wrote: Hi All, I am getting this error, while trying to compile latest git src. ./autogen.sh --prefix=$HOME/usr --enable-compile-warnings=error running bootstrap... ./bootstrap: Error

[libvirt] Unable to compile libvirt git src... python-config error

2012-02-01 Thread Deepak C Shetty
Hi All, I am getting this error, while trying to compile latest git src. ./autogen.sh --prefix=$HOME/usr --enable-compile-warnings=error running bootstrap... ./bootstrap: Error: 'python-config' not found ./bootstrap: Please install the prerequisite programs Failed to bootstrap, please invest

Re: [libvirt] [PATCH v4] Add new attribute wrpolicy to element

2012-01-17 Thread Deepak C Shetty
On 01/18/2012 04:07 AM, Eric Blake wrote: You had lots of trailing whitespace, caught by 'make syntax-check'. +++ b/tests/qemuxml2argvdata/qemuxml2argv-fs9p.args @@ -4,10 +4,10 @@ unix:/tmp/test-monitor,server,nowait -no-acpi -boot c -hda \ /dev/HostVG/QEMUGuest1 -fsdev local,security_model=

Re: [libvirt] [PATCH v3] Add new attribute wrpolicy to element

2012-01-17 Thread Deepak C Shetty
Hello Eric, Posted v4 of this patch with your comments implemented at ... https://www.redhat.com/archives/libvir-list/2012-January/msg00633.html thanx, deepak -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH v4] Add new attribute wrpolicy to element

2012-01-17 Thread Deepak C Shetty
pages in the host page cache as part of the guest file write operation. Usage: Currently this only works with type='mount' for the QEMU/KVM driver. Signed-off-by: Deepak C Shetty --- v2: - added writeout as a qemu cap - cosmetic changes in comments - move

Re: [libvirt] [PATCH v3] Add new attribute wrpolicy to element

2012-01-17 Thread Deepak C Shetty
Please add a new pair of test files to tests/qemuxml2argvdata (both .xml and .args file) to validate the RNG, as well as listing that file name in tests/qemuxml2argvtest.c to validate the qemu command line generation (I find it easiest to start from an existing similar .xml file, and just add in

Re: [libvirt] [PATCH v3] Add new attribute wrpolicy to element

2012-01-12 Thread Deepak C Shetty
On 01/12/2012 03:49 AM, Eric Blake wrote: I'm still trying to understand the qemu semantics: your choices are an explicit writeout=immediate (with explicit requests to the host) or nothing (host writes may be delayed). So, you are saying the a missing attribute lets qemu choose, and an explicit

Re: [libvirt] [PATCH v2] Do not generate security_model when fs driver is anything but 'path'

2012-01-11 Thread Deepak C Shetty
On 01/12/2012 02:18 AM, Eric Blake wrote: On 01/10/2012 06:08 AM, Daniel P. Berrange wrote: On Tue, Jan 10, 2012 at 06:23:31PM +0530, Deepak C Shetty wrote: On 01/10/2012 04:17 PM, Daniel P. Berrange wrote: On Tue, Jan 10, 2012 at 03:35:26PM +0530, Deepak C Shetty wrote: --- v2: - removed

Re: [libvirt] [PATCH v2] Add new attribute writeout to element

2012-01-10 Thread Deepak C Shetty
Hi All, I posted v3 of this patch (considering the inputs from this thread) at ... https://www.redhat.com/archives/libvir-list/2012-January/msg00330.html Comments welcome. thanx, deepak -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH v3] Add new attribute wrpolicy to element

2012-01-10 Thread Deepak C Shetty
pages in the host page cache as part of the guest file write operation. Usage: Currently this only works with type='mount' for the QEMU/KVM driver. Signed-off-by: Deepak C Shetty --- v2: - added writeout as a qemu cap - cosmetic changes in comments - move

Re: [libvirt] [PATCH v2] Do not generate security_model when fs driver is anything but 'path'

2012-01-10 Thread Deepak C Shetty
On 01/10/2012 04:17 PM, Daniel P. Berrange wrote: On Tue, Jan 10, 2012 at 03:35:26PM +0530, Deepak C Shetty wrote: --- v2: - removed xml accessmode changes as suggested by dan. - every fs driver having a default accessmode always, retained. src/qemu/qemu_command.c | 23

Re: [libvirt] [PATCH 2/2] Do not generate security_model when fs driver is anything but 'path'

2012-01-10 Thread Deepak C Shetty
Posted v2 of the patch here... https://www.redhat.com/archives/libvir-list/2012-January/msg00311.html thanx, deepak On 01/03/2012 05:15 PM, Daniel P. Berrange wrote: On Mon, Dec 26, 2011 at 02:49:55PM +0530, Deepak C Shetty wrote: On 12/22/2011 11:00 PM, Daniel P. Berrange wrote: On Thu, Dec

[libvirt] [PATCH v2] Do not generate security_model when fs driver is anything but 'path'

2012-01-10 Thread Deepak C Shetty
ch ensures that the qemu cmdline is correctly generated by taking into account the fs driver type. Signed-off-by: Deepak C Shetty --- v2: - removed xml accessmode changes as suggested by dan. - every fs driver having a default accessmode always, retained. src/qemu/qemu_comman

Re: [libvirt] [PATCH 1/2] Set default fs driver when not provided in input xml

2012-01-02 Thread Deepak C Shetty
Now that I think again, I am not sure if the below is really needed ? accordingly I think that is correct. On 12/21/2011 11:17 AM, Deepak C Shetty wrote: While adding support for fs driver sub-element in virt-manager (vmm) it throws a python exception/error when one tries to list fs device

Re: [libvirt] [PATCH v2] Add new attribute writeout to element

2011-12-26 Thread Deepak C Shetty
On 12/22/2011 11:07 PM, Daniel P. Berrange wrote: On Thu, Dec 22, 2011 at 05:25:49PM +0100, Paolo Bonzini wrote: On 12/22/2011 03:20 PM, Eric Blake wrote: Usage: I still don't like the proposed XML. We need to get consensus on where to put it, with my proposal being:

Re: [libvirt] [PATCH 2/2] Do not generate security_model when fs driver is anything but 'path'

2011-12-26 Thread Deepak C Shetty
On 12/22/2011 11:00 PM, Daniel P. Berrange wrote: On Thu, Dec 22, 2011 at 10:24:57AM +0530, Deepak C Shetty wrote: On 12/21/2011 08:11 PM, Daniel P. Berrange wrote: On Wed, Dec 21, 2011 at 11:17:17AM +0530, Deepak C Shetty wrote: QEMU does not support security_model for anything but 'pat

Re: [libvirt] [PATCH v2] Add new attribute writeout to element

2011-12-22 Thread Deepak C Shetty
On 12/22/2011 10:52 PM, Deepak C Shetty wrote: On 12/22/2011 07:50 PM, Eric Blake wrote: On 12/22/2011 03:10 AM, Deepak C Shetty wrote: I still don't like the proposed XML. We need to get consensus on where to put it, with my proposal being: Currrently, the way its implement

Re: [libvirt] [PATCH v2] Add new attribute writeout to element

2011-12-22 Thread Deepak C Shetty
On 12/22/2011 07:50 PM, Eric Blake wrote: On 12/22/2011 03:10 AM, Deepak C Shetty wrote: I still don't like the proposed XML. We need to get consensus on where to put it, with my proposal being: Currrently, the way its implemented in qemu's hw/9pfs is that the w

Re: [libvirt] [PATCH] qemu: Support copy on read for disk

2011-12-22 Thread Deepak C Shetty
On 12/22/2011 03:20 PM, Osier Yang wrote: Upstream QEMU supports it since commit fb0490f69. --- docs/formatdomain.html.in |9 + docs/schemas/domaincommon.rng | 11 +++ src/conf/domain_conf.c| 24 +++- src/conf/domain_conf.h| 1

Re: [libvirt] [PATCH] Add new attribute writeout to element

2011-12-22 Thread Deepak C Shetty
Posted v2 @ https://www.redhat.com/archives/libvir-list/2011-December/msg00965.html -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH v2] Add new attribute writeout to element

2011-12-22 Thread Deepak C Shetty
pages in the host page cache as part of the guest file write operation. Usage: Currently this only works with type='mount' for the QEMU/KVM driver. Signed-off-by: Deepak C Shetty --- v2: - added writeout as a qemu cap - cosmetic changes in comments - move

Re: [libvirt] [PATCH] Add new attribute writeout to element

2011-12-22 Thread Deepak C Shetty
On 12/22/2011 12:55 PM, Osier Yang wrote: On 2011年12月22日 14:54, Deepak C Shetty wrote: This introduces new attribute writeout with only supported value as immediate. This will be an optional attribute with no defaults. This helps specify whether to skip the host page cache. When writeout is

[libvirt] [PATCH] Add new attribute writeout to element

2011-12-21 Thread Deepak C Shetty
pages in the host page cache as part of the guest file write operation. Usage: Currently this only works with type='mount' for the QEMU/KVM driver. Signed-off-by: Deepak C Shetty --- docs/formatdomain.html.in |8 +++- docs/schemas/domaincommon.rng |5 +

Re: [libvirt] [PATCH 2/2] Do not generate security_model when fs driver is anything but 'path'

2011-12-21 Thread Deepak C Shetty
On 12/21/2011 08:11 PM, Daniel P. Berrange wrote: On Wed, Dec 21, 2011 at 11:17:17AM +0530, Deepak C Shetty wrote: QEMU does not support security_model for anything but 'path' fs driver type. Currently in libvirt, when security_model ( accessmode attribute) is not specified it auto-ge

[libvirt] [PATCH 2/2] Do not generate security_model when fs driver is anything but 'path'

2011-12-20 Thread Deepak C Shetty
_model the input xml passed to libvirt does not contain accessmode attribute, but libvirt generates it as part of the virDomainDefine flow, which should only be done if fs driver is of type 'path', else not. This patch fixes these issues. Signed-off-by: Deepak C Shetty --- src/conf/d

[libvirt] [PATCH 1/2] Set default fs driver when not provided in input xml

2011-12-20 Thread Deepak C Shetty
, which is the case when user selects default driver in vmm. As part of vmm's add new fs device flow, libvirt's DefineXML gets called which does not add the sub-element in case of default fs driver selected in vmm. This patch fixes that. Signed-off-by: Deepak C Shetty --- src/conf/dom

[libvirt] [PATCH 0/2] VirtFS fixes needed for supporting fs driver in VMM

2011-12-20 Thread Deepak C Shetty
These are the libvirt changes needed to ensure that the virt-manager changes done for supporting fs driver work well. --- Deepak C Shetty (2): Set default fs driver when not provided in input xml Do not generate security_model when fs driver is anything but 'path'