Re: Review Request 61501: Prevent users from authenticating if they exceed a configured number of login failures

2017-08-11 Thread Attila Magyar
> On Aug. 10, 2017, 6:26 p.m., Robert Levas wrote: > > Would it have been possble to add the lockout logic in > > `org.apache.ambari.server.security.authentication.AmbariAuthenticationEventHandlerImpl#onSuccessfulAuthentication`? > > I am not sure if chaning the _success_ to a _failure_ is pos

Re: Review Request 61501: Prevent users from authenticating if they exceed a configured number of login failures

2017-08-11 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61501/ --- (Updated Aug. 11, 2017, 7:32 a.m.) Review request for Ambari, Balázs Bence Sári

Review Request 61592: AMBARI-21708. History Server cannot be started due to wrong permissions of /mr-history

2017-08-11 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61592/ --- Review request for Ambari, Andrew Onischuk, Sebastian Toader, and Vitalyi Brodet

Re: Review Request 61592: AMBARI-21708. History Server cannot be started due to wrong permissions of /mr-history

2017-08-11 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61592/#review182671 --- Ship it! Ship It! - Sebastian Toader On Aug. 11, 2017, 11:07

Re: Review Request 61501: Prevent users from authenticating if they exceed a configured number of login failures

2017-08-11 Thread Robert Levas
> On Aug. 10, 2017, 2:26 p.m., Robert Levas wrote: > > Would it have been possble to add the lockout logic in > > `org.apache.ambari.server.security.authentication.AmbariAuthenticationEventHandlerImpl#onSuccessfulAuthentication`? > > I am not sure if chaning the _success_ to a _failure_ is pos

Re: Review Request 61501: Prevent users from authenticating if they exceed a configured number of login failures

2017-08-11 Thread Attila Magyar
> On Aug. 10, 2017, 6:26 p.m., Robert Levas wrote: > > Would it have been possble to add the lockout logic in > > `org.apache.ambari.server.security.authentication.AmbariAuthenticationEventHandlerImpl#onSuccessfulAuthentication`? > > I am not sure if chaning the _success_ to a _failure_ is pos

Re: Review Request 61501: Prevent users from authenticating if they exceed a configured number of login failures

2017-08-11 Thread Robert Levas
> On Aug. 10, 2017, 2:26 p.m., Robert Levas wrote: > > Would it have been possble to add the lockout logic in > > `org.apache.ambari.server.security.authentication.AmbariAuthenticationEventHandlerImpl#onSuccessfulAuthentication`? > > I am not sure if chaning the _success_ to a _failure_ is pos

Re: Review Request 61582: Not able to start Yarn services after restoring the configs to initial value

2017-08-11 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61582/#review182722 --- ambari-server/src/main/java/org/apache/ambari/server/state/cluste

Re: Review Request 61592: AMBARI-21708. History Server cannot be started due to wrong permissions of /mr-history

2017-08-11 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61592/#review182723 --- ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/p

Re: Review Request 61592: AMBARI-21708. History Server cannot be started due to wrong permissions of /mr-history

2017-08-11 Thread Attila Doroszlai
> On Aug. 11, 2017, 7:41 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py > > Lines 330 (patched) > > > > > > Also change this in HDP 3.0 T