[pve-devel] [PATCH ha-manager v2 04/26] rules: introduce location rule plugin

[Daniel Kral]

On 6/20/25 18:17, Jillian Morgan wrote:
> On Fri, Jun 20, 2025 at 10:32 AM Daniel Kral <d.kral at proxmox.com> wrote:
> 
>> Add the location rule plugin to allow users to specify node affinity
>> constraints for independent services.
>>
>> Location rules must specify one or more services, one or more node with
>> optional priorities (the default is 0), and a strictness, which is
>> either
>>
>>    * 0 (loose): services MUST be located on one of the rules' nodes, or
>>    * 1 (strict): services SHOULD be located on one of the rules' nodes.
>>
> 
> Shouldn't these be the other way around (0/"loose" = SHOULD, and 1/"strict"
> = MUST)? The code would seem to bear that out, so it's only your
> description here that was backwards, but still..
> 

Yes, thanks for pointing that out, it should be the other way around, 
will fix that for the follow-up. Otherwise, it is implemented as 
0/"loose" = SHOULD, and 1/"strict" = MUST of course, as documented in 
the module documentation in the patch below.