[pve-devel] [RFC ha-manager 1/3] manager: support older versioning schema for ha group migration

[Daniel Kral <d.kral@proxmox.com>]

Prior to the Proxmox VE 8.x series, the pve-manager is versioned in the
format of `M.m-p`. Versions prior to Proxmox VE 8.x are already EOL'd,
but this change prevents any "Use of uninitialized value" log messages
if one of the nodes is on a version with such a versioning scheme.

Signed-off-by: Daniel Kral <d.kral@proxmox.com>
---
As discussed with @Thomas off-list, this might not be needed for
production use cases, but I stumbled on it while testing as I thought
pve-statd might still broadcast if someone runs a (unsupported) mixed
version cluster.

 src/PVE/HA/Manager.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PVE/HA/Manager.pm b/src/PVE/HA/Manager.pm
index def63ac0..9d7cb73f 100644
--- a/src/PVE/HA/Manager.pm
+++ b/src/PVE/HA/Manager.pm
@@ -535,7 +535,7 @@ my $have_groups_been_migrated = sub {
 my $get_version_parts = sub {
     my ($node_version) = @_;
 
-    return $node_version =~ m/^(\d+)\.(\d+)\.(\d+)(?:~(\d+))?/;
+    return $node_version =~ m/^(\d+)\.(\d+)(?:\.|-)(\d+)(?:~(\d+))?/;
 };
 
 my $has_node_min_version = sub {
-- 
2.47.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel