From 802f1260a84a3890595d247a1917dcba4ba7dbcd Mon Sep 17 00:00:00 2001 From: "Richard W.M. Jones" Date: Thu, 7 Dec 2017 09:07:28 +0000 Subject: [PATCH] Remove for libvirt >= 3.10. --- ...-using-shareable-for-appliance-disk-.patch | 47 +++++++++++++++++++ libguestfs.spec | 8 +++- 2 files changed, 54 insertions(+), 1 deletion(-) create mode 100644 0001-lib-libvirt-stop-using-shareable-for-appliance-disk-.patch diff --git a/0001-lib-libvirt-stop-using-shareable-for-appliance-disk-.patch b/0001-lib-libvirt-stop-using-shareable-for-appliance-disk-.patch new file mode 100644 index 0000000..30915bd --- /dev/null +++ b/0001-lib-libvirt-stop-using-shareable-for-appliance-disk-.patch @@ -0,0 +1,47 @@ +From 614f680dc67d98d16eb21a131cc43db24d08b758 Mon Sep 17 00:00:00 2001 +From: Pino Toscano +Date: Tue, 5 Dec 2017 15:36:34 +0100 +Subject: [PATCH] lib: libvirt: stop using for appliance disk + (RHBZ#1518517) + +Commit aa9e0057b19e29f76c9a81f9aebeeb1cb5bf1fdb made the libvirt backend +use for the disk of the appliance, since at that time all +the instances were using the disk directly. +OTOH, commit 3ad44c866042919374e2d840502e53da2ed8aef0 switched to +overlays for read-only disks, including the appliance, so effectively +protecting the appliance. + +Because of this, the libvirt backend does not need anymore. + +Thanks to: Daniel Berrange, Richard W.M. Jones, Peter Krempa. +--- + lib/launch-libvirt.c | 6 ------ + 1 file changed, 6 deletions(-) + +diff --git a/lib/launch-libvirt.c b/lib/launch-libvirt.c +index e5121799e..5aae1b611 100644 +--- a/lib/launch-libvirt.c ++++ b/lib/launch-libvirt.c +@@ -985,10 +985,6 @@ static int construct_libvirt_xml_appliance (guestfs_h *g, const struct libvirt_x + } \ + } while (0) + +-/* */ +-#define empty_element(element) \ +- do { start_element(element) {} end_element (); } while (0) +- + /* key=value attribute of the current element. */ + #define attribute(key,value) \ + if (xmlTextWriterWriteAttribute (xo, BAD_CAST (key), BAD_CAST (value)) == -1){ \ +@@ -1787,8 +1783,6 @@ construct_libvirt_xml_appliance (guestfs_h *g, + == -1) + return -1; + +- empty_element ("shareable"); +- + } end_element (); + + return 0; +-- +2.13.2 + diff --git a/libguestfs.spec b/libguestfs.spec index b1f8a65..f6abe24 100644 --- a/libguestfs.spec +++ b/libguestfs.spec @@ -36,7 +36,7 @@ Summary: Access and modify virtual machine disk images Name: libguestfs Epoch: 1 Version: 1.37.34 -Release: 3%{?dist} +Release: 4%{?dist} License: LGPLv2+ # Source and patches. @@ -63,6 +63,9 @@ Source7: libguestfs.keyring # Upstream patch to fix locking on NBD drives. Patch1: 0001-launch-direct-Omit-locking-option-for-non-file-disks.patch +# Upstream patch to remove for libvirt >= 3.10. +Patch2: 0001-lib-libvirt-stop-using-shareable-for-appliance-disk-.patch + # Basic build requirements for the library and virt tools. BuildRequires: gcc BuildRequires: supermin-devel >= 5.1.18 @@ -1394,6 +1397,9 @@ install -m 0644 utils/boot-benchmark/boot-benchmark.1 $RPM_BUILD_ROOT%{_mandir}/ %changelog +* Thu Dec 7 2017 Richard W.M. Jones - 1:1.37.34-4 +- Remove for libvirt >= 3.10. + * Sun Dec 3 2017 Richard W.M. Jones - 1:1.37.34-3 - Fix locking on NBD drives.