Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/#review185920 --- Ship it! Ship It! - Aravindan Vijayan On Sept. 21, 2017,

Re: Review Request 62460: Enable logsearch-web unit tests and make them skippable

2017-09-21 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62460/#review185874 --- Ship it! Ship It! - Miklos Gergely On Sept. 21, 2017, 10:47

Review Request 62461: Livy not working with the default configuration

2017-09-21 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62461/ --- Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit

Re: Review Request 62460: Enable logsearch-web unit tests and make them skippable

2017-09-21 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62460/#review185872 --- Ship it! Ship It! - Andrii Babiichuk On Вер. 21, 2017,

Re: Review Request 62461: Livy not working with the default configuration

2017-09-21 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62461/ --- (Updated Sept. 21, 2017, 6:17 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62461: Livy not working with the default configuration

2017-09-21 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62461/ --- (Updated Sept. 21, 2017, 6:24 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62461: Livy not working with the default configuration

2017-09-21 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62461/#review185885 --- Ship it! Ship It! - Gaurav Nagar On Sept. 21, 2017, 12:47

Re: Review Request 62461: Livy not working with the default configuration

2017-09-21 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62461/ --- (Updated Sept. 21, 2017, 6:30 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 62460: Enable logsearch-web unit tests and make them skippable

2017-09-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62460/#review185873 --- Ship it! Ship It! - Attila Magyar On Sept. 21, 2017, 10:47

Re: Review Request 62460: Enable logsearch-web unit tests and make them skippable

2017-09-21 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62460/#review185889 --- Ship it! Ship It! - Robert Nettleton On Sept. 21, 2017,

Review Request 62460: Enable logsearch-web unit tests and make them skippable

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

Re: Review Request 62461: Livy not working with the default configuration

2017-09-21 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62461/#review185886 ---

Review Request 62467: Add UID/GID related issue with external users not listed in /etc/passwd

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

Review Request 62468: OOZIE/OOZIE_SERVER restart failed saying AttributeError: 'NoneType' object has no attribute 'upper' during EU

2017-09-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62468/ --- Review request for Ambari, Dmitro Lisnichenko and Sid Wagle. Bugs:

Re: Review Request 62465: AMBARI-22025. Service auto start broken due to incomplete execution command

2017-09-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62465/#review185898 --- Ship it! Any new tests required? - Nate Cole On Sept. 21,

Re: Review Request 62449: Kerberos identity references should use the "reference" attribute

2017-09-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62449/#review185899 --- Ship it! Ship It! - Nate Cole On Sept. 21, 2017, 7:56 a.m.,

Review Request 62465: AMBARI-22025. Service auto start broken due to incomplete execution command

2017-09-21 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62465/ --- Review request for Ambari, Balázs Bence Sári, Jonathan Hurley, and Nate Cole.

Re: Review Request 62351: AMBARI-21910 Ranger Usersync config to support nested group evaluation for LDAP Sync source property

2017-09-21 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62351/#review185894 --- Fix it, then Ship it! Ship It!

Re: Review Request 62467: Add UID/GID related issue with external users not listed in /etc/passwd

2017-09-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62467/#review185896 --- Ship it! Ship It! - Robert Levas On Sept. 21, 2017, 11:32

Re: Review Request 62449: Kerberos identity references should use the "reference" attribute

2017-09-21 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62449/#review185891 --- Ship it! Ship It! - Jonathan Hurley On Sept. 21, 2017, 7:56

Re: Review Request 62449: Kerberos identity references should use the "reference" attribute

2017-09-21 Thread Robert Levas
> On Sept. 21, 2017, 3:02 a.m., Attila Magyar wrote: > > ambari-server/src/main/resources/stacks/PERF/1.0/services/GRUMPY/kerberos.json > > Lines 69 (patched) > > > > > > is it ok to not have backslash here? It is

Re: Review Request 62449: Kerberos identity references should use the "reference" attribute

2017-09-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62449/#review185879 --- Ship it! Ship It! - Attila Magyar On Sept. 21, 2017, 11:56

Re: Review Request 62449: Kerberos identity references should use the "reference" attribute

2017-09-21 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62449/ --- (Updated Sept. 21, 2017, 7:56 a.m.) Review request for Ambari, Attila Magyar,

Re: Review Request 62449: Kerberos identity references should use the "reference" attribute

2017-09-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62449/#review185846 ---

Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/ --- Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Jaimin Jetly.

Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/#review185913 ---

Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/ --- (Updated Sept. 21, 2017, 7:57 p.m.) Review request for Ambari, Aravindan

Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Sid Wagle
> On Sept. 21, 2017, 7:37 p.m., Aravindan Vijayan wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-env.xml > > Lines 81 (patched) > > > > > > Should we allow minimum

Re: Review Request 62467: Add UID/GID related issue with external users not listed in /etc/passwd

2017-09-21 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62467/ --- (Updated Sept. 21, 2017, 6 p.m.) Review request for Ambari, Dmitro Lisnichenko

Re: Review Request 62465: AMBARI-22025. Service auto start broken due to incomplete execution command

2017-09-21 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62465/ --- (Updated Sept. 21, 2017, 8:29 p.m.) Review request for Ambari, Balázs Bence

Re: Review Request 62468: OOZIE/OOZIE_SERVER restart failed saying AttributeError: 'NoneType' object has no attribute 'upper' during EU

2017-09-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62468/#review185901 --- Ship it! Ship It! - Dmitro Lisnichenko On Sept. 21, 2017,

Re: Review Request 62467: Add UID/GID related issue with external users not listed in /etc/passwd

2017-09-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62467/#review185902 --- Ship it!

Re: Review Request 62465: AMBARI-22025. Service auto start broken due to incomplete execution command

2017-09-21 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62465/#review185907 --- Ship it! Ship It! - Jonathan Hurley On Sept. 21, 2017,

Review Request 62456: Add JMX enable/disable option to Infra Solr

2017-09-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62456/ --- Review request for Ambari, Attila Magyar, Miklos Gergely, Robert Nettleton, and

Re: Review Request 62456: Add JMX enable/disable option to Infra Solr

2017-09-21 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62456/#review185858 --- Ship it! Ship It! - Miklos Gergely On Sept. 21, 2017, 9:29

Re: Review Request 62448: Addition of service component after patching a service still keeps the component at base version

2017-09-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62448/#review185865 --- Ship it! Ship It! - Dmitro Lisnichenko On Sept. 20, 2017,

Re: Review Request 62454: Could not copy tez tarball to HDFS during prepare upgrade (EU)

2017-09-21 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62454/#review185866 --- Ship it! Ship It! - Dmitro Lisnichenko On Sept. 20, 2017,

Re: Review Request 62456: Add JMX enable/disable option to Infra Solr

2017-09-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62456/#review185868 --- Ship it! Ship It! - Attila Magyar On Sept. 21, 2017, 9:29