Re: applied: [PATCH docs 0/2] add requirements for HA-managed resources

["Daniel Kral" <d.kral@proxmox.com>]

On Mon May 4, 2026 at 8:40 PM CEST, Thomas Lamprecht wrote:
> On Mon, 04 May 2026 13:11:53 +0200, Daniel Kral wrote:
>> The first patch cleans up the previous introduction for HA resources a
>> bit and removes some outdated comparison to rgmanager, while the second
>> patch adds a list of requirements with some accompanying text what to do
>> if these requirements are not met.
>> 
>> 
>> Daniel Kral (2):
>>   ha-manager: reword ha resources section
>>   ha-manager: add requirements for ha-managed vms and containers
>> 
>> [...]
>
> Applied, with some wording improvement squashed in, thanks!
>
> Also adapted this slightly to avoid conveying that always all nodes need to
> support all HA resources, as with node affinity that's not a requirement. I
> initially thought about doing that adaption as follow-up, but it was not that
> much change while touching basically quite a few lines due to diff churn, so I
> figured it's better squashed in to keep the commit whole, hope that's OK (I
> did add a notice to the commit message to describe my changes though for
> transparency).

That looks good, the reframing helps in conveying the message that it's
not that these HA resources cannot be added at all, but that admins need
to encode the proper dependencies if they are different from the usual
assumptions. Thanks!

>
> [1/2] ha-manager: reword ha resources section
>       commit: 6e40229424611ea216733259489d605556bb4a32
> [2/2] ha-manager: add requirements for ha-managed vms and containers
>       commit: 8fbf4ab3b38790f6b7219a5e709803ec152055df