[pve-devel] [PATCH manager] pve5to6: Add warning for some Gluster versions

Dominic Jäger d.jaeger at proxmox.com
Thu Aug 1 11:21:27 CEST 2019


Upgrading from PVE 5 to 6 works only with the Gluster packages from the
Debian repository. At least Gluster versions 5.8 to 6.4-1 have conflicts
with our packages.

Signed-off-by: Dominic Jäger <d.jaeger at proxmox.com>
---
Tested on one VM with PVE 5.4 and upstream Gluster (6.4-1) and one
with PVE 5.4 and the Debian Gluster packages (3.8). The upgrade gives
the expected conflicts. The error message remains useful after the
(erroneous) upgrade.

 PVE/CLI/pve5to6.pm | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/PVE/CLI/pve5to6.pm b/PVE/CLI/pve5to6.pm
index 7b1050e3..7748b5e8 100644
--- a/PVE/CLI/pve5to6.pm
+++ b/PVE/CLI/pve5to6.pm
@@ -220,6 +220,23 @@ sub get_vms_with_vmx {
     return $res;
 }
 
+# The upgrade works if Gluster 5.5-3 is installed . This is packaged in Debian.
+# There are conflicts with more recent versions (from gluster.org) of Gluster.
+# Consequently, warn if gluster is not installed from debian packages but
+# from the gluster repos. See bug 2269.
+sub check_gluster {
+    log_info("Checking GlusterFS version.");
+    my $stdout = `dpkg -s glusterfs-common`;
+    # Patrick Matthäi is the maintainer of the Gluster Debian package
+    # The package from the Gluster repo has 'Gluster Packager' as maintainer
+    my $debian_maintainer = 'Maintainer: Patrick Matthäi';
+    if (index($stdout, $debian_maintainer) == -1) {
+	log_warn('Your version of GlusterFS is not maintained by Debian. Beware that there is a problem with PVE 6 and Gluster packages that are from a different source. See https://bugzilla.proxmox.com/show_bug.cgi?id=2269.');
+    } else {
+	log_pass("Your GlusterFS version is supported.");
+    }
+}
+
 sub check_kvm_nested {
     log_info("Checking KVM nesting support, which breaks live migration for VMs using it..");
 
@@ -664,6 +681,7 @@ sub check_misc {
     }
 
     check_kvm_nested();
+    check_gluster();
 }
 
 __PACKAGE__->register_method ({
-- 
2.20.1




More information about the pve-devel mailing list