Mohammed Morsi wrote:
> ---
>  src/dbomatic/dbomatic.rb |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/src/dbomatic/dbomatic.rb b/src/dbomatic/dbomatic.rb
> index 1fd8948..00d815c 100644
> --- a/src/dbomatic/dbomatic.rb
> +++ b/src/dbomatic/dbomatic.rb
> @@ -58,6 +58,12 @@ class CondorEventLog < Nokogiri::XML::SAX::Document
>        InstanceEvent.create! :instance => inst,
>                              :event_type => @event_type,
>                              :event_time => @event_time
> +      if @event_type == 'ExecuteEvent'
> +        inst.state = Instance::STATE_RUNNING
> +      elsif @event_type == 'JobTerminated'
> +        inst.state = Instance::STATE_STOPPED
> +      end
> +      inst.save
>        @tag = @event_type = @event_cmd = @event_time = nil
>      end
>    end
>   
Is there any way to log the instance state with the event? Are we sure 
that 'ExecuteEvent' always results in a running instance? What about 
create instance actions for providers that don't auto-start? What about 
actions other than these two event types that affect instance state?

Also -- what's the best way to test this patch? What needs to be 
running/working/etc?

Scott

_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to