[PATCH conductor] BZ 800837 - Vanished instances are now included in failed?
Jan Provaznik
jprovazn at redhat.com
Tue Mar 20 13:44:33 UTC 2012
On 03/19/2012 03:59 PM, Matt Wagner wrote:
> The failed? method on an instance was not considering STATE_VANISHED
> to be a failure, causing (among other things) the 'error' icon to not
> display in table view. This fixes this, and has the incidental benefit
> of making the code slightly more compact by using the existing FAILED_STATES
> constant instead of a series of checks.
>
> Resolves https://bugzilla.redhat.com/show_bug.cgi?id=800837
> ---
> src/app/controllers/pools_controller.rb | 2 +-
> src/app/models/instance.rb | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/app/controllers/pools_controller.rb b/src/app/controllers/pools_controller.rb
> index 3529b7d..11b86aa 100644
> --- a/src/app/controllers/pools_controller.rb
> +++ b/src/app/controllers/pools_controller.rb
> @@ -313,7 +313,7 @@ class PoolsController< ApplicationController
>
> def statistics
> instances = current_user.owned_instances
> - failed_instances = instances.select {|instance| instance.state == Instance::STATE_CREATE_FAILED || instance.state == Instance::STATE_ERROR}
> + failed_instances = instances.failed
> @statistics = {
> :pools_in_use => @user_pools.select { |pool| pool.instances.pending.count> 0 || pool.instances.deployed.count> 0 }.count,
> :deployments => current_user.deployments.count,
> diff --git a/src/app/models/instance.rb b/src/app/models/instance.rb
> index 967b26c..f074150 100644
> --- a/src/app/models/instance.rb
> +++ b/src/app/models/instance.rb
> @@ -344,7 +344,7 @@ class Instance< ActiveRecord::Base
> end
>
> def failed?
> - state == Instance::STATE_CREATE_FAILED || state == Instance::STATE_ERROR
> + FAILED_STATES.include?(state)
> end
>
> def requires_config_server?
ACK
More information about the aeolus-devel
mailing list