[pve-devel] [PATCH ha-manager 3/4] config, env: allow bulk updates with update_resources_config

[Daniel Kral]

On Mon Jan 19, 2026 at 12:48 PM CET, Fiona Ebner wrote:
> Nit: I feel like the $delete belongs inside the $changes for a clean
> interface. So something like
> $changes = {
>   $sid => {
>     param => $param,
>     delete => $delete,
>   }
> }
>
> I know the single caller using bulk won't need it, but the current
> interface does feel slightly off to me. What do you think?

Sounds good, will do! Just for clarification: Is `param => $param` a
single changed property or is $param the hash of changes?

On Mon Jan 19, 2026 at 12:48 PM CET, Fiona Ebner wrote:
>
> Nit: I'd prefer to iterate after 'sort'

Good idea, will do for both!