deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michal Fojtik <mfoj...@redhat.com>
Subject Re: [PATCH] CIMI: system and system_template support for mock driver, with unit tests. Just GET for now, no subcollections
Date Thu, 21 Feb 2013 11:47:41 GMT
On 02/21, Koper, Dies wrote:
> Hi Michal, Marios,
> 
> > > > > +    set :capability, lambda { |m| driver.respond_to? m }
> > > >
> > > > IIRC, this is the capability check that was giving you trouble -
> is
> > this
> > > > working now for you ?
> > >
> > > Yes it was. It is still not working for me. Still hoping Michal will
> > look at it and figure it out.
> 
> The source code you pointed me to yesterday helped me find the culprit.
> Initially I was working on support for systems with system volumes. As I
> was having issues, I took out all the system volume related code to have
> the basic system code reviewed first. However, I hadn't deleted the
> collections/system_volumes.rb file.
> I think there is code to load all files in the collections directory, so
> that must have confused the rabbit/Sinatra rules.

Yes all collections are picked up automatically and registred to Sinatra.

> David said I should ask you what's going on. :)

Can you check if the system_volumes_collection.rb by accident does not
include the same class as system_collection.rb? In that case they will
overide themself which will lead to this bug :-)

For Rabbit, I need to add check where I will refuse to register collection
that is already registred.

  -- Michal

> 
> Once removed, it starting working as expected. Sorry about that.
> I've pushed the patch this morning and will build on that in the next
> days.
> 
> Thanks for all the help so far.
> Cheers,
> Dies
> 

-- 
Michal Fojtik <mfojtik@redhat.com>
Deltacloud API, CloudForms

Mime
View raw message