Re: [libvirt] [PATCH 2/2] virsh: add new --details option to vol-list

2010-06-17 Thread Justin Clift
On 06/16/2010 10:28 PM, Justin Clift wrote: NameType Capacity Allocation Path CentOS-5.5-x86_64-bin-DVD-1of2.iso file 4.09 GB4.10 GB /var/lib/libvirt/images/CentOS-5.5-x86_64

Re: [libvirt] 1. Domain Destroy , 2. Domain boot on a different host and 3. Domain start time

2010-06-17 Thread IKI-サガル バルウェ
Thanks Justin, Thanks for a fantastic explaination!! This clears up so many of my doubts. I will try out all this and other ways I have in mind while referring to the libvirt docs. Thanks a ton. Thanks and Regards Sagar Barve - Original Message - From: "Justin Clift" To: "IKI-サガル 

[libvirt] [PATCH] nwfilter: extensions of docs with advanced filtering topics

2010-06-17 Thread Stefan Berger
As requested, here a couple of paragraphs about the recently added statematch attribute and some advanced (and tricky) traffic filtering topics. Signed-off-by: Stefan Berger --- docs/formatnwfilter.html.in | 117 1 file changed, 117 insertions(

[libvirt] [PATCH 2/3] Add optional model attribute to the controller element

2010-06-17 Thread Matthias Bolte
This is a step towards controller support for the ESX driver. --- docs/schemas/domain.rng |9 + src/conf/domain_conf.c | 34 ++ src/conf/domain_conf.h | 11 +++ src/qemu/qemu_driver.c |1 + 4 files changed, 55 insertions(+), 0 deletion

[libvirt] [PATCH 0/3] Add controller support for the ESX driver

2010-06-17 Thread Matthias Bolte
This series adds controller support for the ESX driver. Also adds required additions like the optional model attribute for the controller element to get rid of the disk driver name abuse and support for wide SCSI bus addresses. Matthias -- libvir-list mailing list libvir-list@redhat.com https://w

[libvirt] [PATCH 1/3] Add wide SCSI bus disk address generation support

2010-06-17 Thread Matthias Bolte
The domain XML parsing code autogenerates disk address and controller elements when they are not explicitly specified. The code assumes a narrow SCSI bus (7 units per bus). ESX uses a wide SCSI bus (16 units per bus). This is a step towards controller support for the ESX driver. --- src/conf/capa

Re: [libvirt] [PATCH] virsh: add --uuid option to vol-pool

2010-06-17 Thread Eric Blake
On 06/17/2010 02:22 PM, Jiri Denemark wrote: >> Adds an optional switch, --uuid, for telling the virsh vol-pool command >> to return the pool UUID rather than pool name. >> --- >> >> Just added for flexibility. And for BZ 604929. >> >> tools/virsh.c | 13 +++-- >> tools/virsh.pod |

Re: [libvirt] [PATCH] virsh: add --uuid option to vol-pool

2010-06-17 Thread Jiri Denemark
> Adds an optional switch, --uuid, for telling the virsh vol-pool command > to return the pool UUID rather than pool name. > --- > > Just added for flexibility. > > tools/virsh.c | 13 +++-- > tools/virsh.pod |8 +--- > 2 files changed, 16 insertions(+), 5 deletions(-) ACK

Re: [libvirt] [PATCHv3] qemu: reduce file padding requirements

2010-06-17 Thread Eric Blake
On 06/17/2010 01:32 AM, Laine Stump wrote: > On 06/10/2010 03:31 PM, Eric Blake wrote: >> Followup to https://bugzilla.redhat.com/show_bug.cgi?id=599091, >> commit 20206a4b, to reduce disk waste in padding. >> >> * src/qemu/qemu_monitor.h (QEMU_MONITOR_MIGRATE_TO_FILE_BS): Drop >> back to 512. >> (

Re: [libvirt] [PATCH v2] nwfilter: add XML attribute to control match target

2010-06-17 Thread Stefan Berger
On 06/17/2010 02:23 PM, Eric Blake wrote: On 06/11/2010 12:02 PM, Stefan Berger wrote: This patch adds an optional XML attribute to a nwfilter rule to give the user control over whether the rule is supposed to be using the state match or not. A rule may now look like shown in the XML below w

Re: [libvirt] [PATCH v2] nwfilter: add XML attribute to control match target

2010-06-17 Thread Eric Blake
On 06/11/2010 12:02 PM, Stefan Berger wrote: > > This patch adds an optional XML attribute to a nwfilter rule to give the > user control over whether the rule is supposed to be using the state > match or not. A rule may now look like shown in the XML below with the > statematch attribute either ha

Re: [libvirt] [PATCH v2] nwfilter: add XML attribute to control match target

2010-06-17 Thread Stefan Berger
On 06/17/2010 01:26 PM, Daniel P. Berrange wrote: On Fri, Jun 11, 2010 at 02:02:11PM -0400, Stefan Berger wrote: This patch adds an optional XML attribute to a nwfilter rule to give the user control over whether the rule is supposed to be using the state match or not. A rule may now look lik

Re: [libvirt] [PATCHv2] virsh: ensure persistence and autostart are shown for dominfo and pool-info

2010-06-17 Thread Eric Blake
On 06/17/2010 01:57 AM, Justin Clift wrote: > This patch adds the persistence status (yes/no) to the output of the virsh > dominfo and pool-info commands. This patch also adds the autostart status > to the output of the virsh pool-info command. > > Red Hat BZ for this: > > https://bugzilla.red

Re: [libvirt] [PATCHv3] phyp: adding support for IVM

2010-06-17 Thread Eric Blake
On 06/16/2010 06:29 PM, Eduardo Otubo wrote: > On 06/16/2010 07:04 PM, Eric Blake wrote: >> From: Eduardo Otubo >> >> Use virBuffer* API to contionally keep the portion of the command >> line specific to HMC, so that IVM can work. > > Tested here, working on both HMC and IVM. Thanks! Thanks for t

Re: [libvirt] [PATCH v2] nwfilter: add XML attribute to control match target

2010-06-17 Thread Daniel P. Berrange
On Fri, Jun 11, 2010 at 02:02:11PM -0400, Stefan Berger wrote: > > This patch adds an optional XML attribute to a nwfilter rule to give the > user control over whether the rule is supposed to be using the state > match or not. A rule may now look like shown in the XML below with the > statematc

[libvirt] [PATCH 0/4] Change per-connection hashes to be indexed by UUIDs

2010-06-17 Thread Jiri Denemark
While working at the area, I also made few other cleanups... Jiri Denemark (4): Index hashes by UUID instead of name Remove unnecessary check for non-NULL uuid Do not free static buffer with UUID Misc cleanups src/datatypes.c | 83 --

[libvirt] [PATCH 4/4] Misc cleanups

2010-06-17 Thread Jiri Denemark
- Fix documentation for virGetStorageVol: it has 'key' argument instead of 'uuid'. - Remove TODO comment from virReleaseStorageVol: we use volume key as an identifier instead of UUID. - Print human-readable UUID string in debug message in virReleaseSecret. --- src/datatypes.c |6 +++--- 1

[libvirt] [PATCH 2/4] Remove unnecessary check for non-NULL uuid

2010-06-17 Thread Jiri Denemark
The first thing we do in all these functions is to check uuid != NULL and fail if it isn't. --- src/datatypes.c | 12 1 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/datatypes.c b/src/datatypes.c index 20b2d74..8750406 100644 --- a/src/datatypes.c +++ b/src/dataty

[libvirt] [PATCH 3/4] Do not free static buffer with UUID

2010-06-17 Thread Jiri Denemark
As anywhere else, uuid is defined as a fixed size array inside _virSecret structure; we shouldn't try to free it. --- src/datatypes.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/datatypes.c b/src/datatypes.c index 8750406..36a2b55 100644 --- a/src/datatypes.c +++ b

[libvirt] [PATCH 1/4] Index hashes by UUID instead of name

2010-06-17 Thread Jiri Denemark
Per-connection hashes for domains, networks, storage pools and network filter pools were indexed by names which was not the best choice. UUIDs are better identifiers, so lets use them. --- src/datatypes.c | 64 +++ 1 files changed, 36 insertion

Re: [libvirt] [PATCH] move ebiptables script out of /tmp

2010-06-17 Thread Daniel P. Berrange
On Thu, Jun 17, 2010 at 09:37:33AM -0400, Stefan Berger wrote: > On 06/17/2010 06:20 AM, Daniel P. Berrange wrote: > >On Wed, Jun 16, 2010 at 03:57:50PM -0500, Jamie Strandboge wrote: > > > >>On Wed, 2010-06-16 at 14:04 -0600, Eric Blake wrote: > >> > >>>I'm with Stefan - the whole point of

Re: [libvirt] [PATCH] move ebiptables script out of /tmp

2010-06-17 Thread Stefan Berger
On 06/17/2010 06:20 AM, Daniel P. Berrange wrote: On Wed, Jun 16, 2010 at 03:57:50PM -0500, Jamie Strandboge wrote: On Wed, 2010-06-16 at 14:04 -0600, Eric Blake wrote: I'm with Stefan - the whole point of mkstemp is that the created file has 0600 permissions, and /tmp is restricted-d

Re: [libvirt] 1. Domain Destroy , 2. Domain boot on a different host and 3. Domain start time

2010-06-17 Thread Justin Clift
On 06/17/2010 07:04 PM, IKI-サガル バルウェ wrote: > I understood now what the destroy function does. As for "domainUndefine": > To quote from the "undefine" method's documentation: > Undefine a domain but does not stop it if it is running > So, If the Domain remains as it is, what is the significance of

Re: [libvirt] [PATCH] macvtap: work-around for 2.6.32 and older kernels

2010-06-17 Thread Stefan Berger
On 06/16/2010 11:45 PM, Laine Stump wrote: On 06/17/2010 06:25 AM, Stefan Berger wrote: This patch works around a recent extension of the netlink driver I had made use of when building the netlink messages. Unfortunately older kernels don't accept IFLA_IFNAME + name of interface as a replaceme

Re: [libvirt] [PATCH] move ebiptables script out of /tmp

2010-06-17 Thread Daniel P. Berrange
On Wed, Jun 16, 2010 at 03:57:50PM -0500, Jamie Strandboge wrote: > On Wed, 2010-06-16 at 14:04 -0600, Eric Blake wrote: > > I'm with Stefan - the whole point of mkstemp is that the created file > > has 0600 permissions, and /tmp is restricted-deletion, so no other user > > can either overwrite the

Re: [libvirt] [PATCH] Add several missing vir*Free calls in libvirtd's remote code

2010-06-17 Thread Daniel P. Berrange
On Thu, Jun 17, 2010 at 12:29:34AM +0200, Matthias Bolte wrote: > 2010/6/16 Daniel P. Berrange : > > On Sat, Jun 12, 2010 at 06:32:55PM +0200, Matthias Bolte wrote: > >> Justin Clift reported a problem with adding virStoragePoolIsPersistent > >> to virsh's pool-info command, resulting in a strange

Re: [libvirt] [PATCH] virsh: ensure persistence and autostart are shown for dominfo and pool-info

2010-06-17 Thread Justin Clift
On 06/17/2010 10:34 AM, Justin Clift wrote: On 06/17/2010 09:28 AM, Eric Blake wrote: Applying this patch causes 3 test failures in tests/virshtest; I think you need to update the expected output to include your new row of Persistent: information before we can apply this. Ouch. Not sure how I

Re: [libvirt] 1. Domain Destroy , 2. Domain boot on a different host and 3. Domain start time

2010-06-17 Thread IKI-サガル バルウェ
Thanks a lot Justin for such a detailed explanation. > If the domain has an XML definition for it in place, the "Destroy" > command won't touch it. You can start the domain again using the > "Create" function: > > http://libvirt.org/html/libvirt-libvirt.html#virDomainCreate > > If you want to ge

Re: [libvirt] 1. Domain Destroy , 2. Domain boot on a different host and 3. Domain start time

2010-06-17 Thread Justin Clift
On 06/17/2010 05:50 PM, IKI-サガル バルウェ wrote: > Hello, > > Firstly, I have some confusion about the libvirt API "virDomainDestroy" > method. here is the link to API: > http://libvirt.org/html/libvirt-libvirt.html#virDomainDestroy "Destroy" here can be a bit confusing. It doesn't really touch or r

[libvirt] [PATCHv2] virsh: ensure persistence and autostart are shown for dominfo and pool-info

2010-06-17 Thread Justin Clift
This patch adds the persistence status (yes/no) to the output of the virsh dominfo and pool-info commands. This patch also adds the autostart status to the output of the virsh pool-info command. Red Hat BZ for this: https://bugzilla.redhat.com/show_bug.cgi?id=603696 --- This 2nd version of t

[libvirt] 1. Domain Destroy , 2. Domain boot on a different host and 3. Domain start time

2010-06-17 Thread IKI-サガル バルウェ
Hello, Firstly, I have some confusion about the libvirt API "virDomainDestroy" method. here is the link to API: http://libvirt.org/html/libvirt-libvirt.html#virDomainDestroy Does this method completely removes the domain from the system? I have confusion because the "virsh destroy" tool option

Re: [libvirt] libvirt-python apis

2010-06-17 Thread Alex Jia
Hi pengphy, http://libvirt.org/python.html In addition, source code provides some examples about python binding API under the ./libvirt/python/tests/ directory: # ls libvirt/python/tests/ basic.py create.py error.py Makefile.am node.py uuid.py Regards, Alex - Original Message - From

Re: [libvirt] [PATCHv3] qemu: reduce file padding requirements

2010-06-17 Thread Laine Stump
On 06/10/2010 03:31 PM, Eric Blake wrote: Followup to https://bugzilla.redhat.com/show_bug.cgi?id=599091, commit 20206a4b, to reduce disk waste in padding. * src/qemu/qemu_monitor.h (QEMU_MONITOR_MIGRATE_TO_FILE_BS): Drop back to 512. (QEMU_MONITOR_MIGRATE_TO_FILE_TRANSFER_SIZE): New macro. * sr

[libvirt] libvirt-python apis

2010-06-17 Thread pengphy
hi, all I do not know how to use libvirt python apis, and there seems no documents about that, so if anyone has documents or links about python apis please send it to me! thanks! Regards, xpen -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/li