Re: Review Request 62034: Integrate LogSearch new UI with the server and get rid of the old one

2017-09-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62034/ --- (Updated Sept. 1, 2017, 7:17 p.m.) Review request for Ambari, Andrii Babiichuk,

Re: Review Request 62020: Pre-configure services during stack upgrade if Kerberos is enabled to reduce number of core service restarts when services are added

2017-09-01 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62020/#review184368 --- Ship it! Ship It! - Jonathan Hurley On Sept. 1, 2017, 11:29

Re: Review Request 62035: User can't add node via Ambari UI when being part of both "cluster user" and "cluster admin" roles

2017-09-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62035/#review184363 --- Fix it, then Ship it! Ship It! ambari-server/src/test/java/or

Re: Review Request 62020: Pre-configure services during stack upgrade if Kerberos is enabled to reduce number of core service restarts when services are added

2017-09-01 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62020/#review184361 --- Ship it! Ship It! - Attila Magyar On Sept. 1, 2017, 3:29 p.m

Re: Review Request 62020: Pre-configure services during stack upgrade if Kerberos is enabled to reduce number of core service restarts when services are added

2017-09-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62020/ --- (Updated Sept. 1, 2017, 11:29 a.m.) Review request for Ambari, Attila Magyar, B

Re: Review Request 62020: Pre-configure services during stack upgrade if Kerberos is enabled to reduce number of core service restarts when services are added

2017-09-01 Thread Robert Levas
> On Sept. 1, 2017, 3:40 a.m., Attila Magyar wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/PreconfigureKerberosAction.java > > Lines 173 (patched) > > > > > > unnecessary null

Review Request 62035: User can't add node via Ambari UI when being part of both "cluster user" and "cluster admin" roles

2017-09-01 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62035/ --- Review request for Ambari, Dmitro Lisnichenko and Robert Levas. Bugs: AMBARI-21

Re: Review Request 62020: Pre-configure services during stack upgrade if Kerberos is enabled to reduce number of core service restarts when services are added

2017-09-01 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62020/#review184358 --- Ship it! Ship It! - Eugene Chekanskiy On Aug. 31, 2017, 5:55

Re: Review Request 62034: Integrate LogSearch new UI with the server and get rid of the old one

2017-09-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62034/ --- (Updated Sept. 1, 2017, 2:56 p.m.) Review request for Ambari, Andrii Babiichuk,

Review Request 62034: Integrate LogSearch new UI with the server and get rid of the old one

2017-09-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62034/ --- Review request for Ambari, Andrii Babiichuk, Miklos Gergely, and Robert Nettleto

Re: Review Request 61949: Reject PATCH VDFs with Services that are not Included in the Cluster

2017-09-01 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61949/#review184354 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 61949: Reject PATCH VDFs with Services that are not Included in the Cluster

2017-09-01 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61949/#review184353 --- Ship it! Ship It! - Dmytro Grinenko On Sept. 1, 2017, 1:20 p

Re: Review Request 61949: Reject PATCH VDFs with Services that are not Included in the Cluster

2017-09-01 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61949/ --- (Updated Sept. 1, 2017, 4:20 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 61998: Update Ambari Metrics service check to support HTTP SPNEGO authentication

2017-09-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61998/#review184351 --- Ship it! Ship It! ambari-server/src/main/resources/common-ser

Re: Review Request 62026: ambari-server upgrade process still puts the new views jar to default location even if views.dir is customized in ambari.properties

2017-09-01 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62026/#review184348 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 1, 2017, 8:14

Review Request 62026: ambari-server upgrade process still puts the new views jar to default location even if views.dir is customized in ambari.properties

2017-09-01 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62026/ --- Review request for Ambari and Dmytro Sen. Bugs: AMBARI-21684 https://issues

Re: Review Request 62020: Pre-configure services during stack upgrade if Kerberos is enabled to reduce number of core service restarts when services are added

2017-09-01 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62020/#review184341 --- ambari-server/src/main/java/org/apache/ambari/server/serveraction

Re: Review Request 62015: Ranger should handle external Zookeeper properly

2017-09-01 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62015/#review184342 --- Ship it! Ship It! - Mugdha Varadkar On Aug. 31, 2017, 8:41 p