Review Request 55817: AMBARI-19618 Make cohosted components configurable in metainfo.xml instead of hardcoding in UI

2017-01-20 Thread yao lei
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55817/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Re: Review Request 55815: Refine Grafana dashboard(s) for AmbariServer to include server specific metric graphs

2017-01-20 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55815/#review162538 --- Ship it! Ship It! - Sid Wagle On Jan. 21, 2017, 3 a.m.,

Review Request 55815: Refine Grafana dashboard(s) for AmbariServer to include server specific metric graphs

2017-01-20 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55815/ --- Review request for Ambari, Aravindan Vijayan, Jaimin Jetly, Sid Wagle, Xi Wang,

Review Request 55814: Add log rotation settings - handle upgrade scenario - Storm, Hive

2017-01-20 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55814/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-19660

Review Request 55812: Error during Alert: Unable to authenticate through LDAP for Hiveserver2 (also floods HS2 log with error messages)

2017-01-20 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55812/ --- Review request for Ambari, Alejandro Fernandez, Swapan Shridhar, and Thejas

Re: Review Request 55789: Deleted configuration groups are displayed along with active group with is_current value set to true using Ambari API's

2017-01-20 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55789/ --- (Updated Jan. 20, 2017, 11:49 p.m.) Review request for Ambari, Di Li and

Review Request 55789: Deleted configuration groups are displayed along with active group with is_current value set to true using Ambari API's

2017-01-20 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55789/ --- Review request for Ambari, Di Li and Jaimin Jetly. Bugs: AMBARI-19641

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55775/#review162515 --- Ship it! Ship It! - Sumit Mohanty On Jan. 20, 2017, 9:16

Re: Review Request 55784: Log Search should handle new services

2017-01-20 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55784/ --- (Updated Jan. 20, 2017, 10:05 p.m.) Review request for Ambari, Oliver Szabo

Re: Review Request 55762: Ambari db-cleanup tool fixed

2017-01-20 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55762/ --- (Updated Jan. 20, 2017, 9:19 p.m.) Review request for Ambari, Attila

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Sumit Mohanty
> On Jan. 20, 2017, 7:16 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/python/ambari_server_main.py, line 106 > > > > > > Why does this need to be so high? > > What's the root cause of failures? >

Re: Review Request 55784: Log Search should handle new services

2017-01-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55784/#review162502 --- Ship it! Ship It! - Oliver Szabo On Jan. 20, 2017, 8:56

Re: Review Request 55784: Log Search should handle new services

2017-01-20 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55784/#review162501 --- Ship it! Ship It! - Robert Nettleton On Jan. 20, 2017, 8:56

Review Request 55784: Log Search should handle new services

2017-01-20 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55784/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs:

Re: Review Request 53213: Exporting blueprint from kerberos enabled cluster, exports hardcoded values cluster name and realm in principal_name property

2017-01-20 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53213/#review162492 --- Pushed to trunk, commit 83cf094b570731920781192db4f4586a3056e382

Re: Review Request 55777: Update quicklink.json files in stack definitions with authenticated/sso features

2017-01-20 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55777/#review162489 --- Ship it! Ship It! - Attila Magyar On Jan. 20, 2017, 6:30

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Alejandro Fernandez
> On Jan. 20, 2017, 7:16 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/python/ambari_server_main.py, line 106 > > > > > > Why does this need to be so high? > > What's the root cause of failures? >

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Alejandro Fernandez
> On Jan. 20, 2017, 7:16 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/python/ambari_server_main.py, line 106 > > > > > > Why does this need to be so high? > > What's the root cause of failures? >

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Madhuvanthi Radhakrishnan
> On Jan. 20, 2017, 7:16 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/python/ambari_server_main.py, line 106 > > > > > > Why does this need to be so high? > > What's the root cause of failures?

Re: Review Request 54760: Add common log rotation settings to HDFS configs

2017-01-20 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54760/#review162472 ---

Re: Review Request 55590: Add common log rotation settings to HDFS, Zookeeper configs - addendum

2017-01-20 Thread Madhuvanthi Radhakrishnan
> On Jan. 20, 2017, 7:15 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/configuration/hdfs-log4j.xml, > > line 56 > > > > > > Please also modify HDFS 3.0.0.3.0

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55775/#review162469 --- ambari-server/src/main/python/ambari_server_main.py (line 106)

Re: Review Request 55590: Add common log rotation settings to HDFS, Zookeeper configs - addendum

2017-01-20 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55590/#review162468 ---

Re: Review Request 55777: Update quicklink.json files in stack definitions with authenticated/sso features

