On Mon, Feb 11, 2013 at 5:54 PM, Donal Lafferty
<[email protected]> wrote:
> Thanks for the heads up!  A few questions...
>
> 'getCommand()'  or 'getCommands()' ?  I see it written both ways below.  Is 
> one for the pluggable service and the other for the ApiServer?

Where do you see getCommand() it must something else. The method
defined in the PluggableService interface is:
    List<Class<?>> getCommands();


All plugins (management server impl included) which implement a
pluggable service give us a contract that they will expose any api
cmds available to any other class via this interface.
So, in case you're implementing an api within cloud-api, you should
also add the cmd class in management server impl's getCommands().

>
> Also, does this apply to Hypervisor plugins?  Or are Hypervisor plugins 
> merely an implementation of said the 'pluggable service'?

Yes, if they are exposing any API for the api server.

Regards.

>
> DL
>
>> -----Original Message-----
>> From: [email protected] [mailto:[email protected]] On Behalf
>> Of Rohit Yadav
>> Sent: 08 February 2013 09:26
>> To: [email protected]
>> Subject: Note for plugin authors
>>
>> Hi, just wanted to inform you that all plugins which are pluggable services 
>> and
>> expose some API, should implement getCommand() which returns a list of
>> api cmd class. This is used by ApiServer and elsewhere to get information
>> about available apis.
>> While I had tried to fix this for most plugins, I wanted to bring this issue 
>> after
>> recent merges. The issue is filed in detail here, and we need to fix mgmt
>> server's getCommands() as well;
>> https://issues.apache.org/jira/browse/CLOUDSTACK-1210
>>
>> Regards.

Reply via email to