Re: [PATCH pve-ha-manager 6/7] api: status: add load imbalance to status

["Dominik Rusovac" <d.rusovac@proxmox.com>]

On Tue Apr 28, 2026 at 11:10 AM CEST, Daniel Kral wrote:
> On Mon Apr 27, 2026 at 3:20 PM CEST, Dominik Rusovac wrote:
>> This is a very basic measure to enable users to detect the prevailing
>> load imbalance in the UI, which atm reveals nothing about the latter.
>>
>> imo, enabling users to track how the load imbalance changed over time
>> (using RRD graphs, for example) should be considered, in the long run.
>>
>> Signed-off-by: Dominik Rusovac <d.rusovac@proxmox.com>
>> ---
>>  src/PVE/API2/HA/Status.pm | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/PVE/API2/HA/Status.pm b/src/PVE/API2/HA/Status.pm
>> index 4894f3b..acec78e 100644
>> --- a/src/PVE/API2/HA/Status.pm
>> +++ b/src/PVE/API2/HA/Status.pm
>> @@ -199,7 +199,9 @@ __PACKAGE__->register_method({
>>              }
>>              my $datacenter_config = eval { cfs_read_file('datacenter.cfg') } // {};
>>              if (my $crs = $datacenter_config->{crs}) {
>> -                $extra_status .= " - $crs->{ha} load CRS"
>> +                $extra_status .=
>> +                    " - $crs->{ha} load CRS "
>> +                    . sprintf("(load imbalance: %.2f", 100 * $status->{imbalance}) . "%)"
>>                      if $crs->{ha} && $crs->{ha} ne 'basic';
>
> I think this should also check whether the load balancing system is
> enabled to not clutter the status string if it's unused or no action is
> taken if this value is high, but no hard feelings.
>

good point, thx! will adjust the clause

>>              }
>>              my $time_str = localtime($status->{timestamp});