Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-28 Thread Sergey Lukjanov
Additionally, I think that we should explicitly specify the need to ensure that all outputs doesn't contain any sensitive information like credentials. On Tue, Jan 28, 2014 at 4:06 PM, Alexander Ignatov wrote: > "EDP internal" I meant current EDP specific code. And since job configs > are job-sp

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-28 Thread Alexander Ignatov
“EDP internal” I meant current EDP specific code. And since job configs are job-specific I’d prefer to have urls containing /jobs or at least /edp for that. Regards, Alexander Ignatov On 24 Jan 2014, at 23:20, Matthew Farrellee wrote: > what do you consider "EDP internal", and how does it r

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-27 Thread Trevor McKay
We should consider turning "mains" into a string instead of a list for v2. Hive and Pig Oozie actions use mains, and each may only specify a single

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-24 Thread Matthew Farrellee
what do you consider "EDP internal", and how does it relate to the v1.1 or v2 API? i'm ok with making it plugin independent. i'd just suggest moving it out of /jobs and to something like /extra/config-hints/{type}, maybe along with /extra/validations/config. best, matt On 01/22/2014 06:25

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-22 Thread Alexander Ignatov
Current EDP config-hints are not only plugin specific. Several types of jobs must have certain key/values and without it job will fail. For instance, MapReduce (former Jar) job type requires Mapper/Reducer classes parameters to be set[1]. Moreover, for such kind of jobs we already have separated

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-20 Thread Matthew Farrellee
On 01/20/2014 12:50 PM, Andrey Lazarev wrote: Inlined. On Mon, Jan 20, 2014 at 8:15 AM, Matthew Farrellee mailto:m...@redhat.com>> wrote: (inline, trying to make this readable by a text-only mail client that doesn't use tabs to indicate quoting) On 01/20/2014 02:50 AM, Andrey Laza

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-20 Thread Andrey Lazarev
Inlined. On Mon, Jan 20, 2014 at 8:15 AM, Matthew Farrellee wrote: > (inline, trying to make this readable by a text-only mail client that > doesn't use tabs to indicate quoting) > > On 01/20/2014 02:50 AM, Andrey Lazarev wrote: > > -- >> FIX - @rest.get('/jobs/config-hints

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-20 Thread Matthew Farrellee
(inline-ish) On 01/20/2014 02:36 AM, Andrey Lazarev wrote: On Sun, Jan 19, 2014 at 7:53 AM, Matthew Farrellee mailto:m...@redhat.com>> wrote: On 01/16/2014 09:19 PM, Andrey Lazarev wrote: REMOVE - @rest.get('/job-executions/<__job_execution_id>/refresh-__s

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-20 Thread Matthew Farrellee
(inline, trying to make this readable by a text-only mail client that doesn't use tabs to indicate quoting) On 01/20/2014 02:50 AM, Andrey Lazarev wrote: -- FIX - @rest.get('/jobs/config-hints/__') - should move to GET /plugins//<__plugin_version>, similar to

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-19 Thread Andrey Lazarev
Inlined. Thanks, Andrew. On Sun, Jan 19, 2014 at 7:50 AM, Matthew Farrellee wrote: > On 01/16/2014 08:10 AM, Alexander Ignatov wrote: > >> Matthew, >> >> I'm ok with proposed solution. Some comments/thoughts below: >> >> - >> FIX - @rest.post_file('/plugins/// >> convert-config/') >> -

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-19 Thread Andrey Lazarev
On Sun, Jan 19, 2014 at 7:53 AM, Matthew Farrellee wrote: > On 01/16/2014 09:19 PM, Andrey Lazarev wrote: > >> My 5 cents: >> >> -- >> REMOVE - @rest.put('/node-group-templates/') - >> Not Implemented >> REMOVE - @rest.put('/cluster-templates/') - Not >> Implemented >> -- >> Disagree with

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-19 Thread Matthew Farrellee
On 01/16/2014 09:19 PM, Andrey Lazarev wrote: My 5 cents: -- REMOVE - @rest.put('/node-group-templates/') - Not Implemented REMOVE - @rest.put('/cluster-templates/') - Not Implemented -- Disagree with that. Samsung people did great job in both savanna/savanna-dashboard to make this imple

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-19 Thread Matthew Farrellee
On 01/16/2014 08:10 AM, Alexander Ignatov wrote: Matthew, I'm ok with proposed solution. Some comments/thoughts below: - FIX - @rest.post_file('/plugins///convert-config/') - this is an RPC call, made only by a client to do input validation, move to POST /validations/plugins/:name/:vers

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-17 Thread Alexander Ignatov
++ for generic PUT for both ‘cancel’ and ‘refresh-status’, Andrew. Thanks! Regards, Alexander Ignatov On 17 Jan 2014, at 06:19, Andrey Lazarev wrote: > My 5 cents: > > -- > REMOVE - @rest.put('/node-group-templates/') - Not > Implemented > REMOVE - @rest.put('/cluster-templates/') - No

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-16 Thread Andrey Lazarev
My 5 cents: -- REMOVE - @rest.put('/node-group-templates/') - Not Implemented REMOVE - @rest.put('/cluster-templates/') - Not Implemented -- Disagree with that. Samsung people did great job in both savanna/savanna-dashboard to make this implemented [2], [3]. We should leave and support the

Re: [openstack-dev] [savanna] savannaclient v2 api

2014-01-16 Thread Alexander Ignatov
Matthew, I'm ok with proposed solution. Some comments/thoughts below: - FIX - @rest.post_file('/plugins///convert-config/') - this is an RPC call, made only by a client to do input validation, move to POST /validations/plugins/:name/:version/check-config-import - AFAIR, this rest