On Mon, 2010-09-20 at 13:36 -0400, Chris Lalancette wrote:
> On 09/10/10 - 04:45:24PM, David Lutterkort wrote:
> > On Thu, 2010-09-09 at 15:30 -0400, Chris Lalancette wrote:
> > > In particular, make sure we always fill in the "actions"
> > > part of the hash so that we properly create the action
> > > links when fetching an individual instance.
> >
> > ACK
>
> One more push for me?
Pushed now; there was actually one tiny error in the patch - squashed
the fix into yours:
> --- a/server/lib/deltacloud/drivers/mock/mock_driver.rb
> +++ b/server/lib/deltacloud/drivers/mock/mock_driver.rb
> @@ -185,34 +185,28 @@ class MockDriver < Deltacloud::BaseDriver
> Instance.new( instance )
> end
>
> - def start_instance(credentials, id)
> + def update_instance_state(credentials, id, state)
> instance_file = "#...@storage_root}/instances/#{id}.yml"
> instance_yml = YAML.load( File.read( instance_file ) )
> + instance_yml[:id] = id
> instance_yml[:state] = 'RUNNING'
That needs to be 'instance_yml[:state] = state'
David
