[pve-devel] applied: ha-manager: add ignore state v2

Fabian Grünbichler f.gruenbichler at proxmox.com
Fri Oct 13 10:59:34 CEST 2017


please also send a patch for pve-docs/ha-manager.adoc

On Tue, Jan 24, 2017 at 06:37:21PM +0100, Thomas Lamprecht wrote:
> V2 of this series is a refactored version of v1, the implementation behaves
> the same.
> 
> The ignore state is the state where a service has a HA configuration but
> wants to be unmanaged for a temporary time.
> Deleting the service confiuration deletes also its comments, group
> membership, recovery policy settings. While not much this was reported as
> frustrating by some users.
> 
> The implementation choosen was favored over simply dropping the service
> configuration entry on read fopr the following reasons:
> * the Status API call uses the same methods for processing the resource
>   config as the CRM, so the service would get invisible in the UI and CLI
>   tools. This is missleading and may confuse users. You cannot change the
>   service state over the UI anymore, as its entry is missing.
>   This is naturally fixable but generates more changes than my apporach.
> * adding the check in the Manager class is easier then adding a check to
>   each ha environment (PVE2 and Sim)
> * we can differ between a service removal and requesting that a service gets
>   ignored and change the log output respectively, users can thus easier
>   understand what happened.
> 
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel




More information about the pve-devel mailing list