Re: Review Request 64707: AMBARI-22669 Ranger stack script changes to fix missing directory failure for blueprint installation

2017-12-20 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64707/#review194229 --- Ship it! - Vishal Suvagia On Dec. 19, 2017, 10:19 a.m., Mugdh

Re: Review Request 64537: AMBARI-22635 : Ambari should create a dummy core-site.xml for Ranger plugins when namenode is not installed.

2017-12-13 Thread Vishal Suvagia via Review Board
> On Dec. 12, 2017, 4:36 p.m., Sid Wagle wrote: > > Instead of putting empty files on the classpath isn't it easier to check > > for file existence on plugin side and defaul to "simple" auth? > > Sid Wagle wrote: > Correction not empty I meant dummy file. Thank-you Siddharth, that is the c

Re: Review Request 64537: AMBARI-22635 : Ambari should create a dummy core-site.xml for Ranger plugins when namenode is not installed.

2017-12-13 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64537/ --- (Updated Dec. 13, 2017, 3:42 p.m.) Review request for Ambari, Gautam Borad, Jon

Review Request 64537: AMBARI-22635 : Ambari should create a dummy core-site.xml for Ranger plugins when namenode is not installed.

2017-12-12 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64537/ --- Review request for Ambari, Gautam Borad, Jonathan Hurley, Jayush Luniya, Madhuva

Re: Review Request 63907: AMBARI-22463 Removing secure reference configs entries from stack for Ranger KMS service

2017-11-19 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63907/#review191479 --- Ship it! Ship It! - Vishal Suvagia On Nov. 17, 2017, 12:48 p

Re: Review Request 62985: AMBARI-22232 : Need to add a new property to support proxy users property for Atlas service

2017-10-14 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62985/ --- (Updated Oct. 14, 2017, 11:17 a.m.) Review request for Ambari, Gautam Borad, Jo

Re: Review Request 62985: AMBARI-22232 : Need to add a new property to support proxy users property for Atlas service

2017-10-14 Thread Vishal Suvagia via Review Board
> On Oct. 14, 2017, 12:18 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/AtlasProxyUserConfigCalculation.java > > Lines 38 (patched) > > > > > > Hard coded?

Review Request 62985: AMBARI-22232 : Need to add a new property to support proxy users property for Atlas service

2017-10-13 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62985/ --- Review request for Ambari, Gautam Borad, Jonathan Hurley, Madhan Neethiraj, Mugd

Review Request 62239: AMBARI-21925 : Facilitate a parameter to provide HBase conf directory for atlas-env configurations.

2017-09-12 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62239/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mad

Re: Review Request 61527: AMBARI-21649 : Update Atlas log4j to reflect latest configurations

2017-08-09 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61527/ --- (Updated Aug. 10, 2017, 5:42 a.m.) Review request for Ambari, Alejandro Fernand

Review Request 61527: AMBARI-21649 : Update Atlas log4j to reflect latest configurations

2017-08-09 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61527/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mad

Review Request 61217: AMBARI-21588 : MetadataServer start fails if Atlas service is started with custom service user.

2017-07-28 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61217/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mad

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

2017-07-27 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61083/ --- (Updated July 27, 2017, 6:49 p.m.) Review request for Ambari, Alejandro Fernand

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

2017-07-27 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? > >

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

2017-07-27 Thread Vishal Suvagia via Review Board
> On July 25, 2017, 8:29 p.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > > Lines 210 (patched) > > > > > > Multiple copies of core-site is

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

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

2017-07-24 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61083/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mad

Re: Review Request 60842: AMBARI-21460 : Add new kafka client properties to the ambari managed atlas config

