- Devel - libvirt List Archives

[libvirt] [PATCH 0/2] tools: Fix usage of incorrect environment variables
by Erik Skultety 28 Jul '16

28 Jul '16

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

2 4

[libvirt] logging: Libvirtd could not create its own logfile, when the path /var/log/libvirt is not exist.
by weifuqiang 28 Jul '16

28 Jul '16

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

2 1

[libvirt] [PATCH 0/2] Couple of build fixes
by Michal Privoznik 28 Jul '16

28 Jul '16

Contradictory to my previous patch set [1], these are not pushed yet and thus require proper review. 1: https://www.redhat.com/archives/libvir-list/2016-July/msg01147.html Michal Privoznik (2): vshCmddefGetOption: Change type of opt_index vshReadlineParse: Drop some unused variables tools/vsh.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) -- 2.8.4

2 3

[libvirt] [PATCH] Fix RPM spec for wireshark on Fedora < 24
by Daniel P. Berrange 28 Jul '16

28 Jul '16

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

1 0

[libvirt] [PATCH 0/3] libxl hooks
by Cédric Bosdonnat 28 Jul '16

28 Jul '16

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

3 7

[libvirt] [PATCH] qemu: remove panic device with model s390 when migrating
by Boris Fiuczynski 28 Jul '16

28 Jul '16

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

3 3

[libvirt] [libvirt RFC PATCH 00/10] Add support for qemu's JSON pseudo'protocol for backing store
by Peter Krempa 28 Jul '16

28 Jul '16

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

3 18

[libvirt] [PATCH 0/2] Two wireshark fixes
by Michal Privoznik 28 Jul '16

28 Jul '16

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

2 3

[libvirt] [PATCH 0/2] Updates for LUKS support
by Daniel P. Berrange 28 Jul '16

28 Jul '16

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

2 9

[libvirt] [PATCH libvirt-glib] docs: Document gvir_connection_get_{storage_pools, networks, domains}
by Guido Günther 28 Jul '16

28 Jul '16

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

2 2