Re: Review Request 61129: AMBARI-21573. Kafka service failed to start during regenerate keytab after upgrade from Biginsights 4.2.5, 4.2.0 to HDP 2.6.2

2017-07-25 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61129/#review181416 --- Ship it! Ship It! - Jonathan Hurley On July 25, 2017, 8:42 p

Re: Review Request 61129: AMBARI-21573. Kafka service failed to start during regenerate keytab after upgrade from Biginsights 4.2.5, 4.2.0 to HDP 2.6.2

2017-07-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61129/ --- (Updated July 26, 2017, 12:42 a.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 61129: AMBARI-21573. Kafka service failed to start during regenerate keytab after upgrade from Biginsights 4.2.5, 4.2.0 to HDP 2.6.2

2017-07-25 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61129/#review181414 --- Ship it! Ship It! - Sid Wagle On July 26, 2017, 12:17 a.m.,

Review Request 61129: AMBARI-21573. Kafka service failed to start during regenerate keytab after upgrade from Biginsights 4.2.5, 4.2.0 to HDP 2.6.2

2017-07-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61129/ --- Review request for Ambari, Dmytro Grinenko, Di Li, Jonathan Hurley, Sumit Mohant

Re: Review Request 61107: Migrate custom extension support

2017-07-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/#review181409 --- This will also need to be ported over to trunk for Hadoop 3.0. a

Re: Review Request 61107: Migrate custom extension support

2017-07-25 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/#review181405 --- ambari-server/src/main/resources/stacks/HDP/2.6/role_command_orde

Re: Review Request 61107: Migrate custom extension support

2017-07-25 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/ --- (Updated Липень 25, 2017, 11:29 після полудня) Review request for Ambari, Andre

Re: Review Request 61102: AMBARI-21568. Pig service check fails after upgrade

2017-07-25 Thread Alejandro Fernandez
> On July 25, 2017, 8:31 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml > > Lines 75 (patched) > > > > > > HDP 2.6 defaults to exectype=tez,

Re: Review Request 61102: AMBARI-21568. Pig service check fails after upgrade

2017-07-25 Thread Attila Doroszlai
> On July 25, 2017, 10:31 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/params_linux.py > > Lines 43 (patched) > > > > > > We should instead look

Re: Review Request 61102: AMBARI-21568. Pig service check fails after upgrade

2017-07-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61102/#review181372 --- ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/p

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-25 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61083/#review181373 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 61105: Auto populate zookeeper.connect in Ranger Kafka plugin properties

2017-07-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61105/#review181371 --- Ship it! Ship It! - Alejandro Fernandez On July 25, 2017, 10

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-25 Thread Alejandro Fernandez
> On July 24, 2017, 4:22 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > > Lines 210 (patched) > > > > > > Why do we need to delete? > >

Re: Review Request 61107: Migrate custom extension support

2017-07-25 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/ --- (Updated Липень 25, 2017, 7:31 після полудня) Review request for Ambari, Andrew

Re: Review Request 60403: Add Kerberos HTTP SPNEGO authentication support to Ambari Metrics Monitor

2017-07-25 Thread Qin Liu
> On July 5, 2017, 6:38 p.m., Robert Levas wrote: > > ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/spnego_kerberos_auth.py > > Lines 33 (patched) > > > > > > Would it be possible to use > > `res

Re: Review Request 61102: AMBARI-21568. Pig service check fails after upgrade

2017-07-25 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61102/#review181345 --- Ship it! Ship It! - Sebastian Toader On July 25, 2017, 4:11

Re: Review Request 61107: Migrate custom extension support

2017-07-25 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/#review181336 --- ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ST

Re: Review Request 61107: Migrate custom extension support

2017-07-25 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/#review181335 --- ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ST

Review Request 61107: Migrate custom extension support

2017-07-25 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61107/ --- Review request for Ambari, Andrew Onischuk, Sumit Mohanty, and Sid Wagle. Bugs:

Re: Review Request 61079: AMBARI-21559. Alert definition paths should be updated on upgrade

2017-07-25 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61079/#review181334 --- Ship it! Ship It! - Jonathan Hurley On July 25, 2017, 6:06 a

Review Request 61105: Auto populate zookeeper.connect in Ranger Kafka plugin properties

2017-07-25 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61105/ --- Review request for Ambari, Alejandro Fernandez and Gautam Borad. Bugs: AMBARI-2

Re: Review Request 61079: AMBARI-21559. Alert definition paths should be updated on upgrade

2017-07-25 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61079/ --- (Updated July 25, 2017, 12:06 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 61074: Restart due to Stale configurations leads to an internal system exception

2017-07-25 Thread Dmytro Sen
> On Июль 24, 2017, 5:35 п.п., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java > > Lines 1086 (patched) > > > > > > Shouldn't it `r

Re: Review Request 61074: Restart due to Stale configurations leads to an internal system exception

2017-07-25 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61074/ --- (Updated Июль 25, 2017, 8:59 д.п.) Review request for Ambari, Aravindan Vijayan

Review Request 61102: AMBARI-21568. Pig service check fails after upgrade

2017-07-25 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61102/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Sumit Mohanty,

Re: Review Request 61083: AMBARI-21563 : Copy Hadoop core-site.xml to Atlas conf when hadoop is configured for LDAP/AD.

2017-07-25 Thread Vishal Suvagia via Review Board
> On July 24, 2017, 4:22 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > > Lines 210 (patched) > > > > > > Why do we need to delete? Hi J