Review Request 48894: AMBARI-17306. Filter out Message Of The Day (MOTD) logging from 'llapstatus' command in order to get the output which is JSON parsable.

2016-06-17 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48894/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 48797: BG operations windown appeared after 90 seconds on clicking start all services

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48797/#review138358 --- Ship it! Ship It! - Alejandro Fernandez On June 16, 2016,

Re: Review Request 48805: AMBARI-17280. RU to write out client configs that are dependencies of Hive, ATS, and Oozie during upgrades that change configs

2016-06-17 Thread Alejandro Fernandez
> On June 17, 2016, 12:31 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/mapreduce2_client.py, > > line 55 > > > > > > Is there any way to combine

Re: Review Request 48888: AMBARI-17305 - Local User not displaying in View All option in Ambari

2016-06-17 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4/#review138354 --- Ship it! Ship It! - Zhe (Joe) Wang On June 17, 2016, 11:37

Review Request 48888: AMBARI-17305 - Local User not displaying in View All option in Ambari

2016-06-17 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4/ --- Review request for Ambari and Zhe (Joe) Wang. Bugs: AMBARI-17305

Re: Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48549/#review138343 ---

Re: Review Request 48866: Service check for logsearch server

2016-06-17 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48866/#review138344 --- Ship it! Ship It! - Don Bosco Durai On June 17, 2016, 6:50

Re: Review Request 47497: Atlas Integration : Extend Ambari with settings for Atlas environment configuration

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47497/#review138298 ---

Re: Review Request 48727: Log Search default log levels can not be altered

2016-06-17 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48727/ --- (Updated June 17, 2016, 10:35 p.m.) Review request for Ambari, Don Bosco

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Alejandro Fernandez
> On June 17, 2016, 6:44 p.m., Di Li wrote: > > Hello Victor, > > > > so I ran some tests and observed the following. I have a 3 node cluster, > > c1.apache.org, c2.apache.org, and c3.apache.org > > > > 1. Right after finishing the manual steps listed on step "Initialize > > Metadata". I

Re: Review Request 48866: Service check for logsearch server

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48866/#review138331 ---

Re: Review Request 48863: Ambari-server upgrade results in "DB configs consistency check failed. "

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48863/#review138330 ---

Re: Review Request 48819: As part of atlas startup, "/usr/lib/ambari-logsearch-solr-client/solrCloudCli.sh" script is executed by ambari, which fails with "java.io.FileNotFoundException"

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48819/#review138329 --- Pushed to trunk, commit 92237462de2e5f4177113a07129b10834a9c5ca0

Re: Review Request 48819: As part of atlas startup, "/usr/lib/ambari-logsearch-solr-client/solrCloudCli.sh" script is executed by ambari, which fails with "java.io.FileNotFoundException"

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48819/#review138328 --- Ship it! - Alejandro Fernandez On June 16, 2016, 9:31 p.m.,

Re: Review Request 48827: AMBARI-17286 Service config links should use public host name

2016-06-17 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48827/#review138319 --- Ship it! Ship It! - Srimanth Gunturi On June 17, 2016, 9:08

Re: Review Request 48827: AMBARI-17286 Service config links should use public host name

2016-06-17 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48827/#review138316 ---

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Victor Galgo
> On June 17, 2016, 6:44 p.m., Di Li wrote: > > Hello Victor, > > > > so I ran some tests and observed the following. I have a 3 node cluster, > > c1.apache.org, c2.apache.org, and c3.apache.org > > > > 1. Right after finishing the manual steps listed on step "Initialize > > Metadata". I

Re: Review Request 48819: As part of atlas startup, "/usr/lib/ambari-logsearch-solr-client/solrCloudCli.sh" script is executed by ambari, which fails with "java.io.FileNotFoundException"

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48819/#review138299 ---

Re: Review Request 48863: Ambari-server upgrade results in "DB configs consistency check failed. "

2016-06-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48863/#review138285 --- Ship it! Ship It! - Andrew Onischuk On June 17, 2016, 6:30

