On 05/02/2018 03:34 PM, Collin Walling wrote:
> On 05/02/2018 03:43 AM, Jiri Denemark wrote:
>> On Tue, May 01, 2018 at 09:57:08 -0400, John Ferlan wrote:
>>>
>>>
>>> On 04/30/2018 10:55 PM, Chris Venteicher wrote:
This is part of resolution of:
ping?
Still need review on 1-3, & 6
Tks
John
On 04/24/2018 08:28 AM, John Ferlan wrote:
> This should be the "last time" (famous last words) needing to alter
> this processing. I hope to get more than one pair of eyes looking
> at the series. I've looked at it long enough to feel comfortable
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Make sure that 'host_device' is generated for type='block'.
>
> Signed-off-by: Peter Krempa
> ---
> tests/qemublocktest.c | 2 ++
> tests/qemublocktestdata/xml2json/block-raw-noopts.json | 12
ping?
Tks,
John
On 04/19/2018 05:46 PM, John Ferlan wrote:
> Rather than have virJSONValueArraySize return a -1 when the input
> is not an array and then splat an error message, let's check for
> an array before calling and then change the return to be a size_t
> instead of ssize_t.
>
> That
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> The test cases would correspond to the following -drive command lines:
>
> dir-fat-cache.xml:
> -drive
> file=fat:/var/somefiles,if=none,id=drive-dummy,readonly=on,cache=directsync
> -device
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> The test cases would correspond to the following -drive command lines:
>
> file-backing_basic-detect.xml:
> -drive
> file=/var/lib/libvirt/images/a,format=qcow,if=none,id=drive-dummy,detect-zeroes=on
> -device
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> iscsi and rbd support authentication of the connection. Combine it with
> encryption of qcow2.
>
> The top level disk image would generate the following '-drive' cmdline:
>
> -drive
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Add tests for backing chain handling, including a very long chain which
> is fully specified in the XML and an unterminated chain.
>
> The top level disk image would generate the following '-drive':
>
> file-qcow2-backing-chain-encryption.xml:
>
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Apart from adding test data add a function which sets up fake secrets
> for the test.
>
> The top level disk image would generate the following '-drive' cmdline:
>
> -drive
>
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Test that the 'aio' option is applied correctly for the 'file' protocol
> backend and across the backing chain.
>
> The top level disk image would generate the following '-drive' cmdline:
>
> file-backing_basic-aio_threads:
> -drive
>
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Test mapping of the 'FAT' disk format to 'vvfat' in qemu.
>
> The top level disk image would generate the following '-drive' cmdline:
>
> dir-fat-readonly.xml:
> -drive file=fat:/var/somefiles,if=none,id=drive-dummy,readonly=on
> -device
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Similarly to the 'raw' case add tests for bochs, cloop, dmg, ploop, vdi
> vhd, and vpc. Covering all supproted non-backing formats.
supported
>
> Note that the JSON name for 'ploop' maps to 'parallels' and 'vhd' maps
> to 'vhdx'.
>
> Files added
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Formats supporting backing chain such as qed, vmdk, don't have any other
> parameters than the backing store and 'qcow' has only encryption params
> which will be tested extra. Add this test case so they are covered since
> any further test cases
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Test the JSON props generator with a very simple 'raw' image with no
> other options. The node-names for the image are 31 bytes long so that we
> validate our node name detector.
>
> The top level disk image would generate the following '-drive'
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Add a test infrastructure that will allow testing the JSON object
> generator used for generating data to use with blockdev-add.
>
> The resulting disk including the backing chain is validated to conform
> to the QAPI schema and the expected output
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> The function will be reused in the test code where we don't care much
> that the gluster debug level can't be populated from the qemu config.
>
> Set the level only when 'cfg' is passed.
>
> Signed-off-by: Peter Krempa
> ---
>
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> It will be used in the qemublocktest.
>
> Signed-off-by: Peter Krempa
> ---
> src/qemu/qemu_domain.c | 2 +-
> src/qemu/qemu_domain.h | 3 +++
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
Reviewed-by: John Ferlan
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Remove gnulib from _LDADD and move LDADDS to replace it. Also reformat
> the _SOURCES so that they can be easily extended.
>
> Signed-off-by: Peter Krempa
> ---
> tests/Makefile.am | 8 +---
> 1 file changed, 5
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> When using blockdev-add and friends, libvirt will need to create also
> properties for the qcow2/raw/... format handler in qemu. This patch adds
> the infrastructure and implements all formats known to libvirt including
> all properties which are
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> New tests will add new data structures so rename the 'data' structure.
>
> Signed-off-by: Peter Krempa
> ---
> tests/qemublocktest.c | 9 +
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
Reviewed-by: John Ferlan
On 05/02/2018 03:39 PM, John Ferlan wrote:
>
>
> On 04/25/2018 11:15 AM, Peter Krempa wrote:
>> Enabling discard for the storage node allows the format drivers to
>> discard snapshots and other things, while configuration of the format
>> layer actually decides whether to actually discard data
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Enabling discard for the storage node allows the format drivers to
> discard snapshots and other things, while configuration of the format
> layer actually decides whether to actually discard data on request from
> the host.
>
> Signed-off-by: Peter
On 05/02/2018 03:43 AM, Jiri Denemark wrote:
> On Tue, May 01, 2018 at 09:57:08 -0400, John Ferlan wrote:
>>
>>
>> On 04/30/2018 10:55 PM, Chris Venteicher wrote:
>>> This is part of resolution of:
>>> https://bugzilla.redhat.com/show_bug.cgi?id=1511999
>>>
>>> Signed-off-by: Chris Venteicher
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> This will be required when doing blockdev-add to conform with the
> approach qemu would chose to create the disks.
s/would chose/chooses/
If 'would' does stay, then it's still choose as chose would be past tense.
>
> Signed-off-by: Peter Krempa
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> When used directly with blockdev-add/-blockdev the cache mode will need
> to be specified directly for every image rather than just for the disk
> itself. This implements the backing options 'direct' and 'no-flush'.
>
> Signed-off-by: Peter Krempa
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> qemu declares node-name as a 32 byte buffer and silently truncates
> anything longer than that. This is unacceptable for libvirt, so we need
> to make sure that we won't ever supply a node-name exceeding 31 chars.
>
> Add a function which will do
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> The gluster protocol in qemu uses two styles, one of which is legacy and
> not covered by the QAPI schema. To allow using of the new style in the
> blockdev-add code, add a parameter for
> qemuBlockStorageSourceGetBackendProps which will switch
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Block storage should actually be passed to qemu via 'host_device' or
> 'host_cdrom' according to the device type. There were no users of this
> behaviour so we thankfully can change it.
>
> Signed-off-by: Peter Krempa
> ---
>
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Use virFileIsCdrom to detect whether a block device is a cdrom drive and
> store it in virStorageSource. This will be necessary to correctly create
> the 'host_cdrom' backend in qemu when using -blockdev.
>
> We assume that host_cdrom makes only
On 04/25/2018 11:15 AM, Peter Krempa wrote:
> Add detection mechanism which will allow to check whether a path to a
> block device is a physical CDROM drive. This will be useful once we will
> need to pass it to hypervisors.
>
> The linux implementation uses an ioctl to do the detection, while
On Sat, Apr 28, 2018 at 1:11 AM, John Ferlan wrote:
>
>
> On 04/20/2018 04:59 AM, Prerna Saxena wrote:
> > So far libvirt domain XML only allows local filepaths that can be
> > used to specify a loader element or its matching NVRAM disk.
> > Given that Vms may themselves move
On Wed, 2018-05-02 at 17:54 +0100, Maciej Wolny wrote:
> Fixes "can't find libtool" error when running valgrind checks.
>
> Signed-off-by: Maciej Wolny
> ---
> tests/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
Fixes "can't find libtool" error when running valgrind checks.
Signed-off-by: Maciej Wolny
---
tests/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 7b93fbde6..05db6b119 100644
---
On Wednesday, 2 May 2018 18:32:25 CEST Daniel P. Berrangé wrote:
> On Mon, Apr 30, 2018 at 06:52:58PM +0200, Andrea Bolognani wrote:
> > The latter is a glibc extension that's not available on other
> > operating systems, notably FreeBSD.
> >
> > So far we have worked around the issue through
On Wed, 2018-05-02 at 17:32 +0100, Daniel P. Berrangé wrote:
> On Mon, Apr 30, 2018 at 06:52:58PM +0200, Andrea Bolognani wrote:
> > The latter is a glibc extension that's not available on other
> > operating systems, notably FreeBSD.
> >
> > So far we have worked around the issue through gnulib,
On Wed, May 02, 2018 at 05:32:25PM +0100, Daniel P. Berrangé wrote:
> On Mon, Apr 30, 2018 at 06:52:58PM +0200, Andrea Bolognani wrote:
> > The latter is a glibc extension that's not available on other
> > operating systems, notably FreeBSD.
> >
> > So far we have worked around the issue through
On Mon, Apr 30, 2018 at 06:52:58PM +0200, Andrea Bolognani wrote:
> The latter is a glibc extension that's not available on other
> operating systems, notably FreeBSD.
>
> So far we have worked around the issue through gnulib, but that
> makes it difficult to use mocking in our test suite, so
On 02/05/18 17:15, Andrea Bolognani wrote:
> On Wed, 2018-05-02 at 12:24 +0200, Andrea Bolognani wrote:
>> On Tue, 2018-05-01 at 14:49 +0100, Maciej Wolny wrote:
>>> Fixes "can't find libtool" error when running valgrind checks.
>>> ---
>>> tests/Makefile.am | 2 +-
>>> 1 file changed, 1
On Wed, 2018-05-02 at 12:24 +0200, Andrea Bolognani wrote:
> On Tue, 2018-05-01 at 14:49 +0100, Maciej Wolny wrote:
> > Fixes "can't find libtool" error when running valgrind checks.
> > ---
> > tests/Makefile.am | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git
Now that mocking NUMA information works on FreeBSD, there are
no longer any test cases that need to be restricted to Linux
only.
Signed-off-by: Andrea Bolognani
---
tests/qemuxml2argvtest.c | 46 +++---
1 file changed, 15
While the current amount of mocking works just fine on most of
our target platforms, it somehow causes issues when using Clang
on FreeBSD.
Work around the issue by mocking a couple more functions. It's
not pretty, but it makes qemuxml2argvtest pass on FreeBSD at
long last.
Signed-off-by: Andrea
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Domain.xml | 7 +
src/domain.c| 62 +
2 files changed, 69 insertions(+)
diff --git a/data/org.libvirt.Domain.xml b/data/org.libvirt.Domain.xml
index
As planned the release is done, it is tagged into git and I pushed
signed tarball and rpms to the usual place:
ftp://libvirt.org/libvirt/
I also pushed a release of the python bindings libvirt-python 4.3.0 at
ftp://libvirt.org/libvirt/python
As usual this release is a balance of
On 05/02/2018 05:51 AM, Peter Krempa wrote:
> On Tue, May 01, 2018 at 20:06:43 -0400, John Ferlan wrote:
>>
>>
>> On 04/25/2018 11:15 AM, Peter Krempa wrote:
>>> Save and restore node names if we know them in the status XML so that we
>>> don't need to recalculate them or don't lose them in some
Hi all,
As a quick introduction, I'm Ramy Elkest, a student at Georgia Tech and
one of the lucky ones selected by libvirt for this year's Google Summer
of Code.
During the next months I'll be working with my mentors Martin Kletzander
and Erik Skultety to introduce nodejs to libvirt's family of
On Wed, May 02, 2018 at 14:46:33 +0200, Ján Tomko wrote:
> Unused since its introduction in commit <76b644c>.
>
> Signed-off-by: Ján Tomko
> ---
> src/conf/domain_conf.h | 3 ---
> 1 file changed, 3 deletions(-)
ACK
signature.asc
Description: PGP signature
--
libvir-list
s/atttribute/attribute/
Signed-off-by: Ján Tomko
---
Pushed as trivial.
src/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/Makefile.am b/src/Makefile.am
index 0d8d380df1..0c380780c3 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@
Unused since its introduction in commit <76b644c>.
Signed-off-by: Ján Tomko
---
src/conf/domain_conf.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index 3c7eccb8ca..3e135c6364 100644
--- a/src/conf/domain_conf.h
+++
Thanks, John!
--
Julio Cesar Faracco
2018-05-01 19:56 GMT-03:00 John Ferlan :
>
>
> On 05/01/2018 11:56 AM, Julio Faracco wrote:
>> The commits enhace the 'domrename' command of QEMU and Test drivers to
>> avoid slashes chars inside the new domain name.
>>
>> Julio Faracco
Please use 'schema' instead of 'rng'.
On Wed, May 02, 2018 at 13:59:06 +0200, Erik Skultety wrote:
> Panic device has 2 optional sub-elements - and the
> order of which should be interchangeable in the XML.
>
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1456165
>
> Signed-off-by:
Panic device has 2 optional sub-elements - and the
order of which should be interchangeable in the XML.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1456165
Signed-off-by: Erik Skultety
---
docs/schemas/domaincommon.rng | 14 --
1 file changed, 8
On Wed, May 02, 2018 at 11:48:24AM +0100, Maciej Wolny wrote:
> On 02/05/18 08:13, Daniel P. Berrangé wrote:
> > On Tue, May 01, 2018 at 08:22:45PM +0100, Maciej Wolny wrote:
> > > Add SDL graphics gl attribute, modify the domain XML schema, add a
> > > test, modify the documentation to include
On 02/05/18 08:13, Daniel P. Berrangé wrote:
On Tue, May 01, 2018 at 08:22:45PM +0100, Maciej Wolny wrote:
Add SDL graphics gl attribute, modify the domain XML schema, add a
test, modify the documentation to include the new option.
Signed-off-by: Maciej Wolny
---
Since we started using `make distcheck` in travis, the part that's printing the
tests/test-suite.log file is rather pointless. Let's check for both known
locations (keeping the previous one there on purpose) so that the output is
usable again.
Signed-off-by: Martin Kletzander
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 44
2 files changed, 50 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libvirt.Connect.xml
index
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 8
src/connect.c| 42 ++
2 files changed, 50 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libvirt.Connect.xml
index
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 38 ++
2 files changed, 44 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libvirt.Connect.xml
index
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 5 +
src/connect.c| 25 +
tests/test_connect.py| 4
3 files changed, 34 insertions(+)
diff --git a/data/org.libvirt.Connect.xml
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 31 +++
2 files changed, 37 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libvirt.Connect.xml
index 610dda7..e46d28d
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 7 +++
src/connect.c| 41 +
tests/test_connect.py| 4
3 files changed, 52 insertions(+)
diff --git a/data/org.libvirt.Connect.xml
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 35 +++
tests/test_connect.py| 4
3 files changed, 45 insertions(+)
diff --git a/data/org.libvirt.Connect.xml
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 5 +
src/connect.c| 24
tests/test_connect.py| 4
3 files changed, 33 insertions(+)
diff --git a/data/org.libvirt.Connect.xml
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 5 +
src/connect.c| 23 +++
2 files changed, 28 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libvirt.Connect.xml
index 4a1ee4a..610dda7 100644
Changes from v1:
* Added tests were the test driver exported the relevant API
* Removed NodeGetCellsFreeMemory which will be sent in seperate patchset
* Adjusted other comments from review
Katerina Koukiou (9):
Implement NodeGetCPUStats method for Connect Interface
Implement NodeGetFreeMemory
On Tue, 2018-05-01 at 14:49 +0100, Maciej Wolny wrote:
> Fixes "can't find libtool" error when running valgrind checks.
> ---
> tests/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 7b93fbde6..05db6b119 100644
>
On Tue, May 01, 2018 at 20:06:43 -0400, John Ferlan wrote:
>
>
> On 04/25/2018 11:15 AM, Peter Krempa wrote:
> > Save and restore node names if we know them in the status XML so that we
> > don't need to recalculate them or don't lose them in some cases.
> >
> > Signed-off-by: Peter Krempa
On Tue, May 01, 2018 at 20:06:40 -0400, John Ferlan wrote:
>
>
> On 04/25/2018 11:15 AM, Peter Krempa wrote:
> > Everything besides the top of the chain is readonly. Track this when
> > parsing the XML and detecting the chain from the disk. Also fix the
> > state when taking snapshots.
> >
> >
On Mon, Apr 30, 2018 at 12:12:24PM -0400, Laine Stump wrote:
> First - this is a *great* idea, and we should do something similar to
> the network driver (keep track of all the connections to each network so
> they can be re-connected when a network is stopped/started).
Yes, this nwfilter
On Wed, May 02, 2018 at 09:44:08 +0200, Peter Krempa wrote:
> Signed-off-by: Peter Krempa
> ---
> docs/news.xml | 12
> 1 file changed, 12 insertions(+)
>
> diff --git a/docs/news.xml b/docs/news.xml
> index caeda159a8..af57751e0c 100644
> --- a/docs/news.xml
>
On Tue, 2018-05-01 at 12:06 +0100, Daniel P. Berrangé wrote:
> The current socket test is rather crazy in that it sets up a server
> listening for sockets and then runs a client connect call, relying on
> the fact that the kernel will accept this despite the application
> not having called
On Tue, May 01, 2018 at 20:21:11 -0400, John Ferlan wrote:
>
>
> On 04/25/2018 11:15 AM, Peter Krempa wrote:
> > Handle VIR_STORAGE_TYPE_DIR in qemuBlockStorageSourceGetBackendProps so
> > that a 'vvfat' driver is used, which emulates a FAT filesystem
> > containing the folders.
> >
> > qemu
On Tue, 2018-05-01 at 12:06 +0100, Daniel P. Berrangé wrote:
> The test code for UNIX and TCP sockets will need to be rewritten and
> extended later, and will benefit from code sharing.
>
> Signed-off-by: Daniel P. Berrangé
> ---
> tests/virnetsockettest.c | 120
>
On Tue, May 01, 2018 at 20:25:09 -0400, John Ferlan wrote:
>
>
> On 04/25/2018 11:15 AM, Peter Krempa wrote:
> > Add detection mechanism which will allow to check whether a path to a
> > block device is a physical CDROM drive. This will be useful once we will
> > need to pass it to hypervisors.
On Tue, May 01, 2018 at 09:57:08 -0400, John Ferlan wrote:
>
>
> On 04/30/2018 10:55 PM, Chris Venteicher wrote:
> > This is part of resolution of:
> > https://bugzilla.redhat.com/show_bug.cgi?id=1511999
> >
> > Signed-off-by: Chris Venteicher
> >
> > diff to v1:
> > -
Signed-off-by: Peter Krempa
---
docs/news.xml | 12
1 file changed, 12 insertions(+)
diff --git a/docs/news.xml b/docs/news.xml
index caeda159a8..af57751e0c 100644
--- a/docs/news.xml
+++ b/docs/news.xml
@@ -157,6 +157,18 @@
+
+
Hi,
by discussing on bug [1] we found an issue in the ZFS storage backend.
When one defines a zfs pool through XML (virsh pool-create --build) a top
level ZFS pool will be created and works fine. The virsh pool-define-as
counterpart to this is:
$ fallocate -l 100M /tmp/Mzfs
$ sudo zpool create
On Tue, May 01, 2018 at 08:22:45PM +0100, Maciej Wolny wrote:
> Add SDL graphics gl attribute, modify the domain XML schema, add a
> test, modify the documentation to include the new option.
>
> Signed-off-by: Maciej Wolny
> ---
> docs/schemas/domaincommon.rng
77 matches
Mail list logo