Review Request 64502: Downloaded client configs have invalid values for spark properties in yarn-site.xml

2017-12-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64502/ --- Review request for Ambari, Dmitro Lisnichenko and Nate Cole. Bugs:

Re: Review Request 64498: NFSGateway start failing with error : "ERROR: You must be a privileged user in order to run a secure service."

2017-12-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64498/#review193390 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 11, 2017,

Re: Review Request 64499: Reporting host status was broken by merge

2017-12-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64499/#review193391 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 11, 2017,

Re: Review Request 64501: Fix Stacks UT based on secured.json/default.json

2017-12-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64501/#review193392 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 11, 2017,

Review Request 64501: Fix Stacks UT based on secured.json/default.json

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

Review Request 64498: NFSGateway start failing with error : "ERROR: You must be a privileged user in order to run a secure service."

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

Review Request 64499: Reporting host status was broken by merge

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

Re: Review Request 64242: AMBARI-22476: Modify bootstrap API to pass Ambari repo URL from UI and retrieve Host OS types

2017-12-11 Thread Yussuf Shaikh
> On Dec. 4, 2017, 4:20 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/bootstrap/SshHostInfo.java > > Lines 130-132 (patched) > > > > > > How many URLs are we talking about here. If

Re: Review Request 64502: YARN Shuffle Service Can't Be Found On Client-Only Nodes After New Cluster Install

2017-12-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64502/ --- (Updated Dec. 11, 2017, 12:04 p.m.) Review request for Ambari, Dmitro

Re: Review Request 64397: AMBARI-22602. Add 'clusterSettings' and 'stackSettings' parameters in Execution Command.

2017-12-11 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64397/#review193406 ---

Re: Review Request 64502: YARN Shuffle Service Can't Be Found On Client-Only Nodes After New Cluster Install

2017-12-11 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64502/#review193403 ---

Re: Review Request 64502: YARN Shuffle Service Can't Be Found On Client-Only Nodes After New Cluster Install

2017-12-11 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64502/#review193410 --- Ship it! Ship It! - Dmitro Lisnichenko On Dec. 11, 2017,

Re: Review Request 64502: YARN Shuffle Service Can't Be Found On Client-Only Nodes After New Cluster Install

2017-12-11 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64502/#review193416 --- Ship it! Ship It! - Nate Cole On Dec. 11, 2017, 12:04 p.m.,

Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-11 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/ --- Review request for Ambari, Krisztian Kasa, Miklos Gergely, and Sid Wagle.

Re: Review Request 64478: Blueprint V2 API: object model, resource provider, cluster creation

2017-12-11 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64478/#review193429 ---

Re: Review Request 64242: AMBARI-22476: Modify bootstrap API to pass Ambari repo URL from UI and retrieve Host OS types

2017-12-11 Thread Nate Cole
> On Dec. 4, 2017, 11:20 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/bootstrap/SshHostInfo.java > > Lines 130-132 (patched) > > > > > > How many URLs are we talking about here.

Re: Review Request 64397: AMBARI-22602. Add 'clusterSettings' and 'stackSettings' parameters in Execution Command.

2017-12-11 Thread Swapan Shridhar
> On Dec. 8, 2017, 8:01 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java > > Lines 436 (patched) > > > > > > * no need for

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-11 Thread Jungtaek Lim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/#review193486 --- Ship it! Is it safe to put same configuration value from Storm

Re: Review Request 64397: AMBARI-22602. Add 'clusterSettings' and 'stackSettings' parameters in Execution Command.

2017-12-11 Thread Swapan Shridhar
> On Dec. 8, 2017, 11:41 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-server/src/main/java/org/apache/ambari/server/resources/RootLevelSettingsManager.java > > Line 57 (original) > > > > > > Are we using this

Re: Review Request 64397: AMBARI-22602. Add 'clusterSettings' and 'stackSettings' parameters in Execution Command.

2017-12-11 Thread Swapan Shridhar
> On Dec. 11, 2017, 5:44 p.m., Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java > > Lines 291 (patched) > > > > > > settings or clusterSettings instead of

Re: Review Request 64397: AMBARI-22602. Add 'clusterSettings' and 'stackSettings' parameters in Execution Command.

2017-12-11 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64397/ --- (Updated Dec. 12, 2017, 4:07 a.m.) Review request for Ambari, Attila

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-11 Thread Krisztian Kasa
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/#review193502 --- Ship it! Ship It! - Krisztian Kasa On Dec. 11, 2017, 11:47

Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

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

Re: Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

2017-12-11 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64528/#review193507 --- Ship it! Ship It! - venkat sairam On Dec. 12, 2017, 7:39

Re: Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

2017-12-11 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64528/ --- (Updated Dec. 12, 2017, 1:09 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 64478: Blueprint V2 API: object model, resource provider, cluster creation

2017-12-11 Thread Attila Doroszlai
> On Dec. 11, 2017, 11:23 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > > Line 1029 (original), 1023 (patched) > > > >