Re: Review Request 48863: Ambari-server upgrade results in "DB configs consistency check failed. "

2016-06-17 Thread Victor Galgo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48863/#review138284 --- Ship it! Ship It! - Victor Galgo On June 17, 2016, 6:30

Review Request 48866: Service check for logsearch server

2016-06-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48866/ --- Review request for Ambari, Don Bosco Durai, Robert Nettleton, Sumit Mohanty,

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48734/#review138281 --- Hello Jayush and Alejandro, Could you please take a look at my

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48734/#review138280 --- Hello Victor, so I ran some tests and observed the following. I

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48854/ --- (Updated June 17, 2016, 6:38 p.m.) Review request for Ambari, Miklos Gergely,

Review Request 48863: Ambari-server upgrade results in "DB configs consistency check failed. "

2016-06-17 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48863/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, and

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48854/ --- (Updated June 17, 2016, 6:26 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Victor Galgo
> On June 17, 2016, 3:15 p.m., Jayush Luniya wrote: > > ambari-web/app/messages.js, line 1325 > > > > > > Not sure if stopping namenodes is the right way to go about with this. Jayush, it looks right to me because

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Victor Galgo
> On June 17, 2016, 5:44 p.m., Alejandro Fernandez wrote: > > ambari-web/app/controllers/main/admin/highAvailability/nameNode/step9_controller.js, > > line 25 > > > > > > How does this fix the issue? If NN just

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48734/#review138263 ---

Re: Review Request 47514: Kerberos wizard gets reset and does not remember selections/data entered in previous step when you click on Back

2016-06-17 Thread Anita Jebaraj
> On June 17, 2016, 2:33 p.m., Di Li wrote: > > Hello Anita, > > > > could you please close this one if it has been submitted? Sure Di - Anita --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 48766: MySQL service status needs to be more robust

2016-06-17 Thread Juanjo Marron
> On June 16, 2016, 10:52 p.m., Andrew Onischuk wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/mysql_service.py, > > line 63 > > > > > > Else should go into for, not if

Re: Review Request 48858: Ambari Metrics service check failed exceed timeout

2016-06-17 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48858/#review138241 --- Ship it! Ship It! - Dmitro Lisnichenko On June 17, 2016,

Re: Review Request 48859: RU: YARN service check failed during RU

2016-06-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48859/#review138240 --- Ship it! Ship It! - Andrew Onischuk On June 17, 2016, 3:40

Review Request 48859: RU: YARN service check failed during RU

2016-06-17 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48859/ --- Review request for Ambari and Andrew Onischuk. Bugs: AMBARI-17300

Re: Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48549/#review138238 --- Ship it! Ship It! - Jonathan Hurley On June 16, 2016, 7:57

Review Request 48858: Ambari Metrics service check failed exceed timeout

2016-06-17 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48858/ --- Review request for Ambari, Aravindan Vijayan, Sumit Mohanty, and Vitalyi

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

2016-06-17 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48734/#review138234 --- ambari-web/app/messages.js (line 1325)

Re: Review Request 48855: AMBARI-12697 Rolling upgrade: Ambari UI should be able to display rolling upgrade history

2016-06-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48855/ --- (Updated June 17, 2016, 3:08 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-17 Thread Jayush Luniya
> On June 17, 2016, 2:57 p.m., Jayush Luniya wrote: > > Ship It! > > Jayush Luniya wrote: > Update patch in CR with > https://issues.apache.org/jira/secure/attachment/12811383/AMBARI-16920-7.patch Also close CR once done - Jayush

Re: Review Request 48308: [AMBARI-17078] Make Spark2-ThriftServer and Livy Server as optional by default

2016-06-17 Thread Jayush Luniya
> On June 15, 2016, 4:57 p.m., Jayush Luniya wrote: > > Ship It! @Jeff can you close this CR as the patch is committed? - Jayush --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 48651: Add unit tests for Spark2 service definition

2016-06-17 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48651/#review138229 --- @Saisai Looks like you need to rebase again $ git apply

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-17 Thread Jayush Luniya
> On June 17, 2016, 2:57 p.m., Jayush Luniya wrote: > > Ship It! Update patch in CR with https://issues.apache.org/jira/secure/attachment/12811383/AMBARI-16920-7.patch - Jayush --- This is an automatically generated e-mail. To reply,

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-17 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47941/#review138226 --- Ship it! Ship It! - Jayush Luniya On June 16, 2016, 11:06

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-17 Thread Jayush Luniya
> On June 16, 2016, 6:15 p.m., Jayush Luniya wrote: > > This is causing unit test failures. Please take a look. > > > > --- > > Failed tests: > > FAIL: test_hook_default_conf_select > >

Re: Review Request 47514: Kerberos wizard gets reset and does not remember selections/data entered in previous step when you click on Back

2016-06-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47514/#review138222 --- Hello Anita, could you please close this one if it has been

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-17 Thread Jayush Luniya
> On June 16, 2016, 6:15 p.m., Jayush Luniya wrote: > > This is causing unit test failures. Please take a look. > > > > --- > > Failed tests: > > FAIL: test_hook_default_conf_select > >

Review Request 48852: Ambari displays a warning about config values not being at optimal values right after a clean install with no customization

2016-06-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48852/ --- Review request for Ambari and Dmytro Sen. Bugs: AMBARI-17296

Re: Review Request 48850: Views in Ambari UI don't render when proxied by Knox

2016-06-17 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48850/#review138216 --- Ship it! Ship It! - Oleg Nechiporenko On June 17, 2016,

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-06-17 Thread Jayush Luniya
> On June 2, 2016, 5:01 p.m., Jayush Luniya wrote: > > Ship It! @Jeff can you close this CR? - Jayush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review135966

Review Request 48850: Views in Ambari UI don't render when proxied by Knox

2016-06-17 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48850/ --- Review request for Ambari, Andrii Tkach, Oleg Nechiporenko, and Sumit Gupta.

Re: Review Request 48772: [AMBARI-17243] Change livy principal to service type

2016-06-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48772/#review138213 --- Ship it! Ship It! - Robert Levas On June 17, 2016, 8:58

Re: Review Request 48844: Operations during upgrade are permitted by all roles

2016-06-17 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48844/#review138212 --- Ship it! Ship It! - Robert Levas On June 17, 2016, 6:16

Re: Review Request 48772: [AMBARI-17243] Change livy principal to service type

2016-06-17 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48772/ --- (Updated June 17, 2016, 12:58 p.m.) Review request for Ambari, Jayush Luniya

Review Request 48848: [AMBARI-17243] Change livy principal to service type

2016-06-17 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48848/ --- Review request for Ambari, Jayush Luniya and Sumit Mohanty. Bugs: AMBARI-17243

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

2016-06-17 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated June 17, 2016, 12:26 p.m.) Review request for Ambari, Alexander

Re: Review Request 48797: BG operations windown appeared after 90 seconds on clicking start all services

2016-06-17 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48797/#review138199 --- Ship it! Ship It! - Dmitro Lisnichenko On June 16, 2016,

Re: Review Request 48791: All created views disappear after upgrade to 2.4.0.0 [views page]

2016-06-17 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48791/#review138194 --- Ship it! Ship It! - Rohit Choudhary On June 16, 2016, 1:07

Re: Review Request 48766: MySQL service status needs to be more robust

2016-06-17 Thread Andrew Onischuk
> On June 16, 2016, 10:52 p.m., Andrew Onischuk wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/mysql_service.py, > > line 63 > > > > > > Else should go into for, not if

Re: Review Request 48829: AMBARI-17245 Failed to start Hive metastore due to UnicodeDecodeError

2016-06-17 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48829/#review138187 --- Ship it! Ship It! - Andrew Onischuk On June 17, 2016, 12:18

Review Request 48840: AMBARI-17249 Storm metrics sink should include worker host and port to metric name when metrics are coming from SystemBolt

2016-06-17 Thread Jungtaek Lim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48840/ --- Review request for Ambari, Aravindan Vijayan, Sriharsha Chintalapani, and Sid