[libvirt] [PATCH 0/2] tools: Fix usage of incorrect environment variables
by Erik Skultety
According to https://bugzilla.redhat.com/show_bug.cgi?id=1357363, most of
virt-admin's environment variables are not taken into effect when set. As it
turned out, virsh suffers from the very same problem which resides in the
generic vsh.c module.
Erik Skultety (2):
vsh: Make vshInitDebug return int instead of void
tools: Make use of the correct environment variables
tools/virsh.c | 1 +
tools/virt-admin.c | 1 +
tools/vsh.c | 34 +++++++++++++++++++---------------
tools/vsh.h | 1 +
4 files changed, 22 insertions(+), 15 deletions(-)
--
2.5.5
8 years, 4 months
[libvirt] logging: Libvirtd could not create its own logfile, when the path /var/log/libvirt is not exist.
by weifuqiang
Hi all:
Libvirtd could not create its own logfile, such as /var/log/libvirt/libvirtd.log, when the path /var/log/libvirt is not exist.
It may be a problem,I think,so try to make patch when open file failed in virLogAddOutputToFile.
or any better idea?
------------
Libvirtd just creates its logfile, such as /var/log/libvirt/libvirtd.log, DURING INSTALLATION. If the logfile(we mean its path here,that's /var/log/libvirt/) is deleted during its runtime, even if we restart libvirtd, such file would not be generated automatically, thus, the logs are all missing after then, it becomes impossible to debug problems after then as well. So, we try to mkdir the deleted path after restarting libvirtd.
Or, any better idea?
Thanks in advance.
Wei Fuqiang
8 years, 4 months
[libvirt] [PATCH] Fix RPM spec for wireshark on Fedora < 24
by Daniel P. Berrange
This previous commit
commit cd9fcc8be7dcb9126d70f744ce54b0b742eeefb8
Author: Michal Privoznik <mprivozn(a)redhat.com>
Date: Wed Jul 27 16:58:32 2016 +0200
libvirt.spec.in: Adapt to newest wireshark plugindir
Adapted the libvirt spec for wireshark >= 2.1.0 but
this ignored the fact that we enable wireshark from
Fedora 21 and 2.1.0 was only added in Fedora 24
Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com>
---
libvirt.spec.in | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/libvirt.spec.in b/libvirt.spec.in
index 35c7449..5841f45 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -406,7 +406,11 @@ BuildRequires: numad
%endif
%if %{with_wireshark}
+%if 0%{fedora} >= 24
BuildRequires: wireshark-devel >= 2.1.0
+%else
+BuildRequires: wireshark-devel >= 1.12.1
+%endif
%endif
Provides: bundled(gnulib)
@@ -1212,7 +1216,13 @@ rm -f $RPM_BUILD_ROOT%{_libdir}/libvirt/lock-driver/*.a
rm -f $RPM_BUILD_ROOT%{_libdir}/libvirt/connection-driver/*.la
rm -f $RPM_BUILD_ROOT%{_libdir}/libvirt/connection-driver/*.a
%if %{with_wireshark}
+%if 0%{fedora} >= 24
rm -f $RPM_BUILD_ROOT%{_libdir}/wireshark/plugins/libvirt.la
+%else
+rm -f $RPM_BUILD_ROOT%{_libdir}/wireshark/plugins/*/libvirt.la
+mv $RPM_BUILD_ROOT%{_libdir}/wireshark/plugins/*/libvirt.so \
+ $RPM_BUILD_ROOT%{_libdir}/wireshark/plugins/libvirt.so
+%endif
%endif
install -d -m 0755 $RPM_BUILD_ROOT%{_datadir}/lib/libvirt/dnsmasq/
--
2.5.5
8 years, 4 months
[libvirt] [PATCH 0/3] libxl hooks
by Cédric Bosdonnat
Hi there!
Here is small patchset adding hooks support to the libxl driver.
Cédric Bosdonnat (3):
libxl: add a flag to mark guests as tainted by a hook
libxl: fix segfault in libxlReconnectDomain
libxl: add hooks support
docs/hooks.html.in | 53 ++++++++++++++++++++++++++--
src/libxl/libxl_domain.c | 85 +++++++++++++++++++++++++++++++++++++++++++++
src/libxl/libxl_domain.h | 2 ++
src/libxl/libxl_driver.c | 52 ++++++++++++++++++++-------
src/libxl/libxl_migration.c | 58 +++++++++++++++++++++++++++++++
src/util/virhook.c | 16 ++++++++-
src/util/virhook.h | 13 +++++++
7 files changed, 264 insertions(+), 15 deletions(-)
--
2.6.6
8 years, 4 months
[libvirt] [PATCH] qemu: remove panic device with model s390 when migrating
by Boris Fiuczynski
The panic device with model s390 is autogenerated on domains with
S390 architecture since libvirt version 1.3.5.
For backwards compatibility reasons the device is to be removed
when migrating.
Signed-off-by: Boris Fiuczynski <fiuczy(a)linux.vnet.ibm.com>
---
src/qemu/qemu_domain.c | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index ceac22f..0d97ea6 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -3310,6 +3310,28 @@ qemuDomainDefFormatBuf(virQEMUDriverPtr driver,
virDomainControllerDefFree(usb);
}
+ /* Remove the panic device with model s390 if present */
+ for (i = 0; i < def->npanics; i++) {
+ if (def->panics[i]->model == VIR_DOMAIN_PANIC_MODEL_S390) {
+ virDomainPanicDefPtr *panics = def->panics;
+ int npanics = def->npanics;
+ size_t j;
+
+ if (VIR_ALLOC_N(def->panics, npanics - 1) < 0) {
+ def->panics = panics;
+ goto cleanup;
+ }
+
+ def->npanics = 0;
+ for (j = 0; j < npanics; j++) {
+ if (j != i)
+ def->panics[def->npanics++] = panics[j];
+ }
+ VIR_FREE(panics);
+ break; /* only one panic dev with model s390 can exist */
+ }
+ }
+
for (i = 0; i < def->nchannels; i++)
qemuDomainChrDefDropDefaultPath(def->channels[i], driver);
}
--
2.9.0
8 years, 4 months
[libvirt] [libvirt RFC PATCH 00/10] Add support for qemu's JSON pseudo'protocol for backing store
by Peter Krempa
Libvirt didn't handle this for a long time and VMs with such config would not
start we should implement it.
Using JSON is basically the only option to specify advanced configuration for a
backing file.
Field names were taken from qemu's source since there isn't really
documentation for this.
CCing qemu-block for possible comments on patches 2-8,10.
Peter Krempa (10):
tests: Add testing of backing store string parser
util: storage: Add parser for qemu's "json" backing pseudo-protocol
util: storage: Add support for host device backing specified via JSON
util: storage: Add support for URI based backing volumes in qemu's
JSON pseudo-protocol
util: storage: Add json pseudo protocol support for gluster volumes
util: storage: Add json pseudo protocol support for iSCSI volumes
Add JSON backing volume parser for 'nbd' protocol
util: storage: Add JSON backing store parser for 'sheepdog' protocol
util: storage: Add 'ssh' network storage protocol
util: storage: Add JSON backing volume parser for 'ssh' protocol
src/libvirt_private.syms | 1 +
src/libxl/libxl_conf.c | 1 +
src/qemu/qemu_command.c | 7 +
src/qemu/qemu_driver.c | 3 +
src/qemu/qemu_parse_command.c | 1 +
src/util/virstoragefile.c | 288 ++++++++++++++++++++++++++++++++++++++++--
src/util/virstoragefile.h | 4 +
src/xenconfig/xen_xl.c | 1 +
tests/virstoragetest.c | 128 +++++++++++++++++++
9 files changed, 425 insertions(+), 9 deletions(-)
--
2.8.2
8 years, 4 months
[libvirt] [PATCH 0/2] Two wireshark fixes
by Michal Privoznik
While trying to do an RPM, I've ran into two problems. I've
already pushed the fixes under 'build breaker rule'. I'm sending
them for completeness.
Michal Privoznik (2):
virt-wireshark: Properly substract wireshark prefix
libvirt.spec.in: Adapt to newest wireshark plugindir
libvirt.spec.in | 6 ++----
m4/virt-wireshark.m4 | 2 +-
2 files changed, 3 insertions(+), 5 deletions(-)
--
2.8.4
8 years, 4 months
[libvirt] [PATCH 0/2] Updates for LUKS support
by Daniel P. Berrange
Daniel P. Berrange (2):
virstoragefile: refactor virStorageFileMatchesNNN methods
storage: remove "luks" storage volume type
src/qemu/qemu_command.c | 10 +-
src/qemu/qemu_domain.c | 2 +-
src/qemu/qemu_hotplug.c | 2 +-
src/storage/storage_backend.c | 41 ++--
src/storage/storage_backend_fs.c | 17 +-
src/storage/storage_backend_gluster.c | 5 -
src/util/virstoragefile.c | 265 +++++++++++++++------
src/util/virstoragefile.h | 1 -
tests/qemuxml2argvdata/qemuxml2argv-luks-disks.xml | 4 +-
tests/storagevolxml2xmlin/vol-luks-cipher.xml | 2 +-
tests/storagevolxml2xmlin/vol-luks.xml | 2 +-
tests/storagevolxml2xmlout/vol-luks-cipher.xml | 2 +-
tests/storagevolxml2xmlout/vol-luks.xml | 2 +-
13 files changed, 235 insertions(+), 120 deletions(-)
--
2.7.4
8 years, 4 months
[libvirt] [PATCH libvirt-glib] docs: Document gvir_connection_get_{storage_pools, networks, domains}
by Guido Günther
In contrast to libvirt itself all get_* methods need to prefetch the
corresponding information first so document this.
---
libvirt-gobject/libvirt-gobject-connection.c | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/libvirt-gobject/libvirt-gobject-connection.c b/libvirt-gobject/libvirt-gobject-connection.c
index 00d5eda..3f17265 100644
--- a/libvirt-gobject/libvirt-gobject-connection.c
+++ b/libvirt-gobject/libvirt-gobject-connection.c
@@ -700,6 +700,11 @@ void gvir_connection_close(GVirConnection *conn)
* gvir_connection_fetch_domains:
* @conn: a #GVirConnection
* @cancellable: (allow-none)(transfer none): cancellation object
+ *
+ * Use this method to fetch all domains managed by connection
+ * @conn. Use e.g. #gvir_connection_find_domain_by_id or
+ * #gvir_connection_get_domain afterwards to query the fetched
+ * domains.
*/
gboolean gvir_connection_fetch_domains(GVirConnection *conn,
GCancellable *cancellable,
@@ -783,6 +788,11 @@ cleanup:
* gvir_connection_fetch_storage_pools:
* @conn: a #GVirConnection
* @cancellable: (allow-none)(transfer none): cancellation object
+ *
+ * Use this method to fetch all storage pools managed by connection
+ * @conn. Use e.g. #gvir_connection_find_storage_pool_by_name or
+ * #gvir_connection_get_storage_pools afterwards to query the fetched
+ * pools.
*/
gboolean gvir_connection_fetch_storage_pools(GVirConnection *conn,
GCancellable *cancellable,
@@ -1722,6 +1732,11 @@ GVirInterface *gvir_connection_find_interface_by_mac(GVirConnection *conn,
* gvir_connection_fetch_networks:
* @conn: a #GVirConnection
* @cancellable: (allow-none)(transfer none): cancellation object
+ *
+ * Use this method to fetch all networks managed by connection
+ * @conn. Use e.g. #gvir_connection_find_network_by_name or
+ * #gvir_connection_get_networks afterwards to query the fetched
+ * domains.
*/
gboolean gvir_connection_fetch_networks(GVirConnection *conn,
GCancellable *cancellable,
--
2.8.1
8 years, 4 months
