On 04/01/2014 06:04 AM, Nehal J Wani wrote:
On Sun, Mar 16, 2014 at 1:28 PM, Nehal J Wani nehaljw.k...@gmail.com wrote:
I followed the following steps to build a clean development version of
libvirt from source. Can't seem to understand what is wrong. (Entire
buildLog has been attached):
➜
On 03/31/14 17:23, Eric Blake wrote:
On 03/31/2014 06:18 AM, Peter Krempa wrote:
According to our documentation the key value has the following
meaning: Providing an identifier for the volume which identifies a
single volume. The currently used keys for gluster volumes consist of
the gluster
Signed-off-by: Martin Kletzander mklet...@redhat.com
---
Notes:
Pushed as a build-breaker.
src/storage/storage_backend.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/storage/storage_backend.c b/src/storage/storage_backend.c
index 42bd445..adf0311 100644
---
On 04/01/2014 02:46 PM, Jincheng Miao wrote:
On 04/01/2014 06:04 AM, Nehal J Wani wrote:
On Sun, Mar 16, 2014 at 1:28 PM, Nehal J Wani
nehaljw.k...@gmail.com wrote:
I followed the following steps to build a clean development version of
libvirt from source. Can't seem to understand what is
On 31.03.2014 23:43, Eric Blake wrote:
On 03/31/2014 10:00 AM, Julio Faracco wrote:
Hi everybody!
I sent an e-mail to Daniel asking about contributing to the libvirt
community for free.
So, I copied the e-mail to the libvirt mailing list as Daniel suggested me.
If anyone can help me I'd be
This reduces the affect of an unexpected DoS vulnerablity in libvirtd.
---
include/libvirt/libvirt.h.in | 13 +
src/driver.h | 5 +
src/libvirt.c| 32 +++
src/libvirt_private.syms | 1 +
src/libvirt_public.syms
On 04/01/14 09:34, Ján Tomko wrote:
This reduces the affect of an unexpected DoS vulnerablity in libvirtd.
---
include/libvirt/libvirt.h.in | 13 +
src/driver.h | 5 +
src/libvirt.c| 32 +++
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
Pushed already, just sending a note.
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index 73efffa..7b9e38a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -16,7 +16,7 @@ dnl You
On 03/31/2014 08:50 PM, Cole Robinson wrote:
VolOpen notifies the user of a potentially non-fatal failure by
returning -2 and logging a VIR_WARN or VIR_INFO. Unfortunately most
callers treat -2 as fatal but don't actually report any message with
the error APIs.
Change VolOpen to report an
On Tue, Apr 01, 2014 at 09:44:16AM +0200, Peter Krempa wrote:
On 04/01/14 09:34, Ján Tomko wrote:
This reduces the affect of an unexpected DoS vulnerablity in libvirtd.
---
include/libvirt/libvirt.h.in | 13 +
src/driver.h | 5 +
src/libvirt.c|
-Original Message-
From: libvir-list-boun...@redhat.com
[mailto:libvir-list-boun...@redhat.com] On Behalf Of Daniel P. Berrange
Sent: Saturday, March 22, 2014 12:37 AM
To: Yang, Zhiyong/杨 志勇
Cc: libvir-list@redhat.com; Xinghai Yu
Subject: Re: [libvirt] [PATCH 00/13] Add multiple
So as scheduled, libvirt-1.2.3 release is ready, it is tagged in git
and the tarball and rpms are available at the usual place:
ftp://libvirt.org/libvirt/
This release is rather large in the number of commits (more than 400)
but includes a lot of changes made to clean up the code base.
On 03/28/14 23:13, Eric Blake wrote:
On 03/28/2014 04:01 PM, Peter Krempa wrote:
Extract the NFS related stuff into a separate function and tidy up the
rest of the code so we can reuse it to add gluster backend detection.
Additionally avoid reporting of errors from showmount and return an
On 03/28/14 23:57, Ján Tomko wrote:
On 03/28/2014 11:01 PM, Peter Krempa wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1072714
Use the gluster command line tool to retrieve information about remote
volumes on a gluster server to allow storage pool source lookup.
Unfortunately gluster
On 03/28/14 23:22, Eric Blake wrote:
On 03/28/2014 04:01 PM, Peter Krempa wrote:
Use the previously implemented function to lookup glusterfs source pools
for the netfs pool to lookup native gluster pools too.
---
src/storage/storage_backend_gluster.c | 50
On all the places where qemuAgentComand() was called, we did a check
for errors in the reply. Unfortunately, some of the places called
qemuAgentCheckError() without checking for non-null reply which might
have resulted in a crash.
So this patch makes the error-checking part of qemuAgentCommand()
This uses the dbus api of systemd to check the power management
capabilities of the node.
---
This replaces the previous pm-utils dropping patch attempt to implement
Cole's idea.
configure.ac | 11
libvirt.spec.in | 9 +++
src/libvirt_private.syms | 3
On 03/31/14 18:10, Eric Blake wrote:
On 03/31/2014 03:59 AM, Peter Krempa wrote:
On 03/30/14 05:38, Eric Blake wrote:
A fairly smooth transition. And now that domain disks and
storage volumes share a common struct, it opens the doors for
a future patch to expose more details in the XML for
On 04/01/2014 02:56 AM, Ján Tomko wrote:
On 03/31/2014 08:50 PM, Cole Robinson wrote:
VolOpen notifies the user of a potentially non-fatal failure by
returning -2 and logging a VIR_WARN or VIR_INFO. Unfortunately most
callers treat -2 as fatal but don't actually report any message with
the
Hum... it seems I'll have to fix somethings in that patch, ignore it for
the while.
--
Cedric
On Tue, 2014-04-01 at 15:19 +0200, Cédric Bosdonnat wrote:
This uses the dbus api of systemd to check the power management
capabilities of the node.
---
This replaces the previous pm-utils
On 04/01/2014 07:22 AM, Martin Kletzander wrote:
On all the places where qemuAgentComand() was called, we did a check
for errors in the reply. Unfortunately, some of the places called
qemuAgentCheckError() without checking for non-null reply which might
have resulted in a crash.
So this
Every caller checked the return value and logged an error
- one if no device with the specified MAC was found,
other if there were multiple devices matching the MAC address
(except for qemuDomainUpdateDeviceConfig which logged the same
message in both cases).
Move the error reporting into
When looking up a net device by a MAC and PCI address, it is possible
that we've got a match on the MAC address but failed to match the
PCI address.
In that case, outputting just the MAC address can be confusing.
Partially resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=872028
---
Partially fixes https://bugzilla.redhat.com/show_bug.cgi?id=872028
Does nothing to adress the fact that interface type and other
attributes are ignored on device detach
Ján Tomko (2):
Move error reporting into virDomainNetFindIdx
Include PCI address in the error in virDomainNetFindIdx
On 03/31/2014 09:56 AM, Eric Blake wrote:
On 03/31/2014 02:56 AM, Peter Krempa wrote:
On 03/29/14 23:20, Eric Blake wrote:
Now that we have a common struct, it's time to start using it!
Since external snapshots make a longer backing chain, it is
only natural to use the same struct for the
On 03/25/2014 02:14 PM, Eric Blake wrote:
In 'make syntax-check', we have a rule that prevents layering
violations between the various files in src. However, we
forgot to treat conf/ and the more recently-added access/ as
lower-level directories, and were not detecting cases where
they might
With $tck added to the skip conditional, the entire codeblock has to be
after $tck is defined. Also, $tck-cleanup should be added to the skip
conditions.
---
scripts/nwfilter/300-vsitype.t | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git
Trivial, but there are only 5 tests in 110-static-relabel-yes.t.
---
scripts/selinux/110-static-relabel-yes.t | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/selinux/110-static-relabel-yes.t
b/scripts/selinux/110-static-relabel-yes.t
index f558cc9..76781ca 100644
---
On Tue, Mar 25, 2014 at 1:53 PM, Nehal J Wani nehaljw.k...@gmail.com wrote:
When libvirtd is run from a build directory without being installed, it
should not depend on files from a libvirt package installed in the
system. Currently, APIs defined in src/ don't know whether libvirtd
is being
Noticed during my work on storage struct cleanups.
* src/storage/storage_backend_disk.c
(virStorageBackendDiskPartBoundaries): Fix spelling errors.
Signed-off-by: Eric Blake ebl...@redhat.com
---
Pushing under the trivial rule. Particularly odd that we managed
too different ways of
For environments not using a /24 netmask, this test can end up in a hung
state. This patch reads the netmask from the nic and uses it later when
ip addresses are changed and restored.
---
scripts/nwfilter/220-no-ip-spoofing.t | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff
The phyp code claims that it wants a non-zero value, but actually
enforces a capacity of zero. It has been this way since commit
ebc46fe in June 2010. Bummer that it has my name as the committer
- I guess I should have been much more stubborn about not blindly
taking someone else's 1600-line
Some environments (openSUSE 13.1) can report the network is unreachable during
this test. Trap that condition as well.
---
scripts/nwfilter/210-no-mac-spoofing.t | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/nwfilter/210-no-mac-spoofing.t
I have a case where I'm creating a virtio channel between the host and
guest using something like this:
channel type=unix
source mode=bind path=/path/in/host/instance_name/
target type=virtio name=name_in_guest/
/channel
When qemu is started up this gets created as expected,
Version 2: Electric Boogaloo
Version 1: https://www.redhat.com/archives/libvir-list/2014-March/msg01898.html
This version of the patch introduces the following new things:
- Tests (a whole bunch of them, in fact)!
- A new `complete` command to run get newline-separated
completion results from
Previously, virsh was able to complete initial command names,
as well as the names of flag options. However, this completion
was clunky for a number of reasons:
- it would provide a flag as an option for completion
even if it had already been used
- it did not support completion of positional
This commit extracts the parsing logic from vshCommandParse
so that it can be used by other methods. The vshCommandParse
method is designed to parse full commands and error out on
unknown information, so it is not suitable to simply use it
for autocompletion. Instead, the logic has been
This patch introduces a way for completers to retrieve the
current vshControl object by using the vshGetCompleterCtl()
macro. When readline is enabled, the main method stores the
vshControl pointer in a variable that is file-global to virsh.c.
Then, that pointer can be retrieved by the
This patch introduces a custom completer for domains,
and sets it to be used for all of the domain arguments.
In order to facilitate this, the functions involved in
retrieve in a list of domains were moved from
virsh-domain-monitor to virsh-completer, where they
can be included by any file that
A fairly smooth transition. And now that domain disks and
storage volumes share a common struct, it opens the doors for
a future patch to expose more details in the XML for both
objects.
* src/conf/storage_conf.h (_virStorageVolTarget): Delete.
(_virStorageVolDef): Use common type.
*
Some preparatory work before consolidating storage volume
structs with the rest of virstoragefile. Making these
changes allows a volume target to be much closer to (a
subset of) the virStorageSource struct.
Making perms be a pointer allows it to be optional if we
have a storage pool that doesn't
Now that each virStorageSource can track allocation information,
and given that we already have the information without extra
syscalls, it's easier to just always populate the information
directly into the struct than it is to sometimes pass the address
of the struct members down the call chain.
compared to v1, this is rebased on top of Cole's changes,
treats disk partition types slightly differently, and adds
a couple patches.
1/7: was 16/n, unreviewed so far
2/7: was 13/n with weak ack
3-4/7: was 14-15/n; commit messages improved to justify it further
5-7: all new patches
I'm still
A future patch will merge virStorageFileMetadata and virStorageSource,
but I found it easier to do if both structs use the same information
for tracking whether a source file needs encryption keys.
* src/util/virstoragefile.h (_virStorageFileMetadata): Prepare
full encryption struct instead of
Another step towards unification of structures. While we might
not expose everything in XML via domain disk as we do for
storage volume pointer, both places want to deal with (at least
part of) the backing chain; therefore, moving towards a single
struct usable from both contexts will make the
Now that we have a dedicated type for representing a disk source,
we might as well parse and format directly into that type instead
of piecemeal into pointers to members of the type.
* src/conf/domain_conf.h (virDomainDiskSourceDefFormatInternal)
(virDomainDiskSourceDefParse): Rename...
One of the features of qcow2 is that a wrapper file can have
more capacity than its backing file from the guest's perspective;
what's more, sparse files make tracking allocation of both
the active and backing file worthwhile. As such, it makes
more sense to show allocation numbers for each file
---
It's late for me, but I wanted to throw this out there to make
sure I'm on the right track. My premise here is that the current
virStorageFileMetadata is awkward to work with, because you have
to ask the parent for details about the child, instead of
directly storing those details in the
---
...continuing my thoughts from 8/7. These are the additional
fields required in virStorageSource to copy all fields directly
listed in virStorageFileMetadata that are not already rendered
redundant, as mentioned in those comments. By populating these
fields directly in virStorageSource, I
49 matches
Mail list logo