On 02/21/2012 02:48 PM, jprovazn(a)redhat.com wrote:
From: Jan Provaznik<jprovazn(a)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.