-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55987/
-----------------------------------------------------------

(Updated Jan. 27, 2017, 4:02 p.m.)


Review request for Ambari, Attila Doroszlai, Sandor Magyari, and Sebastian 
Toader.


Changes
-------

Fixed file permissions (permissions set after creation, thus umask settings 
don't change them)


Bugs: AMBARI-19726
    https://issues.apache.org/jira/browse/AMBARI-19726


Repository: ambari


Description
-------

The post user creation hook had wired the **hdfs** user, user homes were 
created in the name of this user.
In case of customized clusters this didn;t work due to permssion issues.

The solution is to get the proper hdfs_user from the configureation and access 
the dfs as this user.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/hooks/users/UserHookParams.java
 6970dcc 
  
ambari-server/src/main/java/org/apache/ambari/server/hooks/users/UserHookService.java
 c4ff1e4 
  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/users/CsvFilePersisterService.java
 fe6bf35 
  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/users/PostUserCreationHookServerAction.java
 e713128 
  ambari-server/src/main/resources/scripts/post-user-creation-hook.sh ee8d2d1 
  
ambari-server/src/test/java/org/apache/ambari/server/hooks/users/UserHookServiceTest.java
 293b22a 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/users/PostUserCreationHookServerActionTest.java
 f5cdf48 

Diff: https://reviews.apache.org/r/55987/diff/


Testing
-------

Unit tests running.
Manual testing underway


Thanks,

Laszlo Puskas

Reply via email to