I’d like to add my 2 cents for this:

It’s really hard to read through the changes on pastes. May I ask you to submit 
it on gerrit for review as a WIP patch? This would allow us to give you proper 
feedback.
I +1 Serg’s idea to add a formal spec for this change. May I ask you to submit 
one to murano-specs repo?

If you have any questions on the process of implementing these — feel free to 
come to #murano on IRC or to one of our weekly meetings 
https://wiki.openstack.org/wiki/Meetings/MuranoAgenda (feel free to add your 
ideas to agenda items).

I’m including [murano] in the title to get correct folks’s attention =)

-- 
Kirill Zaitsev
Mirantis, Inc

On 30 March 2016 at 20:33:54, Serg Melikyan (smelik...@mirantis.com) wrote:

Hi wangzhh,

I think this look reasonable, but I would prefer to have a proper spec for this 
feature. I generally like to have extendable API in Murano.

On Thu, Mar 24, 2016 at 8:49 PM, 王正浩 <wang...@awcloud.com> wrote:
I'm sorry that I forgot to tell you murano-paste.ini should be modified to
...
[app:apiv1app]
paste.app_factory = murano.api.v1.router:APIRouterV1.factory
...
And the file [4] is murano/api/v1/extensions_base.py 
 
------------------ Original ------------------
From:  "王正浩"<wang...@awcloud.com>;
Date:  Fri, Mar 25, 2016 11:10 AM
To:  "List"<OpenStack-dev@lists.openstack.org>;
Cc:  "smelikyan"<smelik...@mirantis.com>;
Subject:  Extending API via Plugins
 
Hi Serg Melikyan,
I don't know much about CF Broker API. I'm sorry that I
have no real use-case. But here is a test one which I plan to
complete.

I modified murano/common/wsgi.py[0], murano/api/v1/router.py[1],
added
...
murano.api.v1.extensions =
    test = murano.api.v1.extensions.test:testAPI
...
to the setup.cfg[2]. Imply the class testAPI[3].
The class testAPI  inherit a base class APIExtensionBase[4].
I'll show you my code.
P.S. I copied it from nova. So there are some extra code, hope you don't mind.
[0] http://paste.openstack.org/show/491841/
[1] http://paste.openstack.org/show/491840/
[2] http://paste.openstack.org/show/491842/
[3] http://paste.openstack.org/show/491845/
[4] http://paste.openstack.org/show/491843/



--
Serg Melikyan, Development Manager at Mirantis, Inc.
http://mirantis.com | smelik...@mirantis.com | +1 (650) 440-8979
__________________________________________________________________________  
OpenStack Development Mailing List (not for usage questions)  
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe  
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev  
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to