came across a very minor (but deadly) typo which needs fixing before push:

* in ec2 endpoint_for_service - Deltacloud::Drivers::driver_config[:ec2][:entry_points] should be [:entrypoints] or you can't get at any of the collections

otherwise, ACK

general thoughts:

*should we list '/drivers/' in '/api' - i know its not a collection but seems wrong to 'have to know its there' to request it, without it being advertised anywhere

*related to above, do we want to add routes to individual drivers, like 'get /api/drivers/ec2' for example, or even 'get /api/drivers/ec2/credentials' or 'get /api/drivers/ec2/entrypoints'

*i know we don't take the html interface too seriously but seems difference in xml vs html for /drivers is very big (id/name, vs credentials, entrypoints)


On 18/02/11 15:32, [email protected] wrote:
Hi,

I'm really sorry for this spam ;-) but I forget to include
config/drivers.yaml file, which is the most important part of
this patchset :)

   -- Michal


Reply via email to