We're going to need information about the OS outside of
Ansible soon, so drop the OS detection code and start
storing the information as host variables instead.
Signed-off-by: Andrea Bolognani
---
guests/host_vars/libvirt-centos-7/main.yml| 4 ++
guests/host_vars/libvirt-debian-8/main.yml
This is basically the exact same algorithm used by the
Ansible playbooks to process package mappings, implemented
as a standalone script.
There's little to none error handling, and some information
is hardcoded instead of being configurable at runtime; more
importantly, before it can be considered
See 2/2 for details.
Andrea Bolognani (2):
guests: Remove OS detection
guests: Add Dockerfile generator
guests/dockerfile-generator | 72 +++
guests/host_vars/libvirt-centos-7/docker.yml | 2 +
guests/host_vars/libvirt-centos-7/main.yml| 4 ++
guests/
On Thu, Jun 14, 2018 at 05:59:40PM +0200, Katerina Koukiou wrote:
> pytest collects functions starting with test prefix
> to run them as tests by default.
>
> Rename functions that are not supposed to be run as tests,
> to avoid unexpected failures.
>
> Note: this commit also renames domain funct
On Thu, Jun 14, 2018 at 05:59:41PM +0200, Katerina Koukiou wrote:
> Signed-off-by: Katerina Koukiou
> ---
> data/org.libvirt.StoragePool.xml | 7 +++
> src/storagepool.c| 34 ++
> tests/libvirttest.py | 31 ++
On Thu, Jun 14, 2018 at 05:59:39PM +0200, Katerina Koukiou wrote:
> Signed-off-by: Katerina Koukiou
> ---
> tests/test_connect.py | 47 +++
> tests/xmldata.py | 37 +
> 2 files changed, 44 insertions(+), 40 delet
On Thu, Jun 14, 2018 at 05:59:37PM +0200, Katerina Koukiou wrote:
> Signed-off-by: Katerina Koukiou
> ---
> data/Makefile.am| 4 ++-
> data/org.libvirt.StorageVol.xml | 7
> src/Makefile.am | 4 ++-
> src/connect.c | 6
> src/connect
On Thu, Jun 14, 2018 at 09:09:48AM +0100, Daniel P. Berrangé wrote:
> On Wed, Jun 13, 2018 at 03:05:08PM -0300, Eduardo Habkost wrote:
> > Getting back to this discussion:
> >
> > On Tue, Jun 05, 2018 at 09:43:00AM +0100, Daniel P. Berrangé wrote:
> > > On Tue, Jun 05, 2018 at 09:27:46AM +0200, Ge
On 06/11/2018 06:52 AM, Chen Hanxiao wrote:
> From: Chen Hanxiao
>
> Update news for cmdDomblkinfo --all option.
>
> Signed-off-by: Chen Hanxiao
> ---
> docs/news.xml | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/docs/news.xml b/docs/news.xml
> index e0cf381f33..c07db048
On 06/11/2018 06:52 AM, Chen Hanxiao wrote:
> From: Chen Hanxiao
>
> This patch introduces --all to show all block devices info
> of guests like:
>
> virsh # domblkinfo --all
> Target CapacityAllocation Physical
> ---
> hda
On 06/11/2018 06:52 AM, Chen Hanxiao wrote:
> From: Chen Hanxiao
>
> Introduce helper cmdDomblkinfoPrint for printing.
>
> Signed-off-by: Chen Hanxiao
> ---
> tools/virsh-domain-monitor.c | 39 ++--
> 1 file changed, 24 insertions(+), 15 deletions(-)
>
Revi
Signed-off-by: Lubomir Rintel
---
src/qemu/qemu_domain_address.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/qemu/qemu_domain_address.c b/src/qemu/qemu_domain_address.c
index 01cfdbeadf..d3259212f5 100644
--- a/src/qemu/qemu_domain_address.c
+++ b/src/qemu/qemu_do
Signed-off-by: Lubomir Rintel
---
docs/schemas/basictypes.rng | 2 ++
1 file changed, 2 insertions(+)
diff --git a/docs/schemas/basictypes.rng b/docs/schemas/basictypes.rng
index 1a18cd31b1..14a3670e5c 100644
--- a/docs/schemas/basictypes.rng
+++ b/docs/schemas/basictypes.rng
@@ -398,6 +398,8 @@
Signed-off-by: Lubomir Rintel
---
tests/qemuxml2argvdata/riscv64-virt.args | 31 +++
tests/qemuxml2argvdata/riscv64-virt.xml | 32 +++
tests/qemuxml2argvtest.c | 3 ++
.../riscv64-virt.xml | 39 +++
It's ARM specific.
Signed-off-by: Lubomir Rintel
---
src/qemu/qemu_capabilities.c | 4 ++--
src/qemu/qemu_command.c| 4 ++--
src/qemu/qemu_domain.c | 16
src/qemu/qemu_domain.h | 2 +-
src/qemu/qemu_domain_address.c | 4 ++--
5 files changed, 15 ins
Signed-off-by: Lubomir Rintel
---
tests/testutilsqemu.c | 72 +++
1 file changed, 72 insertions(+)
diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c
index dc7e90b952..c6376262b9 100644
--- a/tests/testutilsqemu.c
+++ b/tests/testutilsqemu.c
@@ -28
Signed-off-by: Lubomir Rintel
---
src/qemu/qemu_domain.c | 2 ++
src/util/virarch.c | 3 +++
src/util/virarch.h | 6 ++
3 files changed, 11 insertions(+)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index dfaba677b1..275090fb58 100644
--- a/src/qemu/qemu_domain.c
+++
Signed-off-by: Lubomir Rintel
---
tests/capabilityschemadata/caps-qemu-kvm.xml | 36
1 file changed, 36 insertions(+)
diff --git a/tests/capabilityschemadata/caps-qemu-kvm.xml
b/tests/capabilityschemadata/caps-qemu-kvm.xml
index 88ec89e733..1aa98d140f 100644
--- a/tests/cap
Signed-off-by: Lubomir Rintel
---
tests/vircapstest.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/tests/vircapstest.c b/tests/vircapstest.c
index 1df3fa091f..e016e466d2 100644
--- a/tests/vircapstest.c
+++ b/tests/vircapstest.c
@@ -180,6 +180,12 @@ test_virCapsDomainDataLookupQEMU(c
It's ARM specific.
Signed-off-by: Lubomir Rintel
---
src/qemu/qemu_capabilities.c | 2 +-
src/qemu/qemu_domain.c | 6 +++---
src/qemu/qemu_domain.h | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
ind
Signed-off-by: Lubomir Rintel
---
src/qemu/qemu_domain.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 275090fb58..72b72364d5 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -3198,6 +3198,11 @@
Hello,
here's a second version of the RISV-V support patch set, initally
submitted just under a month ago [1].
[1] https://www.redhat.com/archives/libvir-list/2018-May/thread.html#01008
Thanks to the Andrea's suggestions the patch is now considerably
thinner, and split to hopefully more manageab
On Thu, Jun 14, 2018 at 06:53:33PM +0200, Andrea Bolognani wrote:
> Changes from [v1]:
>
> * keep DOCKER_CMD around instead of removing it completely,
> which will help once we introduce MinGW builds;
>
> * the last two patches are new.
>
>
> [v1] https://www.redhat.com/archives/libvir-list/2
The new name is arguably more informative, especially now
that we have MACOS_CMD and knowing that MINGW_CMD will
be introduced later on.
We still use DOCKER_CMD when calling Docker, and we assign
the actual script on a per-image basis. Having this
separation will help us when we introduce MinGW bu
We already perform a full build as part of distcheck, so
we can speed things up a bit by skipping the first
compilation.
Signed-off-by: Andrea Bolognani
---
.travis.yml | 1 -
1 file changed, 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 1e2e307ac5..a4a0bbb072 100644
--- a/.travis.
It mirrors the existing DOCKER_CMD, both in how it's defined
as part of the environment and how it's called by passing it
directly to the shell.
In addition to making the configuration more consistent, this
also allows us to move from having the macOS build script
divided into four steps, some of
Upgrading takes quite a bit of time, doesn't offer any
real value and causes a lot of grief. Let's just skip it.
We need to install xz explicitly now, since it's required
to make dist and no longer being dragged in by Python 3.
Signed-off-by: Andrea Bolognani
---
.travis.yml | 4 +---
1 file ch
Changes from [v1]:
* keep DOCKER_CMD around instead of removing it completely,
which will help once we introduce MinGW builds;
* the last two patches are new.
[v1] https://www.redhat.com/archives/libvir-list/2018-June/msg00579.html
Andrea Bolognani (6):
travis: Skip macOS upgrade
travis:
Without a proper separator, all commands in the error path
end up being interpreted as a single command, which is not
what we want.
Signed-off-by: Andrea Bolognani
---
.travis.yml | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index a90
All variable parts are taken from the environment, so we
can exploit inheritance and avoid duplication. This will
become more important as we start building on additional
platforms.
Signed-off-by: Andrea Bolognani
---
.travis.yml | 31 +++
1 file changed, 11 insertion
On 06/14/2018 10:22 AM, Michal Privoznik wrote:
> On 06/14/2018 12:11 AM, John Ferlan wrote:
>>
>>
>> On 06/08/2018 09:45 AM, Michal Privoznik wrote:
>>> These jobs can be used to mark job type over qemu agent.
>>>
>>> Signed-off-by: Michal Privoznik
>>> ---
>>> src/qemu/qemu_domain.c | 4 +++-
On 06/14/2018 10:32 AM, Michal Privoznik wrote:
> On 06/14/2018 12:10 AM, John Ferlan wrote:
>>
>>
>> On 06/08/2018 09:45 AM, Michal Privoznik wrote:
>>> The point is to break QEMU_JOB_* into smaller pieces which
>>> enables us to achieve higher throughput. For instance, if there
>>> are two thr
On Thu, Jun 14, 2018 at 04:54:43PM +0100, Daniel P. Berrangé wrote:
> On Thu, Jun 14, 2018 at 04:30:29PM +0200, Erik Skultety wrote:
> > Signed-off-by: Erik Skultety
> > ---
> > domain.go | 135
> > +-
> > 1 file changed, 134 insertions
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 25 +
tests/test_storage.py | 6 ++
3 files changed, 36 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xml b/da
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 30 ++
tests/test_connect.py| 15 +++
3 files changed, 51 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 29 +
2 files changed, 35 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xml b/data/org.libvirt.StorageVol.xml
index aed3
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 23 +++
tests/test_storage.py | 1 +
3 files changed, 29 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xml b/data/org.
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 44 +
tests/test_storage.py | 11 +++
3 files changed, 60 insertions(+)
diff --git a/data/org.libvir
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 26 ++
2 files changed, 32 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xml b/data/org.libvirt.StorageVol.xml
index c1fecf3
pytest collects functions starting with test prefix
to run them as tests by default.
Rename functions that are not supposed to be run as tests,
to avoid unexpected failures.
Note: this commit also renames domain function to follow
the naming convention for same purpose functions.
Signed-off-by:
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StoragePool.xml | 9 +
src/storagepool.c| 41
tests/test_storage.py| 22 +
3 files changed, 72 insertions(+)
diff --git a
Signed-off-by: Katerina Koukiou
---
tests/test_connect.py | 47 +++
tests/xmldata.py | 37 +
2 files changed, 44 insertions(+), 40 deletions(-)
create mode 100644 tests/xmldata.py
diff --git a/tests/test_connec
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 29 +
tests/test_storage.py | 7 +++
3 files changed, 42 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xm
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StoragePool.xml | 7 +++
src/storagepool.c| 34 ++
tests/libvirttest.py | 31 +++
tests/test_storage.py| 3 +++
tests/xmldata.py
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 26 ++
2 files changed, 32 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xml b/data/org.libvirt.StorageVol.xml
index fdde430
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 30 ++
tests/test_connect.py| 1 +
3 files changed, 37 insertions(+)
diff --git a/data/org.libvirt.Connect.xml b/data/org.libv
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 23 +++
tests/test_storage.py | 1 +
3 files changed, 29 insertions(+)
diff --git a/data/org.libvirt.StorageVol.xml b/data/org.
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StoragePool.xml | 6 ++
src/storagepool.c| 43
tests/test_storage.py| 8
3 files changed, 57 insertions(+)
diff --git a/data/org.libvir
Signed-off-by: Katerina Koukiou
Reviewed-by: Ján Tomko
---
data/org.libvirt.StoragePool.xml | 6 ++
src/storagepool.c| 34 ++
2 files changed, 40 insertions(+)
diff --git a/data/org.libvirt.StoragePool.xml b/data/org.libvirt.StoragePool.xml
i
On Thu, Jun 14, 2018 at 02:28:03PM +0200, Erik Skultety wrote:
We only formatted the element when QEMU supported the feature when
in fact we should always format the element to make clear that libvirt
knows about the feature and the fact whether it is or isn't supported
depends on QEMU version,
Changes from v1:
- Fixes from reviews
- And more:
* tests: Moved all xml definitions for tests in separate file.
* tests: Renamed all functions that are not supposed to be run as
tests, so that they don't have test prefix. I found out that these
were running as tests, when I add
Signed-off-by: Katerina Koukiou
---
data/Makefile.am| 4 ++-
data/org.libvirt.StorageVol.xml | 7
src/Makefile.am | 4 ++-
src/connect.c | 6
src/connect.h | 1 +
src/storagevol.c| 77 ++
On Thu, Jun 14, 2018 at 02:28:02PM +0200, Erik Skultety wrote:
The whole element was optional which it shouldn't be as if the
platform doesn't support SEV we should format .
Signed-off-by: Erik Skultety
---
docs/schemas/domaincaps.rng | 19 ++-
1 file changed, 10 insertions(+),
On Thu, Jun 14, 2018 at 04:30:29PM +0200, Erik Skultety wrote:
> Signed-off-by: Erik Skultety
> ---
> domain.go | 135
> +-
> 1 file changed, 134 insertions(+), 1 deletion(-)
>
> diff --git a/domain.go b/domain.go
> index aeeb24a..27e2
On Thu, Jun 14, 2018 at 04:30:30PM +0200, Erik Skultety wrote:
> Signed-off-by: Erik Skultety
> ---
> domain_capabilities.go | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git a/domain_capabilities.go b/domain_capabilities.go
> index 3f5a752..4844fd9 100644
> --- a/domain_capabilities
On Thu, Jun 14, 2018 at 02:28:02PM +0200, Erik Skultety wrote:
The whole element was optional which it shouldn't be as if the
platform doesn't support SEV we should format .
Signed-off-by: Erik Skultety
---
docs/schemas/domaincaps.rng | 19 ++-
1 file changed, 10 insertions(+),
On Thu, Jun 14, 2018 at 04:30:01PM +0200, Erik Skultety wrote:
> Signed-off-by: Erik Skultety
> ---
> connect.go| 59
> +++
> connect_compat.go | 12 +++
> connect_compat.h | 7 +++
> 3 files changed, 78 insertions(+)
>
>
On Thu, Jun 14, 2018 at 04:30:00PM +0200, Erik Skultety wrote:
> Signed-off-by: Erik Skultety
> ---
> domain.go| 41 +
> domain_compat.go | 13 +
> domain_compat.h | 6 ++
> 3 files changed, 60 insertions(+)
>
> diff --git a/domai
On Thu, Jun 14, 2018 at 06:59:53AM +0200, Fabiano Fidêncio wrote:
There are still a few places using virConfGetValue(), checking for the
specific type of the pointers and so on.
Those places are not going to be converted as:
- Using virConfGetValue*() would trigger virReportError() in the curren
On Thu, Jun 14, 2018 at 11:07:43AM +0200, Michal Privoznik wrote:
> On 06/13/2018 05:34 PM, John Ferlan wrote:
> >
> > $SUBJ: "Introduce" and "NO_WAIT"
> >
> >
> > On 06/07/2018 07:59 AM, Michal Privoznik wrote:
> >> https://bugzilla.redhat.com/show_bug.cgi?id=1552092
> >>
> >> If there's a long
On Thu, Jun 14, 2018 at 05:15:09PM +0200, Ján Tomko wrote:
> On Wed, Jun 13, 2018 at 12:48:37PM +0200, Erik Skultety wrote:
> > This binding allows to query the AMD's SEV firmware for various platform
> > specific things, like a PDH certificate and a certificate chain to
> > establish a trusted con
On 06/14/2018 04:10 PM, John Ferlan wrote:
>
> [...]
>
diff --git a/include/libvirt/libvirt-domain.h
b/include/libvirt/libvirt-domain.h
index da773b76cb..1a1d34620d 100644
--- a/include/libvirt/libvirt-domain.h
+++ b/include/libvirt/libvirt-domain.h
@@ -2055,6 +2055
On Thu, Jun 14, 2018 at 06:59:52AM +0200, Fabiano Fidêncio wrote:
xenParseVif() does a lot of stuff and, in order to make things cleaner,
let's split it in two new functions:
- xenParseVif(): it's a new function that keeps the old name. It's
responsible for the whole per-Vif logic from the old xe
On Thu, Jun 14, 2018 at 11:48:41AM +0100, Daniel P. Berrangé wrote:
> AMD x86 CPUs have two separate ways to mitigate the Speculative Store
> Bypass hardware flaw. In current processors only non-architectural MSRs
> are available, and so hypervisors must expose a virtualized MSR and CPU
> flag "vir
On Wed, Jun 13, 2018 at 12:48:37PM +0200, Erik Skultety wrote:
This binding allows to query the AMD's SEV firmware for various platform
specific things, like a PDH certificate and a certificate chain to
establish a trusted connection with the firmware. Because the API uses
typed params, it's exem
On Wed, Jun 13, 2018 at 12:48:36PM +0200, Erik Skultety wrote:
Libvirt recently introduced support for getting launch security
parameters, most notably AMD SEV VM memory measurement. This API can't
be generated as it's using typed parameters which we need to allocate.
Signed-off-by: Erik Skultet
[...]
>>>
>>> +/* The following members are for QEMU_AGENT_JOB_* */
>>> +qemuDomainAgentJob agentActive; /* Currently running agent job */
>>
>> Hmm... I seem to recall a recent patch that spoke against using an enum
>> type for struct fields:
>>
>> https://www.redhat.com/archives/li
On Thu, Jun 14, 2018 at 06:26:51PM +0800, Lily Zhu wrote:
For the 2 optional sub-elements - and of input device, make
the
order of them interchangeable in the XML.
They should be interchangeable even with the other sub-elements. In
other words, every sub-element of should be in a big
.
Ja
On 06/14/2018 12:10 AM, John Ferlan wrote:
>
>
> On 06/08/2018 09:45 AM, Michal Privoznik wrote:
>> The point is to break QEMU_JOB_* into smaller pieces which
>> enables us to achieve higher throughput. For instance, if there
>> are two threads, one is trying to query something on qemu
>> monitor
Signed-off-by: Erik Skultety
---
domain_capabilities.go | 7 +++
1 file changed, 7 insertions(+)
diff --git a/domain_capabilities.go b/domain_capabilities.go
index 3f5a752..4844fd9 100644
--- a/domain_capabilities.go
+++ b/domain_capabilities.go
@@ -106,6 +106,7 @@ type DomainCapsFeatures st
Signed-off-by: Erik Skultety
---
domain.go | 135 +-
1 file changed, 134 insertions(+), 1 deletion(-)
diff --git a/domain.go b/domain.go
index aeeb24a..27e2fdc 100644
--- a/domain.go
+++ b/domain.go
@@ -1863,6 +1863,15 @@ type DomainFea
*** BLURB HERE ***
Erik Skultety (2):
Add support for domain launch security
Add support for SEV in domain capabilities XML
domain.go | 135 -
domain_capabilities.go | 7 +++
2 files changed, 141 insertions(+), 1 deletion(-)
--
*** BLURB HERE ***
Erik Skultety (2):
Add support for AMD SEV launch security info
Add support for AMD SEV platform info
connect.go| 59 +++
connect_compat.go | 12 +++
connect_compat.h | 7 +++
domain.go | 41
Signed-off-by: Erik Skultety
---
domain.go| 41 +
domain_compat.go | 13 +
domain_compat.h | 6 ++
3 files changed, 60 insertions(+)
diff --git a/domain.go b/domain.go
index a3049aa..6499cfa 100644
--- a/domain.go
+++ b/domain.go
Signed-off-by: Erik Skultety
---
connect.go| 59 +++
connect_compat.go | 12 +++
connect_compat.h | 7 +++
3 files changed, 78 insertions(+)
diff --git a/connect.go b/connect.go
index e3e643e..8bb5fe6 100644
--- a/connect.
On 06/14/2018 12:11 AM, John Ferlan wrote:
>
>
> On 06/08/2018 09:45 AM, Michal Privoznik wrote:
>> These jobs can be used to mark job type over qemu agent.
>>
>> Signed-off-by: Michal Privoznik
>> ---
>> src/qemu/qemu_domain.c | 4 +++-
>> src/qemu/qemu_domain.h | 2 ++
>> 2 files changed, 5 i
On 06/14/2018 12:04 AM, John Ferlan wrote:
>
>
> On 06/08/2018 09:45 AM, Michal Privoznik wrote:
>> This enum will list all possible jobs for guest agent. The idea
>> is when a thread needs to talk to guest agent only it will take
>> the QEMU_AGENT_JOB instead of QEMU_JOB helping better
>> concur
On 06/14/2018 12:14 AM, John Ferlan wrote:
>
>
> On 06/08/2018 09:45 AM, Michal Privoznik wrote:
>> There are two sets of functions here:
>> 1) some functions talk on both monitor and agent monitor,
>> 2) some functions only talk on agent monitor.
>>
>> For functions from set 1) we need to use
>>
[...]
>>> diff --git a/include/libvirt/libvirt-domain.h
>>> b/include/libvirt/libvirt-domain.h
>>> index da773b76cb..1a1d34620d 100644
>>> --- a/include/libvirt/libvirt-domain.h
>>> +++ b/include/libvirt/libvirt-domain.h
>>> @@ -2055,6 +2055,7 @@ typedef enum {
>>> VIR_CONNECT_GET_ALL_DOMA
On Thu, Jun 14, 2018 at 04:31:57PM +0300, Mathieu Tarral wrote:
> Hi Daniel,
>
> Thanks for your help, i was able to run the libvirt daemon from master,
> and after add some debug messages, i understood that the call
> to virDomainIsActive was actually hanging from the client.
>
> It never reache
[...]
>>> qemuDomainObjPrivatePtr priv = obj->privateData;
>>> unsigned long long now;
>>> @@ -6395,12 +6400,18 @@ qemuDomainObjBeginJobInternal(virQEMUDriverPtr
>>> driver,
>>> }
>>>
>>> while (!nested && !qemuDomainNestedJobAllowed(priv, job)) {
>>> +if (instant)
>
On Tue, Jun 12, 2018 at 11:00:28AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StoragePool.xml | 9 +
src/storagepool.c| 41
tests/test_storage.py| 21
3 files
On Tue, Jun 12, 2018 at 11:00:27AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 25 +
tests/test_storage.py | 6 ++
3 files changed, 36 insertions(+)
Reviewe
On Tue, Jun 12, 2018 at 11:00:26AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 29 +
2 files changed, 35 insertions(+)
Reviewed-by: Ján Tomko
Jano
signature.asc
On 06/13/2018 09:42 PM, Laine Stump wrote:
> On 06/13/2018 07:16 AM, John Ferlan wrote:
>>
>> On 06/13/2018 04:15 AM, Ján Tomko wrote:
>>> On Tue, Jun 12, 2018 at 10:32:06AM -0400, John Ferlan wrote:
Add a check during qemuDomainAttachDeviceConfig whether the
new/incoming device would
On Tue, Jun 12, 2018 at 11:00:25AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 26 ++
2 files changed, 32 insertions(+)
+if (!storageVol)
+return;
+
+if (v
On Tue, Jun 12, 2018 at 11:00:25AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 26 ++
2 files changed, 32 insertions(+)
Reviewed-by: Ján Tomko
Jano
signature.asc
Des
Most probably this was a limitation in older ESX versions, and it seems
it does not exist anymore in more recent versions; see the following
thread:
https://www.redhat.com/archives/libvir-list/2018-May/msg02159.html
https://www.redhat.com/archives/libvir-list/2018-June/msg00043.html
Hence, allow a
On Tue, Jun 12, 2018 at 11:00:24AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 26 ++
2 files changed, 32 insertions(+)
Reviewed-by: Ján Tomko
Jano
signature.asc
Des
Hi Daniel,
Thanks for your help, i was able to run the libvirt daemon from master,
and after add some debug messages, i understood that the call
to virDomainIsActive was actually hanging from the client.
It never reached the daemon.
Looking back at the call again:
https://gist.github.com/Wenzel/
On Tue, Jun 12, 2018 at 11:00:21AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 30 ++
tests/test_connect.py| 14 ++
3 files changed, 50 insertions(+)
diff
On Tue, Jun 12, 2018 at 11:00:23AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 29 +
tests/test_connect.py| 1 +
3 files changed, 36 insertions(+)
diff --git a/src/c
On Tue, Jun 12, 2018 at 11:00:19AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 23 +++
tests/test_storage.py | 1 +
3 files changed, 29 insertions(+)
diff --git a/dat
On Tue, Jun 12, 2018 at 11:00:22AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StoragePool.xml | 6 ++
src/storagepool.c| 34 ++
2 files changed, 40 insertions(+)
diff --git a/src/storagepool.c b/src/stor
On Tue, Jun 12, 2018 at 11:00:21AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.Connect.xml | 6 ++
src/connect.c| 30 ++
tests/test_connect.py| 14 ++
3 files changed, 50 insertions(+)
Re
On Tue, Jun 12, 2018 at 11:00:20AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 6 ++
src/storagevol.c| 29 +
tests/test_storage.py | 7 +++
3 files changed, 42 insertions(+)
R
On Tue, Jun 12, 2018 at 11:00:19AM +0200, Katerina Koukiou wrote:
Signed-off-by: Katerina Koukiou
---
data/org.libvirt.StorageVol.xml | 5 +
src/storagevol.c| 23 +++
tests/test_storage.py | 1 +
3 files changed, 29 insertions(+)
Reviewed-by: J
On Tue, Jun 12, 2018 at 11:00:16AM +0200, Katerina Koukiou wrote:
> Signed-off-by: Katerina Koukiou
> ---
> data/org.libvirt.StoragePool.xml | 7 +++
> src/storagepool.c| 34 ++
> tests/libvirttest.py | 17 +
> tests
Now that the nwfilter driver keeps a list of bindings that it has
created, there is no need for the complex virt driver callbacks. It is
possible to simply iterate of the list of recorded filter bindings.
This means that rebuilding filters no longer has to acquire any locks on
the virDomainObj obj
Currently the nwfilter driver does not keep any record of what filter
bindings it has active. This means that when it needs to recreate
filters, it has to rely on triggering callbacks provided by the virt
drivers. This introduces a hash table recording the virNWFilterBinding
objects so the driver h
1 - 100 of 152 matches
Mail list logo