2017-07-13 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60842/ --- (Updated July 14, 2017, 6:34 a.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 60842: AMBARI-21460 : Add new kafka client properties to the ambari managed atlas config

2017-07-13 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60842/ --- (Updated July 13, 2017, 3:24 p.m.) Review request for Ambari, Alejandro Fernand

Review Request 60842: AMBARI-21460 : Add new kafka client properties to the ambari managed atlas config

2017-07-13 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60842/ --- Review request for Ambari, Alejandro Fernandez, Ayub Pathan, Gautam Borad, Jayus

Re: Review Request 60560: AMBARI-21374 : Replication factor should be configurable in Atlas to create solr collections

2017-07-02 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60560/ --- (Updated July 3, 2017, 5:35 a.m.) Review request for Ambari, Alejandro Fernande

Review Request 60560: AMBARI-21374 : Replication factor should be configurable in Atlas to create solr collections

2017-06-30 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60560/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mad

Re: Review Request 60388: AMBARI-21256 : As part of START_ALL of services Ranger kms starts after hbase and hive causing operation failure

2017-06-26 Thread Vishal Suvagia via Review Board
> On June 26, 2017, 8:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/role_command_order.json > > Lines 2 (patched) > > > > > > This file is also needed for HBa

Review Request 60388: AMBARI-21256 : As part of START_ALL of services Ranger kms starts after hbase and hive causing operation failure

2017-06-23 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60388/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mug

Re: Review Request 59701: AMBARI-21154 : Add JAAS config properties for Atlas Hive hook in HiveCli to use kerberos ticket-cache

2017-06-23 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59701/ --- (Updated June 23, 2017, 1:04 p.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 59701: AMBARI-21154 : Add JAAS config properties for Atlas Hive hook in HiveCli to use kerberos ticket-cache

2017-06-23 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59701/ --- (Updated June 23, 2017, 12:44 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 59701: AMBARI-21154 : Add JAAS config properties for Atlas Hive hook in HiveCli to use kerberos ticket-cache

2017-06-23 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59701/ --- (Updated June 23, 2017, 12:38 p.m.) Review request for Ambari, Alejandro Fernan

Review Request 59701: AMBARI-21154 : Add JAAS config properties for Atlas Hive hook in HiveCli to use kerberos ticket-cache

2017-05-31 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59701/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Re: Review Request 58995: AMBARI-20932 : Ranger plugin setup should handle Nifi service

2017-05-06 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58995/ --- (Updated May 6, 2017, 6:45 p.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 58995: AMBARI-20932 : Ranger plugin setup should handle Nifi service

2017-05-05 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58995/ --- (Updated May 5, 2017, 1:44 p.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 58993: AMBARI-20930 : stackadvisor recommendations for atlas hooks should be disabled for external Atlas

2017-05-05 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58993/ --- (Updated May 5, 2017, 12:17 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 58995: AMBARI-20932 : Ranger plugin setup should handle Nifi service

2017-05-04 Thread Vishal Suvagia via Review Board
> On May 4, 2017, 7:48 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/setup_ranger_plugin_xml.py > > Lines 221 (patched) > > > > > > We should use stack_roo

Review Request 58995: AMBARI-20932 : Ranger plugin setup should handle Nifi service

2017-05-04 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58995/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya, Mug

Review Request 58993: AMBARI-20930 : stackadvisor recommendations for atlas hooks should be disabled for external Atlas

2017-05-04 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58993/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Re: Review Request 58797: AMBARI-20868 : Ranger admin start fails when Ambari Infra is turned off.

2017-05-04 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58797/ --- (Updated May 4, 2017, 1:27 p.m.) Review request for Ambari, Alejandro Fernandez

Re: Review Request 58965: AMBARI-20920 : Atlas restart fails due to missing hbase dependency in role-command

2017-05-03 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58965/ --- (Updated May 3, 2017, 3:12 p.m.) Review request for Ambari, Alejandro Fernandez

Review Request 58965: AMBARI-20920 : Atlas restart fails due to missing hbase dependency in role-command

2017-05-03 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58965/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Re: Review Request 58797: AMBARI-20868 : Ranger admin start fails when Ambari Infra is turned off.

2017-05-02 Thread Vishal Suvagia via Review Board
> On May 2, 2017, 10:10 a.m., Oliver Szabo wrote: > > Im feeling that is not really the right solution to put the whole code into > > a big try catch block. it wont fail with even syntactic errors. Hi Oliver, This is basically done so as to start Ranger irrespective of zookeeper error

Re: Review Request 58797: AMBARI-20868 : Ranger admin start fails when Ambari Infra is turned off.

2017-04-27 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58797/ --- (Updated April 27, 2017, 3:51 p.m.) Review request for Ambari, Alejandro Fernan

Review Request 58798: AMBARI-20873 : Atlas gc-worker logs are not generated due to incorrect configuration

2017-04-27 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58798/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Review Request 58797: AMBARI-20868 : Ranger admin start fails when Ambari Infra is turned off.

2017-04-27 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58797/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Mugdha Varadkar, O

Review Request 58576: AMBARI-20792 : Atlas admin username and password should be configurable through ambari

2017-04-20 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58576/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Re: Review Request 57809: AMBARI-20505 : Changes needed to handle AMBARI-19623 in EU/RU scenario.

2017-03-30 Thread Vishal Suvagia via Review Board
> On March 26, 2017, 4:47 p.m., Nate Cole wrote: > > Any other upgrade paths other that 2.5 to 2.5,2.6? 2.4 to 2.6 or anything? Atlas upgrade from stacks 2.3 / 2.4 to stacks 2.5 and above is not supported. - Vishal --- This is an automa

Re: Review Request 57809: AMBARI-20505 : Changes needed to handle AMBARI-19623 in EU/RU scenario.

2017-03-30 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57809/ --- (Updated March 30, 2017, 1:58 p.m.) Review request for Ambari, Alejandro Fernan

Review Request 57909: AMBARI-20555 : Add Ambari cluster name for Ranger-Tagsync to sync generated atlas tags.

2017-03-24 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57909/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Re: Review Request 57809: AMBARI-20505 : Changes needed to handle AMBARI-19623 in EU/RU scenario.

2017-03-21 Thread Vishal Suvagia via Review Board
> On March 21, 2017, 8:06 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml > > Line 164 (original), 164 (patched) > > > > > >

Re: Review Request 57809: AMBARI-20505 : Changes needed to handle AMBARI-19623 in EU/RU scenario.

2017-03-21 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57809/ --- (Updated March 22, 2017, 6:22 a.m.) Review request for Ambari, Alejandro Fernan

Review Request 57809: AMBARI-20505 : Changes needed to handle AMBARI-19623 in EU/RU scenario.

2017-03-21 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57809/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Madhan Neethiraj,

Re: Review Request 57139: AMBARI-20208 : Atlas kafka servers should be configured using kafka listeners.

2017-03-02 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57139/ --- (Updated March 2, 2017, 10:52 a.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 57139: AMBARI-20208 : Atlas kafka servers should be configured using kafka listeners.

2017-03-01 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57139/ --- (Updated March 1, 2017, 12:36 p.m.) Review request for Ambari, Alejandro Fernan