On Thu, Apr 25, 2013 at 10:59 AM, Prasanna Santhanam <t...@apache.org> wrote:
> On Wed, Apr 24, 2013 at 06:08:49AM -0400, Sebastien Goasguen wrote: > > > > On Apr 24, 2013, at 4:38 AM, Prasanna Santhanam <t...@apache.org> wrote: > > > > > Vijay added the ability to send userdata as POST for the > > > deployVirtualMachine API in review [1]. What I'd like to address here > > > is how to expose this via ApiDiscovery so that clients like marvin, > > > cloudmonkey can autogenerate themselves to support APIs of this > > > kind. This also needs to be clearly specified in our API docs. > > > > > > I'm guessing we'll have to put in additional annotations on our APIs > > > that support POST so that API discovery can print the methods > > > supported (GET/POST). Right now it's only the deployVMCmd (AFAIK). But > > > I expect this will need to be done for others soon. > > > > > > I've included POST support for _every_ command in marvin but that's > > > just brute-force. To make it more intelligent I think we should apply > > > it to only apis that make sense as POST (causing side-effects). But > > > that needs to be exposed by the api endpoint. > > > > > > Thoughts? > > > > Prasanna, this seems to me like a bigger discussion as you say, we > > could see more api start having a POST. > > > Will we later see DELETE and PATCH? > > > > Could be that we are talking about making the API from RESTfull > > which would be a big undertaking. > > I think some work was already underway - Min/Rohit started working on > a complete REST based service. It is a significant change and I'll let > them speak about the scale of that change. In my case, I just want to > auto-generate marvin classes without having to hand edit anything. > Nothing from my side on REST based service. > > > > > I started a toy REST example for a talk: > > https://github.com/runseb/cloudstack-flask/blob/master/flasktest.py > +1 cool Cheers. > This is cool! Will check it out! > > > > > It would be a bit silly to create a REST wrapper on our API but might > give ideas... > > > > -Sebastien > > > > > > > > [1] https://reviews.apache.org/r/10294/ > > > > > > -- > > > Prasanna., > > > > > > ------------------------ > > > Powered by BigRock.com > > > > > -- > Prasanna., > > ------------------------ > Powered by BigRock.com > >
