Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/ --- (Updated May 27, 2016, 11:40 a.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/#review135246 --- Ship it! Ship It! - Jonathan Hurley On May 27, 2016, 9:52

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-27 Thread Robert Levas
> On May 24, 2016, 4:41 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > Should special permissions

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-27 Thread Robert Levas
> On May 25, 2016, 4:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > This is a little odd -

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-26 Thread Robert Levas
> On May 25, 2016, 4:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > This is a little odd -

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-26 Thread Robert Levas
> On May 25, 2016, 4:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql, line 1296 > > > > > > The description says that both the cluster operator and the admin

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-25 Thread Nate Cole
> On May 24, 2016, 4:41 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > Should special permissions

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-25 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/#review134823 --- Fix it, then Ship it!

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/#review134630 --- Fix it, then Ship it!

Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/ --- Review request for Ambari, Jonathan Hurley, Myroslav Papirkovskyy, and Nate