2017-01-20 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55777/#review162467 --- Ship it! Ship It! - Sebastian Toader On Jan. 20, 2017, 7:30

Re: Review Request 55777: Update quicklink.json files in stack definitions with authenticated/sso features

2017-01-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55777/#review162466 --- Ship it! Ship It! - Oliver Szabo On Jan. 20, 2017, 6:30

Re: Review Request 55774: AMBARI-19636: Provide default values for Kafka nofile and nproc limit properties

2017-01-20 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55774/#review162465 --- Ship it! Ship It! - Alejandro Fernandez On Jan. 20, 2017,

Re: Review Request 55777: Update quicklink.json files in stack definitions with authenticated/sso features

2017-01-20 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55777/#review162460 --- Ship it! Ship It! - Laszlo Puskas On Jan. 20, 2017, 6:30

Re: Review Request 55765: llow user to add custom property for log4j

2017-01-20 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55765/#review162457 --- Fix it, then Ship it!

Re: Review Request 55718: AMBARI-19624:Missing fields for workflow SLA

2017-01-20 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55718/#review162456 --- Ship it! Ship It! - belliraj hb On Jan. 20, 2017, 5:55

Re: Review Request 55590: Add common log rotation settings to HDFS, Zookeeper configs - addendum

2017-01-20 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55590/ --- (Updated Jan. 20, 2017, 6:44 p.m.) Review request for Ambari and Sumit

Review Request 55777: Update quicklink.json files in stack definitions with authenticated/sso features

2017-01-20 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55777/ --- Review request for Ambari, Attila Doroszlai, Attila Magyar, Laszlo Puskas,

Re: Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55775/#review162450 --- Ship it! Ship It! - Sumit Mohanty On Jan. 20, 2017, 5:29

Re: Review Request 55762: Ambari db-cleanup tool fixed

2017-01-20 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55762/#review162444 --- Ship it! Ship It! - Attila Doroszlai On Jan. 20, 2017, 2:17

Review Request 55775: Added log messages to ambari_main_server and increased the timeout for server start

2017-01-20 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55775/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-19654

Re: Review Request 55764: Ldap sync fails when there are special characters in distinguished names

2017-01-20 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55764/#review162439 --- Ship it! Ship It! - Laszlo Puskas On Jan. 20, 2017, 4:31

Re: Review Request 55764: Ldap sync fails when there are special characters in distinguished names

2017-01-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55764/#review162438 --- Ship it! Ship It! - Oliver Szabo On Jan. 20, 2017, 4:31

Re: Review Request 55764: Ldap sync fails when there are special characters in distinguished names

2017-01-20 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55764/ --- (Updated Jan. 20, 2017, 11:31 a.m.) Review request for Ambari, Eugene

Review Request 55774: AMBARI-19636: Provide default values for Kafka nofile and nproc limit properties

2017-01-20 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55774/ --- Review request for Ambari and Tim Thorpe. Bugs: AMBARI-19636

Re: Review Request 55758: Intermittent Zeppelin Server start failure during service install and start from Ambari

2017-01-20 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55758/#review162434 --- Ship it! Ship It! - Attila Doroszlai On Jan. 20, 2017,

Re: Review Request 55717: Cannot start Zeppelin notebook if Hive is not installed

2017-01-20 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55717/#review162433 --- Ship it! Ship It! - Attila Doroszlai On Jan. 19, 2017, 4:58

Re: Review Request 55764: Ldap sync fails when there are special characters in distinguished names

2017-01-20 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55764/#review162432 --- Ship it! Ship It! - Sebastian Toader On Jan. 20, 2017, 2:42

Re: Review Request 55763: Remove hard coded smartsense Log Feeder config from Log Search stack

2017-01-20 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55763/#review162426 --- Ship it! Ship It! - Miklos Gergely On Jan. 20, 2017, 1:22

Re: Review Request 55764: Ldap sync fails when there are special characters in distinguished names

2017-01-20 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55764/#review162425 --- Ship it! Ship It! - Jonathan Hurley On Jan. 20, 2017, 8:42

Re: Review Request 55755: Log Search should use Credential Store API to store keystore/truststore passwords

2017-01-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55755/#review162423 ---

Re: Review Request 55755: Log Search should use Credential Store API to store keystore/truststore passwords

2017-01-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55755/#review162422 ---

Re: Review Request 55755: Log Search should use Credential Store API to store keystore/truststore passwords

2017-01-20 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55755/#review162421 --- Ship it! The patch looks fine to me. Can you also please

Re: Review Request 55763: Remove hard coded smartsense Log Feeder config from Log Search stack

