[PATCH conductor] BZ 788558 - allow terminate inaccessible instances on 'stop' action

Jan Provaznik jprovazn at redhat.com
Tue Feb 21 14:48:10 UTC 2012


On 02/21/2012 02:48 PM, jprovazn at redhat.com wrote:
> From: Jan Provaznik<jprovazn at redhat.com>
>
> Theis patch allows locally stopping of instances or deployments if
> the instance's/deployment's provider is not accessible.
>
> It fixes a bug in passing user param to queue_action method, also simplifies
> used error messages - only (stop|reboot)_invalid_action can be raised.
>
> - this patch doesn't cover deletion of deployments which are not accessible - this is difficult
> because there is a before_destroy hook on deployment which tries to destroy instances
> on provider side.
> - instances in folowong states are stopped locally:
> STATE_NEW, STATE_PENDING, STATE_RUNNING, STATE_SHUTTING_DOWN
> I think it makes sense to include STATE_SHUTTING_DOWN too. But this is not
> consistent w/ termination when disabling provider when only
> STATE_NEW, STATE_PENDING, STATE_RUNNING are stopped.
>
> Not sure what is better, any suggestions are welcomed.

Ignore this patch - going to update provider disabling action to stop 
also instances in SHUTTING_DOWN state too.



More information about the aeolus-devel mailing list