You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Andrew Onischuk <ao...@hortonworks.com> on 2015/11/10 13:21:55 UTC

Review Request 40133: Ambari does not change permissions and owner/group for log and PID directories during install

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

Review request for Ambari and Dmitro Lisnichenko.


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


Repository: ambari


Description
-------

Steps:

  1. Before deploy were created log and PID dirs with root owners, root group and 000 permissions.
  2. Deployed cluster via blueprint.

Result: some services (HDFS, MapReduce2, YARN, Hive) were started and log, pid
dirs have valid credentials/owners/group, but a lot of services were failed
(ZooKeeper, Knox, Ambari Metrics etc.), permissions/owners/groups were not
changed.


Diffs
-----

  ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py d30a460 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py caac937 
  ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py b617313 
  ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py 4d2aaf8 
  ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py 699a868 
  ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py c78acd9 
  ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py 7add394 
  ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py ee9377f 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py 911fd1f 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py f8d5545 
  ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py 256b478 
  ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py 120f895 
  ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py fccc19e 
  ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py daf4bfe 

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


Testing
-------

mvn clean test


Thanks,

Andrew Onischuk


Re: Review Request 40133: Ambari does not change permissions and owner/group for log and PID directories during install

Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40133/#review105837
-----------------------------------------------------------

Ship it!


Ship It!

- Dmitro Lisnichenko


On Nov. 10, 2015, 2:21 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40133/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2015, 2:21 p.m.)
> 
> 
> Review request for Ambari and Dmitro Lisnichenko.
> 
> 
> Bugs: AMBARI-13808
>     https://issues.apache.org/jira/browse/AMBARI-13808
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Steps:
> 
>   1. Before deploy were created log and PID dirs with root owners, root group and 000 permissions.
>   2. Deployed cluster via blueprint.
> 
> Result: some services (HDFS, MapReduce2, YARN, Hive) were started and log, pid
> dirs have valid credentials/owners/group, but a lot of services were failed
> (ZooKeeper, Knox, Ambari Metrics etc.), permissions/owners/groups were not
> changed.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/ACCUMULO/1.6.1.2.2.0/package/scripts/accumulo_configuration.py d30a460 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py caac937 
>   ambari-server/src/main/resources/common-services/FLUME/1.4.0.2.0/package/scripts/flume.py b617313 
>   ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py 4d2aaf8 
>   ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py 699a868 
>   ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py c78acd9 
>   ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py 7add394 
>   ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py ee9377f 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_master.py 911fd1f 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_hbase_regionserver.py f8d5545 
>   ambari-server/src/test/python/stacks/2.0.6/HBASE/test_phoenix_queryserver.py 256b478 
>   ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_client.py 120f895 
>   ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py fccc19e 
>   ambari-server/src/test/python/stacks/2.1/STORM/test_storm_base.py daf4bfe 
> 
> Diff: https://reviews.apache.org/r/40133/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>