2017-01-20 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55763/#review162419 --- Ship it! Ship It! - Robert Nettleton On Jan. 20, 2017, 1:22

Review Request 55765: llow user to add custom property for log4j

2017-01-20 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55765/ --- Review request for Ambari, Madhuvanthi Radhakrishnan and Sumit Mohanty. Bugs:

Re: Review Request 55762: Ambari db-cleanup tool fixed

2017-01-20 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55762/#review162418 --- Ship it! Ship It! - Sebastian Toader On Jan. 20, 2017, 2:17

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

2017-01-20 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/#review162417 --- Ship it! Ship It! - Sebastian Toader On Jan. 20, 2017,

Review Request 55764: Ldap sync fails when there are special characters in distinguished names

2017-01-20 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55764/ --- Review request for Ambari, Eugene Chekanskiy, Jonathan Hurley, Laszlo Puskas,

Re: Review Request 55762: Ambari db-cleanup tool fixed

2017-01-20 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55762/#review162415 --- Ship it! Ship It! - Sandor Magyari On Jan. 20, 2017, 1:17

Review Request 55762: Ambari db-cleanup tool fixed

2017-01-20 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55762/ --- Review request for Ambari, Attila Doroszlai, Sandor Magyari, and Sebastian

Re: Review Request 55757: AMBARI-19647. Issue while submitting workflow as its not able to register as a project

2017-01-20 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55757/#review162413 --- Ship it! Ship It! - belliraj hb On Jan. 20, 2017, 11:57

Re: Review Request 55758: Intermittent Zeppelin Server start failure during service install and start from Ambari

2017-01-20 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55758/#review162411 --- Ship it! Ship It! - Prabhjyot Singh On Jan. 20, 2017, 11:59

Re: Review Request 55758: Intermittent Zeppelin Server start failure during service install and start from Ambari

2017-01-20 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55758/#review162412 --- Ship it! Ship It! - Prabhjyot Singh On Jan. 20, 2017, 11:59

Re: Review Request 55758: Intermittent Zeppelin Server start failure during service install and start from Ambari

2017-01-20 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55758/#review162410 --- Ship it! Ship It! - Rohit Choudhary On Jan. 20, 2017, 11:59

Review Request 55758: Intermittent Zeppelin Server start failure during service install and start from Ambari

2017-01-20 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55758/ --- Review request for Ambari, Attila Doroszlai, Alejandro Fernandez, DIPAYAN

Review Request 55757: AMBARI-19647. Issue while submitting workflow as its not able to register as a project

2017-01-20 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55757/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj

Re: Review Request 55756: Make the config changes for Add / Delete Service flow more prominent

2017-01-20 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55756/#review162409 --- Ship it! Ship It! - Andrii Tkach On Jan. 20, 2017, 11:27

Review Request 55756: Make the config changes for Add / Delete Service flow more prominent

2017-01-20 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55756/ --- Review request for Ambari and Andrii Tkach. Bugs: AMBARI-19644

Re: Review Request 55680: On secure NN HA clusters ZKFC connects to zookeeper securely

2017-01-20 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55680/ --- (Updated Jan. 20, 2017, 11:21 a.m.) Review request for Ambari, Attila Magyar,

Review Request 55755: Log Search should use Credential Store API to store keystore/truststore passwords

2017-01-20 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55755/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Nahappan

Review Request 55752: Not completed Upgrade History items show End Time as "Not started"

2017-01-20 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55752/ --- Review request for Ambari and Denys Buzhor. Bugs: AMBARI-19643

Re: Review Request 55515: Perf: Create Rolling and Express Upgrade Packs

2017-01-20 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55515/#review162405 --- Ship it! Ship It! - Vitalyi Brodetskyi On Січ. 19, 2017,

Re: Review Request 55615: Hive View 2.0: Show query logs

2017-01-20 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55615/#review162400 --- Ship it! Ship It! - Gaurav Nagar On Jan. 20, 2017, 7:44

Re: Review Request 55615: Hive View 2.0: Show query logs

2017-01-20 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55615/#review162399 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Jan. 20, 2017, 7:44

Re: Review Request 55717: Cannot start Zeppelin notebook if Hive is not installed

2017-01-20 Thread Renjith Kamath
> On Jan. 19, 2017, 11:21 p.m., Attila Doroszlai wrote: > > The patches fix the problem with starting Zeppelin without Hive. > > > > However, the [patch against > > trunk](https://issues.apache.org/jira/secure/attachment/12848330/AMBARI-19627-trunk-v2.patch) > > has problems with the unit