Haah :) Honestly, I don’t like it. “invoke” doesn’t seem to be carrying any useful information here. And “invoke_mistral” looks completely confusing since it’s not clear it’s related with HTTP.
Renat Akhmerov @ Mirantis Inc. On 27 Feb 2014, at 23:42, Manas Kelshikar <ma...@stackstorm.com> wrote: > How about ... > > invoke_http & invoke_mistral to fit the verb_noun pattern. > > > On Wed, Feb 26, 2014 at 6:04 AM, Renat Akhmerov <rakhme...@mirantis.com> > wrote: > Ooh, I was wrong. Sorry. We use dash naming. We have “on-success”, “on-error” > and so forth. > > Please let us know if you see other inconsistencies. > > Thanks > > Renat Akhmerov > @ Mirantis Inc. > > > > On 26 Feb 2014, at 21:00, Renat Akhmerov <rakhme...@mirantis.com> wrote: > > > Thanks Jay. > > > > Regarding underscore naming. If you meant using underscore naming for > > “createVM” and “novaURL” then yes, “createVM” is just a task name and it’s > > a user preference. The same about “novaURL” which will be defined by users. > > As for keywords, seemingly we follow underscore naming. > > > > Renat Akhmerov > > @ Mirantis Inc. > > > > > > > > On 26 Feb 2014, at 17:58, Jay Pipes <jaypi...@gmail.com> wrote: > > > >> On Wed, 2014-02-26 at 14:38 +0700, Renat Akhmerov wrote: > >>> Folks, > >>> > >>> I’m proposing to rename these two action types REST_API and > >>> MISTRAL_REST_API to HTTP and MISTRAL_HTTP. Words “REST” and “API” > >>> don’t look correct to me, if you look at > >>> > >>> > >>> Services: > >>> Nova: > >>> type: REST_API > >>> parameters: > >>> baseUrl: {$.novaURL} > >>> actions: > >>> createVM: > >>> parameters: > >>> url: /servers/{$.vm_id} > >>> method: POST > >>> > >>> There’s no information about “REST” or “API” here. It’s just a spec > >>> how to form an HTTP request. > >> > >> +1 on HTTP and MISTRAL_HTTP. > >> > >> On an unrelated note, would it be possible to use under_score_naming > >> instead of camelCase naming? > >> > >> Best, > >> -jay > >> > >> > >> > >> _______________________________________________ > >> OpenStack-dev mailing list > >> OpenStack-dev@lists.openstack.org > >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > > > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev@lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev@lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
_______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev