Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-05 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49507/#review140828 --- Ship it! Ship It! - Alejandro Fernandez On July 5, 2016,

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-05 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49507/#review140788 --- Ship it! Ship It! - Robert Levas On July 5, 2016, 9:01

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-05 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49507/ --- (Updated July 5, 2016, 1:01 p.m.) Review request for Ambari, Alejandro

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-04 Thread Robert Levas
> On July 1, 2016, 9:45 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py, > > line 303 > > > > > > Why are we hard-coding

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-02 Thread Gautam Borad
> On July 1, 2016, 1:45 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py, > > line 303 > > > > > > Why are we hard-coding

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-01 Thread Velmurugan Periasamy
> On July 1, 2016, 1:45 p.m., Robert Levas wrote: > > Was this tested with the Storm Kerberos identity set to something like > > `storm1234@${realm}`? Looks like it is working fine. I also applied the patch and tried enabling storm plugin. Noticed Ranger policies were created with permissions

Re: Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-01 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49507/#review140346 --- Was this tested with the Storm Kerberos identity set to something

Review Request 49507: AMBARI-17520: Update the policy_user property to use storm user principal specified in Storms config

2016-07-01 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49507/ --- Review request for Ambari, Alejandro Fernandez, Mugdha Varadkar, Robert Levas,