From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id CC0871FF138 for ; Wed, 18 Feb 2026 12:14:39 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 95AB11AC7C; Wed, 18 Feb 2026 12:15:37 +0100 (CET) From: Maximiliano Sandoval To: Daniel Kral Subject: Re: [RFC cluster 2/2] datacenter config: add auto rebalancing options In-Reply-To: <20260217141437.584852-14-d.kral@proxmox.com> (Daniel Kral's message of "Tue, 17 Feb 2026 15:14:07 +0100") References: <20260217141437.584852-1-d.kral@proxmox.com> <20260217141437.584852-14-d.kral@proxmox.com> User-Agent: mu4e 1.12.9; emacs 30.1 Date: Wed, 18 Feb 2026 12:15:32 +0100 Message-ID: MIME-Version: 1.0 Content-Type: text/plain X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1771413324925 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.087 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: 4IUDEGJSBYHW2RVSUHPOG74P4WUTWFMY X-Message-ID-Hash: 4IUDEGJSBYHW2RVSUHPOG74P4WUTWFMY X-MailFrom: m.sandoval@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: pve-devel@lists.proxmox.com X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Daniel Kral writes: > Signed-off-by: Daniel Kral > --- > src/PVE/DataCenterConfig.pm | 38 +++++++++++++++++++++++++++++++++++++ > 1 file changed, 38 insertions(+) > > diff --git a/src/PVE/DataCenterConfig.pm b/src/PVE/DataCenterConfig.pm > index 5c91f80..86bd06a 100644 > --- a/src/PVE/DataCenterConfig.pm > +++ b/src/PVE/DataCenterConfig.pm > @@ -30,6 +30,44 @@ my $crs_format = { > "Set to use CRS for selecting a suited node when a HA services request-state" > . " changes from stop to start.", > }, > + 'ha-auto-rebalance' => { > + type => 'boolean', > + optional => 1, > + default => 0, > + description => "Set to use CRS for balancing HA resources automatically depending on" > + . " the current node imbalance.", For boolean parameters I would personally prefer something like "Whether to use..." or "If true, uses...", similar for the rest. > + 'ha-auto-rebalance-threshold' => { > + type => 'number', > + optional => 1, > + default => 0.7, > + requires => 'ha-auto-rebalance', > + description => "The threshold for the node load, which will trigger the automatic" > + . " HA resource balancing if the threshold is exceeded.", > + }, > + 'ha-auto-rebalance-method' => { > + type => 'string', > + enum => ['bruteforce', 'topsis'], > + optional => 1, > + default => 'bruteforce', > + requires => 'ha-auto-rebalance', > + }, > + 'ha-auto-rebalance-hold-duration' => { > + type => 'number', > + optional => 1, > + default => 3, > + requires => 'ha-auto-rebalance', > + description => "The duration the threshold must be exceeded for to trigger an automatic" > + . " HA resource balancing migration in HA rounds.", > + }, > + 'ha-auto-rebalance-margin' => { > + type => 'number', > + optional => 1, > + default => 0.1, > + requires => 'ha-auto-rebalance', > + description => "The minimum relative improvement in cluster node imbalance to commit to" > + . " a HA resource rebalancing migration.", > + }, > }; > > my $migration_format = { -- Maximiliano