v2-v4 change:
Addressed Michal Privoznik's comments
Maxim Nestratov (7):
parallels: introduce and use string constants for network types and
names
parallels: fix parallelsLoadNetworks
parallels: better bridge network interface support
parallels: set correct network adapter device
Don't fail initialization of parallels driver if
parallelsLoadNetwork fails for optional networks.
This can happen when some of them are added manually
and configured incompletely. PCS requires only two networks
created automatically (named Host-Only and Bridged), others
are optional and their
and set adapter model specified in xml
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_driver.c |7 ++-
src/parallels/parallels_sdk.c| 15 +++
2 files changed, 21 insertions(+), 1 deletions(-)
diff --git
network adapter is used
---
src/parallels/parallels_sdk.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
index 3140757..1a448f1 100644
--- a/src/parallels/parallels_sdk.c
+++ b/src/parallels/parallels_sdk.c
which is on by default when a new VM/CT is created.
We should do this because this feature can't be controlled
by libvirt now and it sets up some iptables rules. So it's
better to do this to avoid potential conflict of different
set of rules or to avoid unexpected behavior.
Signed-off-by: Maxim
In order to support 'bridge' network adapters in parallels
driver we need to plug our veth devices into corresponding
linux bridges.
We are going to do this by reusing our abstraction of
Virtual Networks in terms of PCS. On a domain creation, we
create a new Virtual Network naming it with the same
when a new network adapter device is added
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_sdk.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
index 58c52a8..d699613
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_network.c |6 +++---
src/parallels/parallels_sdk.c |6 +++---
src/parallels/parallels_utils.h |8 +++-
3 files changed, 13 insertions(+), 7 deletions(-)
diff --git
On 17 March 2015 at 19:04, Daniel P. Berrange berra...@redhat.com wrote:
On Tue, Mar 17, 2015 at 06:52:31PM +, Dimitri John Ledkov wrote:
libvirt runs correctly without any configuration files, as sensible
defaults are used throughout. This commit introduces a layer for
nwfilter
Hi,
I am using libvirt version 1.2.9.2 on F21 and i am unable to get the
complete backing chain info in the virsh dumpxml output. Details below :
*My backing chain per qemu-img :*
[stack@devstack-f21 test]$ qemu-img info --backing-chain snap4.qcow2
image: snap4.qcow2
file format: qcow2
virtual
On 03/12/2015 04:10 PM, hujianwei wrote:
On 12/03/15 11:11, Jincheng Miao wrote:
create some test cases for libvirt-python API setTime and getTime
v2:
user should pass 'seconds' to set_guest_time, and 'nseconds' is
optional.
Jincheng Miao (2):
Add guest setTime and getTime testing
18.03.2015 10:20, Maxim Nestratov пишет:
v3 change:
Addressed Michal Privoznik's comments
Maxim Nestratov (7):
parallels: introduce and use string constants for network types and
names
parallels: fix parallelsLoadNetworks
parallels: better bridge network interface support
On Tue, Mar 17, 2015 at 11:57:20 -0600, Eric Blake wrote:
On 03/17/2015 06:39 AM, Jiri Denemark wrote:
Signed-off-by: Jiri Denemark jdene...@redhat.com
---
src/libvirt_private.syms | 1 +
src/util/vircgroup.c | 19 ++--
src/util/vircgrouppriv.h
Issue #1 - A call to virBitmapNew did not check if the allocation
failed which could lead to a NULL dereference
Issue #2 - When deleting the pin entries from the config file, the
code loops from the number of elements down to the new vcpu count;
however, the pin id values are numbered 0..n-1 not
On Wed, Mar 11, 2015 at 14:20:02 -0600, Jim Fehlig wrote:
Jiri Denemark wrote:
When libvirt is starting a domain, it reports the state as SHUTOFF until
it's RUNNING. This is not ideal because domain startup may take a long
time (usually because of some configuration issues, firewalls
On Wed, Mar 18, 2015 at 10:09:53AM +, Dimitri John Ledkov wrote:
On 17 March 2015 at 19:04, Daniel P. Berrange berra...@redhat.com wrote:
On Tue, Mar 17, 2015 at 06:52:31PM +, Dimitri John Ledkov wrote:
libvirt runs correctly without any configuration files, as sensible
defaults are
On 15.03.2015 18:15, swat wrote:
Updated the documentation for command cd
The documentation for command cd did not have information about the command.
Added the required information.
---
source/cd.xml | 36 +---
1 file changed, 33 insertions(+), 3
From: Nehal J Wani nehaljw.k...@gmail.com
examples/Makefile.am:
* Add new file domipaddrs.py
examples/README:
* Add documentation for the python example
libvirt-override-api.xml:
* Add new symbol for virDomainInterfacesAddresses
libvirt-override.c:
* Hand written python api
Example:
On 2015/3/13 17:17, zhang bo wrote:
Problem Description:
When we set boot order for a vhost-user network interface, we found the boot
index
doesn't work.
Cause of the Problem:
In the function qemuBuildVhostuserCommandLine(), it forcely set the arg
bootindex of
function
On 03/17/2015 11:49 AM, lcheng wrote:
The connection_security_model.py uses getSecurityModel() to validate new API
virNodeGetSecurityModel of libvirt.
---
cases/linux_domain.conf| 4 ++
repos/virconn/connection_security_model.py | 101 +
2
When parsing integer values, we only used 'long' data type
in the virConfValue struct. This is insufficiently large
to deal with things like guest memory sizes on 32-bit platforms
which are using PAE for addressing 4 GB of RAM.
---
daemon/libvirtd-config.c | 6 +--
On 03/18/2015 03:47 AM, Deepak Shetty wrote:
I spent quite some time figuring that backingStore info
isn't included in the dom xml, unless guest is up and
running. Hopefully putting that in the doc should help.
This patch adds that info to the doc.
Signed-off-by: Deepak C Shetty
Currently the QEMU processes inherit their core dump rlimit
from libvirtd, which is really suboptimal. This change allows
their limit to be directly controller from qemu.conf instead.
---
src/libvirt_private.syms | 2 ++
src/qemu/libvirtd_qemu.aug | 1 +
src/qemu/qemu.conf
Currently QEMU inherits core size limits from libvirtd which
is rather inconvenient
Daniel P. Berrange (2):
conf: parse integers into long long, instead of long
qemu: add a max_core setting to qemu.conf for core dump size
daemon/libvirtd-config.c | 6 +--
On 03/17/2015 05:14 PM, Michal Privoznik wrote:
On 13.03.2015 16:52, Maxim Nestratov wrote:
network adapter is used
---
src/parallels/parallels_sdk.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
index
- Original Message -
| From: Deepak Shetty dpkshe...@gmail.com
| To: libvir-list@redhat.com
| Sent: Wednesday, March 18, 2015 7:19:05 PM
| Subject: [libvirt] Domain XML isn't dumping full backing chain
| Hi,
| I am using libvirt version 1.2.9.2 on F21 and i am unable to get the complete
On Wed, Mar 18, 2015 at 07:25:29AM -0400, John Ferlan wrote:
Issue #1 - A call to virBitmapNew did not check if the allocation
failed which could lead to a NULL dereference
Issue #2 - When deleting the pin entries from the config file, the
code loops from the number of elements down to the new
Hmm interesting, so without adding backing_fmt, if i enable the below in
/etc/libvirt/qemu.conf, will it work ?
# If allow_disk_format_probing is enabled, libvirt will probe disk
# images to attempt to identify their format, when not otherwise
# specified in the XML. This is disabled by default.
On 03/18/2015 07:03 AM, Deepak Shetty wrote:
[please don't top-post on technical lists]
Hmm interesting, so without adding backing_fmt, if i enable the below in
/etc/libvirt/qemu.conf, will it work ?
# If allow_disk_format_probing is enabled, libvirt will probe disk
# images to attempt to
The folder vircgroupdata was introduced in commit 2dbfa716 as part of
the test suite for virCgroupDetectMounts
---
tests/Makefile.am | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 9277c13..27cee63 100644
--- a/tests/Makefile.am
+++
On Wed, Mar 18, 2015 at 07:21:22AM -0600, Eric Blake wrote:
On 03/18/2015 07:03 AM, Deepak Shetty wrote:
[please don't top-post on technical lists]
Hmm interesting, so without adding backing_fmt, if i enable the below in
/etc/libvirt/qemu.conf, will it work ?
# If
My commit 2dbfa71 added test data for vircgrouptest but forgot to
distribute the new directory.
Signed-off-by: Jiri Denemark jdene...@redhat.com
---
Pushed as a build-breaker.
tests/Makefile.am | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/Makefile.am b/tests/Makefile.am
index
On 18.03.2015 09:33, Maxim Nestratov wrote:
Don't fail initialization of parallels driver if
parallelsLoadNetwork fails for optional networks.
This can happen when some of them are added manually
and configured incompletely. PCS requires only two networks
created automatically (named
On 18.03.2015 09:33, Maxim Nestratov wrote:
In order to support 'bridge' network adapters in parallels
driver we need to plug our veth devices into corresponding
linux bridges.
We are going to do this by reusing our abstraction of
Virtual Networks in terms of PCS. On a domain creation, we
On 18.03.2015 09:33, Maxim Nestratov wrote:
and set adapter model specified in xml
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_driver.c |7 ++-
src/parallels/parallels_sdk.c| 15 +++
2 files changed, 21 insertions(+), 1
On 13.03.2015 10:17, zhang bo wrote:
Problem Description:
When we set boot order for a vhost-user network interface, we found the boot
index
doesn't work.
Cause of the Problem:
In the function qemuBuildVhostuserCommandLine(), it forcely set the arg
bootindex of
function
Investigation of a problem with creating passthrough macvtap devices
(https://bugzilla.redhat.com/show_bug.cgi?id=1185501) has shown that
this slightly more verbose failure message is useful. In particular,
the mac address can be used to determine the domain. You could also
figure this out by
Hi Michal,
We're testing libvirt-1.2.13 and found it failed to live migrate domains
with attached network drives. The change to libvirt was made in commit
cf54c606, announced here:
https://www.redhat.com/archives/libvir-list/2014-November/msg01053.html
It's not necessary to precreate network
v3 change:
Addressed Michal Privoznik's comments
Maxim Nestratov (7):
parallels: introduce and use string constants for network types and
names
parallels: fix parallelsLoadNetworks
parallels: better bridge network interface support
parallels: set correct network adapter device state
Don't fail initialization of parallels driver if
parallelsLoadNetwork fails for optional networks.
This can happen when some of them are added manually
and configured incompletely. PCS requires only two networks
created automatically (named Host-Only and Bridged), others
are optional and their
In order to support 'bridge' network adapters in parallels
driver we need to plug our veth devices into corresponding
linux bridges.
We are going to do this by reusing our abstraction of
Virtual Networks in terms of PCS. On a domain creation, we
create a new Virtual Network naming it with the same
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_network.c |6 +++---
src/parallels/parallels_sdk.c |6 +++---
src/parallels/parallels_utils.h |8 +++-
3 files changed, 13 insertions(+), 7 deletions(-)
diff --git
when a new network adapter device is added
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_sdk.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
index 58c52a8..d699613
network adapter is used
---
src/parallels/parallels_sdk.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
index 465cc41..fb75d2d 100644
--- a/src/parallels/parallels_sdk.c
+++ b/src/parallels/parallels_sdk.c
and set adapter model specified in xml
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_driver.c |7 ++-
src/parallels/parallels_sdk.c| 17 +
2 files changed, 23 insertions(+), 1 deletions(-)
diff --git
which is on by default when a new VM/CT is created.
We should do this because this feature can't be controlled
by libvirt now and it sets up some iptables rules. So it's
better to do this to avoid potential conflict of different
set of rules or to avoid unexpected behavior.
Signed-off-by: Maxim
18.03.2015 10:20, Maxim Nestratov пишет:
and set adapter model specified in xml
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_driver.c |7 ++-
src/parallels/parallels_sdk.c| 17 +
2 files changed, 23 insertions(+), 1
xen.git commit babeca32 added a pkgconfig file for libxenlight,
allowing libxl apps to determine the location of Xen binaries
such as firmware blobs, device emulator, etc.
This patch adds support for xenlight.pc in the libxl driver, falling
back to the previous configure logic if not found. It
Actually I messed that up slightly on the case. My C is a bit rusty:
diff -U3 -r libvirt-1.2.13.orig/src/qemu/qemu_migration.c
libvirt-1.2.13/src/qemu/qemu_migration.c
--- libvirt-1.2.13.orig/src/qemu/qemu_migration.c 2015-02-23
22:04:12.0 -0800
+++
When converting domXML from native, the libxl driver was overwriting
useful errors from the xenconfig parsing code with a useless, generic
error. E.g. internal error: parsing xm config failed vs
internal error: config value usbdevice was malformed. Remove the
redundant (and useless) error
On 03/18/2015 04:21 PM, Jim Fehlig wrote:
When converting domXML from native, the libxl driver was overwriting
useful errors from the xenconfig parsing code with a useless, generic
error. E.g. internal error: parsing xm config failed vs
internal error: config value usbdevice was malformed.
https://bugzilla.redhat.com/show_bug.cgi?id=1196934
When start/migrate/restore a vm failed, libvirt will try to catch the log in
/var/log/libvirt/qemu/vm.log and output them before. However we add a check in
qemuDomainObjExitMonitor after commit dc2fd51f, this will overwrite
the error set in
Eric Blake wrote:
On 03/18/2015 04:21 PM, Jim Fehlig wrote:
When converting domXML from native, the libxl driver was overwriting
useful errors from the xenconfig parsing code with a useless, generic
error. E.g. internal error: parsing xm config failed vs
internal error: config value
On 18.03.2015 19:30, Laine Stump wrote:
Investigation of a problem with creating passthrough macvtap devices
(https://bugzilla.redhat.com/show_bug.cgi?id=1185501) has shown that
this slightly more verbose failure message is useful. In particular,
the mac address can be used to determine the
On 18.03.2015 09:33, Maxim Nestratov wrote:
v2-v4 change:
Addressed Michal Privoznik's comments
Maxim Nestratov (7):
parallels: introduce and use string constants for network types and
names
parallels: fix parallelsLoadNetworks
parallels: better bridge network interface
On 18.03.2015 09:33, Maxim Nestratov wrote:
when a new network adapter device is added
s/device/link/ in $SUBJ.
Signed-off-by: Maxim Nestratov mnestra...@parallels.com
---
src/parallels/parallels_sdk.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git
18.03.2015 20:34, Michal Privoznik пишет:
I've fixed all the nits I've found, ACKed and pushed whole series.
Thank you!
One more thing though, look at the rest of our commit messages. They
usually consist of small subject, and start freshly with new sentence.
For some reason we don't like if
On 03/18/2015 08:04 AM, Nehal J Wani wrote:
The folder vircgroupdata was introduced in commit 2dbfa716 as part of
the test suite for virCgroupDetectMounts
---
tests/Makefile.am | 1 +
1 file changed, 1 insertion(+)
NACK; it was already added several lines later (after
virsh-uriprecedence)
The documentation for command cd did not have information about the command.
Added the required information.
---
source/cd.xml | 38 ++
1 file changed, 34 insertions(+), 4 deletions(-)
diff --git a/source/cd.xml b/source/cd.xml
index 78a13df..d15c74c 100644
On 03/18/2015 07:48 AM, Deepak Shetty wrote:
I spent quite some time figuring that backingStore info
isn't included in the dom xml, unless guest is up and
running. Hopefully putting that in the doc should help.
This patch adds that info to the doc.
Signed-off-by: Deepak C Shetty
On 03/09/2015 03:07 AM, Jiri Denemark wrote:
On Fri, Mar 06, 2015 at 09:34:59 -0700, Eric Blake wrote:
In qemu 2.3, the migration status will include 'cancelling' in the
window between when an asynchronous cancel has been requested and
when the migration is actually halted. Previously, qemu
61 matches
Mail list logo