Re: [RFC ha-manager 17/21] usage: add dynamic usage scheduler

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

On Tue, 17 Feb 2026, Daniel Kral wrote:
> --- a/src/PVE/HA/Usage/Dynamic.pm
> +++ b/src/PVE/HA/Usage/Dynamic.pm
> @@ -0,0 +1,99 @@
> [...]
> +        ->log('err', "unable to score nodes according to dynamic usage for service '$sid' - $@");

Missing `if $@` guard here, so this logs an error on every successful
call AFAICT?

Seems to get fixed in patch 20/21, but that means patches 17-19 spam the
log on every service placement, while we do not plan to partially apply
this, it still should be fixed in this commit.

btw. the error message also says "static" in get_service_usage, fixed to
"dynamic" also only in 20/21 -- same thing, should be correct from the
start.

> --- a/src/PVE/HA/Manager.pm
> +++ b/src/PVE/HA/Manager.pm
> @@ -14,6 +14,7 @@ use PVE::HA::Rules::NodeAffinity qw(get_node_affinity);
> [...]
> +use PVE::HA::Usage::Dynamic;

This is unconditional, unlike the Static module which uses the guarded
`eval { require ... }` + `$have_static_scheduling` pattern. If the Rust
library is not installed, the entire HA Manager fails to load.
Should either follow the same guarded approach or we just make the rust
library unconditional, which is probably more than fine to do now.