Re: Review Request 64769: UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2017-12-21 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64769/ --- (Updated Dec. 21, 2017, 12:31 p.m.) Review request for Ambari and Nate Cole.

Review Request 64769: UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2017-12-21 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64769/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-22406

Re: Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

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

Re: Review Request 64769: AMBARI-22406 : UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2017-12-21 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64769/ --- (Updated Dec. 22, 2017, 4:41 a.m.) Review request for Ambari and Nate Cole.

Re: Review Request 64789: Fix AMS compilation issues and unit test with hbase, hadoop and phoenix upgraded

2017-12-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64789/ --- (Updated Dec. 21, 2017, 11:52 p.m.) Review request for Ambari, Aravindan

Re: Review Request 64693: Add service group dependencies

2017-12-21 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64693/ --- (Updated Гру. 21, 2017, 11:07 після полудня) Review request for Ambari, Jayush

Review Request 64789: Fix AMS compilation issues and unit test with hbase, hadoop and phoenix upgraded

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

Re: Review Request 64789: Fix AMS compilation issues and unit test with hbase, hadoop and phoenix upgraded

2017-12-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64789/#review194373 --- Fix it, then Ship it!

Review Request 64772: ServiceAdvisor for OneFS

2017-12-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64772/ --- Review request for Ambari, Attila Doroszlai, Jonathan Hurley, and Nate Cole.

Re: Review Request 64771: AMBARI-22681 Ambari web UI failed to update stack repo directly from the Versions tab

2017-12-21 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64771/#review194338 --- Ship it! Ship It! - Tim Thorpe On Dec. 21, 2017, 2:05 p.m.,

Re: Review Request 64751: Fix Broken Symlinks on Stack Distribution

2017-12-21 Thread Dmitro Lisnichenko
> On Dec. 20, 2017, 5:03 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > > Lines 199-201 (patched) > > > > > > I'd say just return earlier like this: >

Re: Review Request 64751: Fix Broken Symlinks on Stack Distribution

2017-12-21 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64751/#review194333 --- Ship it! Ship It! - Jonathan Hurley On Dec. 21, 2017, 8:25

Review Request 64771: AMBARI-22681 Ambari web UI failed to update stack repo directly from the Versions tab

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

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-12-21 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64118/#review194329 --- Ship it! Ship It! - Attila Magyar On Dec. 13, 2017, 6:14

Re: Review Request 64751: Fix Broken Symlinks on Stack Distribution

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

Review Request 64785: Disabled stack still appears in the UI if VDF is available

2017-12-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64785/ --- Review request for Ambari, Aravindan Vijayan, Jonathan Hurley, and Nate Cole.

Re: Review Request 64607: Ambari Infra Manager: solr data exporting jobs and integration test environment

2017-12-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64607/#review194364 --- Ship it! Ship It! - Sid Wagle On Dec. 15, 2017, 8:57 a.m.,

Re: Review Request 64785: Disabled stack still appears in the UI if VDF is available

2017-12-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64785/#review194365 --- Ship it! Ship It! - Nate Cole On Dec. 21, 2017, 4 p.m., Sid

Re: Review Request 64772: ServiceAdvisor for OneFS

2017-12-21 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64772/#review194345 --- Fix it, then Ship it! Nice.

Re: Review Request 64775: AMBARI-22394: Store and retrieve the ambari repo urls for os family during bootstrap

2017-12-21 Thread Sneha Kanekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64775/ --- (Updated Dec. 21, 2017, 3:25 p.m.) Review request for Ambari and Nate Cole.

Review Request 64775: Store and retrieve the ambari repo urls for os family during bootstrap

2017-12-21 Thread Sneha Kanekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64775/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-22394

Review Request 64781: AMBARI-22680 Ambari web UI does not fail an invalid repo base URL during cluster installation

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

Re: Review Request 64781: AMBARI-22680 Ambari web UI does not fail an invalid repo base URL during cluster installation

2017-12-21 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64781/#review194347 --- Ship it! Ship It! - Tim Thorpe On Dec. 21, 2017, 4:29 p.m.,

Re: Review Request 64772: ServiceAdvisor for OneFS

2017-12-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64772/#review194357 --- Ship it! Ship It! - Nate Cole On Dec. 21, 2017, 9:46 a.m.,

Re: Review Request 64751: Fix Broken Symlinks on Stack Distribution

2017-12-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64751/#review194353 --- Ship it! Ship It! - Nate Cole On Dec. 21, 2017, 8:25 a.m.,

Re: Review Request 63955: Validate blueprint does not allow lzo enable without setup with license agreement.

2017-12-21 Thread Nate Cole
> On Dec. 13, 2017, 10:37 a.m., Nate Cole wrote: > > Has this been pushed? If so, please close the review. Ping - Nate --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63955/#review193685

Review Request 64782: Add service dependencies

2017-12-21 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64782/ --- Review request for Ambari, Dmytro Sen and Madhuvanthi Radhakrishnan. Bugs:

Re: Review Request 64782: Add service dependencies

2017-12-21 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64782/#review194349 --- Ship it! Ship It! - Dmytro Sen On Dec. 21, 2017, 5:03 p.m.,

Re: Review Request 64772: ServiceAdvisor for OneFS

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