Mailing-List: contact deltacloud-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: deltacloud-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of clalance@redhat.com
 designates 209.132.183.28 as permitted sender)
Date: Tue, 7 Dec 2010 11:51:32 -0500
From: Chris Lalancette <clalance@redhat.com>
To: deltacloud-dev@incubator.apache.org
Subject: Re: [PATCH] very minor fix to ec2 driver for reporting when instance
 is terminated - removes the ability to 'start' a terminated instance
Message-ID: <20101207165131.GF2875@localhost.localdomain>
Reply-To: Chris Lalancette <clalance@redhat.com>
References: <1291738523-2881-1-git-send-email-marios@redhat.com>
 <1291738523-2881-2-git-send-email-marios@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1291738523-2881-2-git-send-email-marios@redhat.com>
User-Agent: Mutt/1.5.21 (2010-09-15)

On 12/07/10 - 06:15:23PM, marios@redhat.com wrote:
> From: marios <marios@redhat.com>
> 
> ---
>  server/lib/deltacloud/drivers/ec2/ec2_driver.rb |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/server/lib/deltacloud/drivers/ec2/ec2_driver.rb b/server/lib/deltacloud/drivers/ec2/ec2_driver.rb
> index fff3663..fb5a0f4 100644
> --- a/server/lib/deltacloud/drivers/ec2/ec2_driver.rb
> +++ b/server/lib/deltacloud/drivers/ec2/ec2_driver.rb
> @@ -574,7 +574,7 @@ class EC2Driver < Deltacloud::BaseDriver
>    def convert_state(ec2_state)
>      case ec2_state
>      when "terminated"
> -      "STOPPED"
> +      "TERMINATED"
>      when "stopped"
>        "STOPPED"
>      when "running"

As we discussed on IRC, we do want to introduce a TERMINATED state back, but
I think we need to do it in a slightly less ad-hoc fashion.  These states
are visible to the clients, and as such, they need to be consistently defined
across all of the drivers.  If we are going to add this back, what I would
like to see is:

1)  This applied consistently to all of the drivers where it makes sense
2)  The meaning of the various client-visible states documented for future
driver writers
3)  Some sort of ruby construct to try to enforce a consistent set of
externally visible states (this may not be possible).

NACK for now.

-- 
Chris Lalancette