Hi John,
thank you for your comment.
As you mentioned this problem is solved (and now pushed) by Pavel. Thanks Pavel
for fixing this issue.
Kind regards,
Daniel
On 03.12.2014 15:29, John Ferlan wrote:
On 11/20/2014 05:08 AM, Daniel Hansel wrote:
For Intel and PowerPC the implementation
On 03.12.2014 17:39, Eric Blake wrote:
On 12/03/2014 05:56 AM, Erik Skultety wrote:
If someone removes blockcopy storage file when still in mirroring phase
That's fishy behavior on the part of the user. But since the patch is
simple enough, it's still worth protecting ourselves from stupid
On 12/03/2014 12:02 PM, Daniel P. Berrange wrote:
On Tue, Dec 02, 2014 at 12:08:30PM -0500, Laine Stump wrote:
(Everyone - see the request for opinions/ideas towards the bottom)
The idea behind these patches is the following:
1) most virtual machines only have a single MAC address behind
On 25.11.2014 18:09, Kyle DeFrancia wrote:
This resolves: https://bugzilla.redhat.com/show_bug.cgi?id=907779
A dhcp element can exist in only one IPv4 address and one IPv6
address per network. This patch enforces that in virNetworkUpdate.
---
src/conf/network_conf.c | 31
On 12/01/2014 10:59 AM, Cristian Klein wrote:
Qemu currently implements pre-copy live migration. VM memory pages are
first copied from the source hypervisor to the destination, potentially
multiple times as pages get dirtied during transfer, then VCPU state
is migrated. Unfortunately, if the
On Thu, Dec 04, 2014 at 03:56:53AM -0500, Laine Stump wrote:
On 12/03/2014 12:02 PM, Daniel P. Berrange wrote:
On Tue, Dec 02, 2014 at 12:08:30PM -0500, Laine Stump wrote:
(Everyone - see the request for opinions/ideas towards the bottom)
The idea behind these patches is the following:
On Thu, Dec 04, 2014 at 01:16:32PM +0800, Zhe Peng wrote:
Constants of GET_ALL_STATS_* incorrect,this patch fix them in Domain.pm
---
lib/Sys/Virt/Domain.pm | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
ACK
Regards,
Daniel
--
|: http://berrange.com -o-
On 2014-12-04 10:40, Laine Stump wrote:
On 12/01/2014 10:59 AM, Cristian Klein wrote:
Qemu currently implements pre-copy live migration. VM memory pages are
first copied from the source hypervisor to the destination, potentially
multiple times as pages get dirtied during transfer, then VCPU
On 14.11.2014 17:03, Conrad Meyer wrote:
Reboot requires more sophistication and is left as a future work item --
but at least part of the plumbing is in place.
---
Looking for feedback. I'm pretty unfamiliar with libvirt; maybe this isn't
quite the right way to do this. Sorry. If you want me
On 28.11.2014 15:36, Ian Campbell wrote:
On a Debian system lacking the w3c-dtd-xhtml package the build fails
with:
$ make -C docs/ formatcaps.html
make: Entering directory '/local/scratch/ianc/devel/libvirt.git/docs'
Generating formatcaps.html.tmp
I/O error
2014-11-28 16:40 GMT+03:00 Vasiliy Tolstov v.tols...@selfip.ru:
If user not specify script in network type ethernet, assume that user
needs simple tap device created with libvirt.
This patch does not need to run external script to create tap device or
add root to qemu process.
Any news. But
The apibuild.py script did not handle whitespace in preprocessor
macros, so it failed to detect constants declared with '# define'
instead of '#define'. Since we now correctly indent our public
header files, we have silently lost all constants from
libvirt-api.xml. This also caused us to not
On 04.12.2014 13:08, Daniel P. Berrange wrote:
The apibuild.py script did not handle whitespace in preprocessor
macros, so it failed to detect constants declared with '# define'
instead of '#define'. Since we now correctly indent our public
header files, we have silently lost all constants from
ping...
Tks,
John
On 11/26/2014 01:11 PM, John Ferlan wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1082521
Patch 1 is innocuous and perhaps could have been pushed as trivial...
For Patch 2 I wasn't sure if I should use the virSecuritySELinuxSetFilecon
or
On 12/04/2014 04:56 AM, Daniel P. Berrange wrote:
On Thu, Dec 04, 2014 at 03:56:53AM -0500, Laine Stump wrote:
Maybe the problem is that you're providing a good name based entirely on
the differences in behavior the guests can see, but what I'm looking for
is a way to control how that behavior
On 11/19/2014 05:50 AM, Wang Rui wrote:
Signed-off-by: Wang Rui moon.wang...@huawei.com
---
src/qemu/qemu_hotplug.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
index b9a0cee..1c75861 100644
---
On 11/19/2014 05:50 AM, Wang Rui wrote:
We can change vnc password by using virDomainUpdateDeviceFlags API with
live flag. But it can't be changed with config flag. Error is reported as
below.
error: Operation not supported: persistent update of device 'graphics' is not
supported
This
When user tries to insert element metadata providing a namespace
declaration as well, currently we insert the element without any validation
check for XML prefix (if provided). The next VM start would then
fail with parse error. This patch fixes this issue by adding a call to
xmlValidateNCName
Move entering the job into the thread to simplify the program flow. Also
as the code holds a separate reference to the domain object some
conditions can be simplified.
---
Notes:
Version 2:
- fix leak of 'data' when not reconnecting
src/qemu/qemu_process.c | 161
On Wed, Dec 03, 2014 at 15:58:45 +, Daniel Berrange wrote:
On Wed, Dec 03, 2014 at 04:52:25PM +0100, Jiri Denemark wrote:
Both qemu-img and its stripped down version qcow-create are searched for
dynamically with virFindFileInPath, functions using them report errors
when the required
On Thu, Dec 4, 2014 at 5:18 AM, Michal Privoznik mpriv...@redhat.com wrote:
On 14.11.2014 17:03, Conrad Meyer wrote:
+status = kev.data;
+if (WIFSIGNALED(status) WCOREDUMP(status)) {
+VIR_ERROR(Guest %s got signal %d and crashed,
mon-vm-def-name,
+
On Thu, Dec 04, 2014 at 02:40:46PM +0100, Peter Krempa wrote:
Move entering the job into the thread to simplify the program flow. Also
as the code holds a separate reference to the domain object some
conditions can be simplified.
---
Notes:
Version 2:
- fix leak of 'data' when not
On Thu, Dec 04, 2014 at 02:42:31PM +0100, Jiri Denemark wrote:
On Wed, Dec 03, 2014 at 15:58:45 +, Daniel Berrange wrote:
On Wed, Dec 03, 2014 at 04:52:25PM +0100, Jiri Denemark wrote:
Both qemu-img and its stripped down version qcow-create are searched for
dynamically with
Fix format of the secret XML in the example. The XML had an extraneousof
type='iscsi' (which is used by the disk definitions)
The world wide node name had a typo in the acronym (wwwn).
Signed-off-by: John Ferlan jfer...@redhat.com
---
docs/formatstorage.html.in | 4 ++--
1 file changed, 2
Add 3 new optional options for the pool-create-as and pool-define-as
command in order to define the 3 elements required in order to add
an auth element, such as:
auth type='chap' username='myuser'
secret usage='libvirtiscsi'/
/auth
Signed-off-by: John Ferlan jfer...@redhat.com
---
Properly format the options and provide meaningful descriptions for
the various options.
Signed-off-by: John Ferlan jfer...@redhat.com
---
tools/virsh.pod | 33 +++--
1 file changed, 27 insertions(+), 6 deletions(-)
diff --git a/tools/virsh.pod b/tools/virsh.pod
Add the optional adapter options for pool create/define. Results in
either:
adapter type='scsi_host' name='scsi_host2'/
or (on one line)
adapter type='fc_host' parent='scsi_host5'
wwnn='2000c9831b4b' wwpn='1000c9831b4b'/
being generated.
Signed-off-by: John Ferlan
Recently while reviewing some documentation for iSCSI authentication
description I found that virsh pool-{create|define}-as did not provide
a mechanism to add the auth and/or adapter options on the command
line - so I figured adding them would be simple. In the course of doing
this I also noted a
On 12/04/14 15:19, Martin Kletzander wrote:
On Thu, Dec 04, 2014 at 02:40:46PM +0100, Peter Krempa wrote:
Move entering the job into the thread to simplify the program flow. Also
as the code holds a separate reference to the domain object some
conditions can be simplified.
---
Notes:
3ecebf07110ca8d3413072557f29137943e848e3 breaks the build as it adds a
way to jump to cleanup before the 'cfg' object is retrieved and 'priv'
is initialized.
---
Notes:
Pushed under the build-breaker rule.
src/qemu/qemu_process.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
If probing capabilities via QMP fails, we now have a check
that prevents us falling back to -help parsing. Unfortunately
the error message
Failed to probe capabilities for /usr/bin/qemu-kvm:
unsupported configuration: QEMU 2.1.2 is too new for help parsing
is proving rather unhelpful to the
---
Pushed as trivial
docs/storage.html.in | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/docs/storage.html.in b/docs/storage.html.in
index 9933548..f920dae 100644
--- a/docs/storage.html.in
+++ b/docs/storage.html.in
@@ -551,12 +551,12 @@
On Thu, Dec 04, 2014 at 15:33:38 +, Daniel Berrange wrote:
If probing capabilities via QMP fails, we now have a check
that prevents us falling back to -help parsing. Unfortunately
the error message
Failed to probe capabilities for /usr/bin/qemu-kvm:
unsupported configuration: QEMU
A user asked me about setting max downtime prior to migration, which I thought
could be done similar to MigrateSetMaxSpeed. I was surprised to find that it
can only be set when migration is in progress
# virsh migrate-setmaxdowntime test 1000
error: Requested operation is not valid: domain is
On 12/03/2014 11:08 AM, Luyao Huang wrote:
When use qemuProcessAttach to attach a qemu process, cannot
get a right DAC label. Add a new func to get process label
via stat func. Do not remove virDomainDefGetSecurityLabelDef
before try to use stat to get process DAC label, because
There are
On 12/04/2014 05:09 AM, Cristian KLEIN wrote:
On 2014-12-04 10:40, Laine Stump wrote:
Currently, libvirt does create and activate simultaneously (and also
qemu does a gratuitous ARP request at some point, although I haven't
checked if it happens when qemu starts or when the guest CPUs are
If you remember correctly, about 1.2 years ago, we had...
https://www.redhat.com/archives/libvir-list/2013-September/msg00351.html
In short, the patch proposed a generalized mechanism for finding out
IP addresses, with flags for choosing method(s) (SNOOP|LEASE|AGENT).
But since not all methods
On 12/04/14 17:53, Jim Fehlig wrote:
A user asked me about setting max downtime prior to migration, which I
thought could be done similar to MigrateSetMaxSpeed. I was surprised to
find that it can only be set when migration is in progress
# virsh migrate-setmaxdowntime test 1000
error:
To ease debugging permission problems add uid/gid values to the debug
message when initializing a storage file backend.
---
src/storage/storage_backend_fs.c | 4 ++--
src/storage/storage_backend_gluster.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git
On 12/03/2014 12:24 PM, Michal Privoznik wrote:
In one of the previous commits (eafb53fe) we disallowed
network-wide bandwidth to some network types. However, we
forgot about portgroups/ which can have bandwidth/ too.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
For example on FreeBSD the yajl library is located at /usr/local/lib
and it's not in default LIBS and therefore the configure fails that
yajl not installed.
We can use the PKG_CHECK_MODULES to get the correct library path in
case the library provides pkg-config file definition, otherwise the old
On 12/04/2014 12:24 PM, Peter Krempa wrote:
To ease debugging permission problems add uid/gid values to the debug
message when initializing a storage file backend.
---
src/storage/storage_backend_fs.c | 4 ++--
src/storage/storage_backend_gluster.c | 4 ++--
2 files changed, 4
On 12/04/2014 01:30 PM, Pavel Hrdina wrote:
For example on FreeBSD the yajl library is located at /usr/local/lib
and it's not in default LIBS and therefore the configure fails that
yajl not installed.
We can use the PKG_CHECK_MODULES to get the correct library path in
case the library
This resolves: https://bugzilla.redhat.com/show_bug.cgi?id=907779
A dhcp element can exist in only one IPv4 address and one IPv6
address per network. This patch enforces that in virNetworkUpdate.
---
Rebased to latest master, hopefully this works better.
My original message:
On 12/04/2014 09:41 PM, Eric Blake wrote:
On 12/04/2014 01:30 PM, Pavel Hrdina wrote:
For example on FreeBSD the yajl library is located at /usr/local/lib
and it's not in default LIBS and therefore the configure fails that
yajl not installed.
We can use the PKG_CHECK_MODULES to get the correct
On 12/04/2014 03:36 PM, Pavel Hrdina wrote:
On 12/04/2014 09:41 PM, Eric Blake wrote:
On 12/04/2014 01:30 PM, Pavel Hrdina wrote:
For example on FreeBSD the yajl library is located at /usr/local/lib
and it's not in default LIBS and therefore the configure fails that
yajl not installed.
We
On 12/05/2014 12:58 AM, Ján Tomko wrote:
On 12/03/2014 11:08 AM, Luyao Huang wrote:
When use qemuProcessAttach to attach a qemu process, cannot
get a right DAC label. Add a new func to get process label
via stat func. Do not remove virDomainDefGetSecurityLabelDef
before try to use stat to get
On 12/04/2014 11:59 PM, Eric Blake wrote:
On 12/04/2014 03:36 PM, Pavel Hrdina wrote:
On 12/04/2014 09:41 PM, Eric Blake wrote:
On 12/04/2014 01:30 PM, Pavel Hrdina wrote:
For example on FreeBSD the yajl library is located at /usr/local/lib
and it's not in default LIBS and therefore the
48 matches
Mail list logo