[pve-devel] [PATCH container/docs/ha-manager/manager/proxmox-widget-toolkit/qemu-server v2 0/8] fix #6613: update HA rules upon resource deletion

[Daniel Kral]

On Wed Sep 24, 2025 at 6:07 PM CEST, Michael Köppl wrote:
> This patch series aims to fix #6613 [0]. Although an implementation was
> proposed in the past, it was not applied since it was unclear how to
> handle the case where the removed resource is the last resource in a
> rule.

Hi!

I left one inline note for the ConfirmRemoveDialog.. I'm not sure if we
should change all other usages for the special case, but as I said, it
might just be nit-picking from my end.

We could also just go for a light-weight yes/no confirm dialog without
replacing the users of Proxmox.window.SafeDestroy but only make the HA
resource delete button the only user of ConfirmRemoveDialog.

Otherwise, it looks good and works as expected. With the inline note /
above addressed, consider this as:

Tested-by: Daniel Kral <d.kral at proxmox.com>
Reviewed-by: Daniel Kral <d.kral at proxmox.com>