[pve-devel] [PATCH ha-manager 2/3] Status: improve feedback for 'ignored' state transitions

Dietmar Maurer dietmar at proxmox.com
Wed Nov 30 06:52:51 CET 2016


Why do you call that state "setting ignored"? I would prefer a simple "ignored".
Else we have to document two different states with same semantics.

> diff --git a/src/PVE/API2/HA/Status.pm b/src/PVE/API2/HA/Status.pm
> index dbf23d5..0e081bc 100644
> --- a/src/PVE/API2/HA/Status.pm
> +++ b/src/PVE/API2/HA/Status.pm
> @@ -162,10 +162,14 @@ __PACKAGE__->register_method ({
>  			$state = 'starting';
>  		    } elsif ($req eq 'disabled') {
>  			$state = 'disabled';
> +		    } elsif ($req eq 'ignored') {
> +			$state = 'setting ignored';




More information about the pve-devel mailing list