You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Andrew Onischuk <ao...@hortonworks.com> on 2017/05/24 18:43:19 UTC

Review Request 59535: Setting yarn.node-labels.fs-store.root-dir to a "path" changes the permission of the "root path"

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

Review request for Ambari and Mahadev Konar.


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


Repository: ambari


Description
-------

1\. Set the following configs to run nodeLabels test via Ambari Rest Call

    
    
    
     yarnProperties = {'yarn.acl.enable': 'true',
                          'yarn.node-labels.enabled' : "True",
                          'yarn.node-labels.fs-store.root-dir': NODE_LABEL_STORE_DIR,
                          'yarn.admin.acl': yarn_user + ',' + qa_user}
    

where NODE_LABEL_STORE_DIR = "/tmp/node-labels"  
2Restart ResourceManager and NodeManagers via ambari  
3\. After running these commands, /tmp directory's permission changes to  
[drwx------ - yarn hadoop]BUG/drwx-
-----+++-+yarn+++hadoop) from [drwxrwxrwx - hdfs hadoop drwxrwxrwx+++-+hdfs+++hadoop) causing other tests to
fail since it cannot access /tmp


Diffs
-----

  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py 3e5c51d 


Diff: https://reviews.apache.org/r/59535/diff/1/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk

Re: Review Request 59535: Setting yarn.node-labels.fs-store.root-dir to a "path" changes the permission of the "root path"

Posted by Vitalyi Brodetskyi <vb...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59535/#review175973
-----------------------------------------------------------


Ship it!




Ship It!

- Vitalyi Brodetskyi


On Травень 24, 2017, 6:43 після полудня, Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59535/
> -----------------------------------------------------------
> 
> (Updated Травень 24, 2017, 6:43 після полудня)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-21115
>     https://issues.apache.org/jira/browse/AMBARI-21115
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1\. Set the following configs to run nodeLabels test via Ambari Rest Call
> 
>     
>     
>     
>      yarnProperties = {'yarn.acl.enable': 'true',
>                           'yarn.node-labels.enabled' : "True",
>                           'yarn.node-labels.fs-store.root-dir': NODE_LABEL_STORE_DIR,
>                           'yarn.admin.acl': yarn_user + ',' + qa_user}
>     
> 
> where NODE_LABEL_STORE_DIR = "/tmp/node-labels"  
> 2Restart ResourceManager and NodeManagers via ambari  
> 3\. After running these commands, /tmp directory's permission changes to  
> [drwx------ - yarn hadoop]BUG/drwx-
> -----+++-+yarn+++hadoop) from [drwxrwxrwx - hdfs hadoop drwxrwxrwx+++-+hdfs+++hadoop) causing other tests to
> fail since it cannot access /tmp
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py 3e5c51d 
> 
> 
> Diff: https://reviews.apache.org/r/59535/diff/1/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>

Re: Review Request 59535: Setting yarn.node-labels.fs-store.root-dir to a "path" changes the permission of the "root path"

Posted by Mahadev Konar <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59535/#review175975
-----------------------------------------------------------


Ship it!




Ship It!

- Mahadev Konar


On May 24, 2017, 6:55 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59535/
> -----------------------------------------------------------
> 
> (Updated May 24, 2017, 6:55 p.m.)
> 
> 
> Review request for Ambari and Mahadev Konar.
> 
> 
> Bugs: AMBARI-21116
>     https://issues.apache.org/jira/browse/AMBARI-21116
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1\. Set the following configs to run nodeLabels test via Ambari Rest Call
> 
>     
>     
>     
>      yarnProperties = {'yarn.acl.enable': 'true',
>                           'yarn.node-labels.enabled' : "True",
>                           'yarn.node-labels.fs-store.root-dir': NODE_LABEL_STORE_DIR,
>                           'yarn.admin.acl': yarn_user + ',' + qa_user}
>     
> 
> where NODE_LABEL_STORE_DIR = "/tmp/node-labels"  
> 2Restart ResourceManager and NodeManagers via ambari  
> 3\. After running these commands, /tmp directory's permission changes to  
> [drwx------ - yarn hadoop]BUG/drwx-
> -----+++-+yarn+++hadoop) from [drwxrwxrwx - hdfs hadoop drwxrwxrwx+++-+hdfs+++hadoop) causing other tests to
> fail since it cannot access /tmp
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py 3e5c51d 
> 
> 
> Diff: https://reviews.apache.org/r/59535/diff/1/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>

Re: Review Request 59535: Setting yarn.node-labels.fs-store.root-dir to a "path" changes the permission of the "root path"

Posted by Andrew Onischuk <ao...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59535/
-----------------------------------------------------------

(Updated May 24, 2017, 6:55 p.m.)


Review request for Ambari and Mahadev Konar.


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


Repository: ambari


Description
-------

1\. Set the following configs to run nodeLabels test via Ambari Rest Call

    
    
    
     yarnProperties = {'yarn.acl.enable': 'true',
                          'yarn.node-labels.enabled' : "True",
                          'yarn.node-labels.fs-store.root-dir': NODE_LABEL_STORE_DIR,
                          'yarn.admin.acl': yarn_user + ',' + qa_user}
    

where NODE_LABEL_STORE_DIR = "/tmp/node-labels"  
2Restart ResourceManager and NodeManagers via ambari  
3\. After running these commands, /tmp directory's permission changes to  
[drwx------ - yarn hadoop]BUG/drwx-
-----+++-+yarn+++hadoop) from [drwxrwxrwx - hdfs hadoop drwxrwxrwx+++-+hdfs+++hadoop) causing other tests to
fail since it cannot access /tmp


Diffs
-----

  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py 3e5c51d 


Diff: https://reviews.apache.org/r/59535/diff/1/


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk