You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Ivan Kozlov <ik...@hortonworks.com> on 2014/03/03 18:49:38 UTC

Re: Review Request 18628: Refactor temporary shared resource for HDFS and MR initialization

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

(Updated March 3, 2014, 5:49 p.m.)


Review request for Ambari and Andrew Onischuk.


Changes
-------

Diff for 1-st and 2-nd stack


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


Repository: ambari


Description
-------

Refactor (and remove if possible) the resource that has been created in a previous jira. All logic specific to MR should be moved to MR scripts. The same thing for HDFS.

Partially done. Diff for 1-st stack.


Diffs (updated)
-----

  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/hook.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/params.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/shared_initialization.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/templates/hadoop-env.sh.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py fa19ca3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py 4e51c17 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/hook.py e11bfac 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/params.py 61ad367 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/shared_initialization.py 89b29c6 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hadoop-env.sh.j2 51e2bac 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hdfs.conf.j2 ca7baa2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/slaves.j2 cbcf6c3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/taskcontroller.cfg.j2 d01d37e 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/datanode.py 57fdb35 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_client.py 8180689 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_datanode.py 50ef5e5 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_namenode.py 0397817 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_snamenode.py dcd7ac4 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 98d536c 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/metainfo.xml 643b64c 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/mapreduce.py 95ace16 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/params.py ac1e5ff 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py d938e74 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py f786eed 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/hook.py e11bfac 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py 38494da 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/shared_initialization.py a8d2fa3 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/hdfs.conf.j2 ca7baa2 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/slaves.j2 cbcf6c3 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/taskcontroller.cfg.j2 d01d37e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/datanode.py 57fdb35 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_client.py ec24c7d 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_datanode.py 5ad9eeb 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_namenode.py 86a7913 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_snamenode.py dcd7ac4 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py d55488d 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/params.py 37d467f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/yarn.py 108eb89 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_datanode.py b096a0d 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_namenode.py 6efa948 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_snamenode.py 058c19a 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_client.py a0d3dca 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_historyserver.py 7a11d6e 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_jobtracker.py afc6d3f 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_tasktracker.py 601bdd0 
  ambari-server/src/test/python/stacks/1.3.2/hooks/after-INSTALL/test_after_install.py PRE-CREATION 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-INSTALL/test_before_install.py 3867f48 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-START/test_before_start.py 2663839 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 39374e8 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_namenode.py 8e94be3 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_snamenode.py 45b84bd 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py 98bc3a6 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py ef5c05f 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py a28a47e 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py f8ff29e 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_client.py 59ac114 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py be5c23e 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-START/test_before_start.py 88448e6 
  ambari-server/src/test/python/stacks/2.1.1/YARN/test_apptimelineserver.py fc1f547 

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


Testing
-------

security and default deployment on multinode cluster. deployment of mr, hdfs and "only client" components on separate hosts (default and secure cluster).


Thanks,

Ivan Kozlov


Re: Review Request 18628: Refactor temporary shared resource for HDFS and MR initialization

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



ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/hook.py
<https://reviews.apache.org/r/18628/#comment67156>

    Add comment (this is for only client hosts which ...)



ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/shared_initialization.py
<https://reviews.apache.org/r/18628/#comment67155>

    Try to remove this 2 dirs from afterinstall



ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/params.py
<https://reviews.apache.org/r/18628/#comment67157>

    Le'ts remove this comments



ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/shared_initialization.py
<https://reviews.apache.org/r/18628/#comment67158>

    let's remove useless for



ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_client.py
<https://reviews.apache.org/r/18628/#comment67159>

    we should call this from configure, since we may have configure command in future



ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_datanode.py
<https://reviews.apache.org/r/18628/#comment67160>

    Move this all calls to DataNode class configure, the same for others



ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/metainfo.xml
<https://reviews.apache.org/r/18628/#comment67161>

    what's that? can we remove this?



ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/params.py
<https://reviews.apache.org/r/18628/#comment67162>

    remove comment



ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/params.py
<https://reviews.apache.org/r/18628/#comment67163>

    -/-


Let's also call AFTER-INSTALL from BEFORE-START, and add proper comment for this things

- Andrew Onischuk


On March 3, 2014, 5:49 p.m., Ivan Kozlov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18628/
> -----------------------------------------------------------
> 
> (Updated March 3, 2014, 5:49 p.m.)
> 
> 
> Review request for Ambari and Andrew Onischuk.
> 
> 
> Bugs: AMBARI-4885
>     https://issues.apache.org/jira/browse/AMBARI-4885
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Refactor (and remove if possible) the resource that has been created in a previous jira. All logic specific to MR should be moved to MR scripts. The same thing for HDFS.
> 
> Partially done. Diff for 1-st stack.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/hook.py PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/params.py PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/shared_initialization.py PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/templates/hadoop-env.sh.j2 PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py fa19ca3 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py 4e51c17 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/hook.py e11bfac 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/params.py 61ad367 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/shared_initialization.py 89b29c6 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hadoop-env.sh.j2 51e2bac 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hdfs.conf.j2 ca7baa2 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/slaves.j2 cbcf6c3 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/taskcontroller.cfg.j2 d01d37e 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/datanode.py 57fdb35 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_client.py 8180689 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_datanode.py 50ef5e5 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_namenode.py 0397817 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_snamenode.py dcd7ac4 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 98d536c 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/metainfo.xml 643b64c 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/mapreduce.py 95ace16 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/params.py ac1e5ff 
>   ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py d938e74 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py f786eed 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/hook.py e11bfac 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py 38494da 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/shared_initialization.py a8d2fa3 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/hdfs.conf.j2 ca7baa2 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/slaves.j2 cbcf6c3 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/taskcontroller.cfg.j2 d01d37e 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/datanode.py 57fdb35 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_client.py ec24c7d 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_datanode.py 5ad9eeb 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_namenode.py 86a7913 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_snamenode.py dcd7ac4 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py d55488d 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/params.py 37d467f 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/yarn.py 108eb89 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
>   ambari-server/src/test/python/stacks/1.3.2/HDFS/test_datanode.py b096a0d 
>   ambari-server/src/test/python/stacks/1.3.2/HDFS/test_namenode.py 6efa948 
>   ambari-server/src/test/python/stacks/1.3.2/HDFS/test_snamenode.py 058c19a 
>   ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_client.py a0d3dca 
>   ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_historyserver.py 7a11d6e 
>   ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_jobtracker.py afc6d3f 
>   ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_tasktracker.py 601bdd0 
>   ambari-server/src/test/python/stacks/1.3.2/hooks/after-INSTALL/test_after_install.py PRE-CREATION 
>   ambari-server/src/test/python/stacks/1.3.2/hooks/before-INSTALL/test_before_install.py 3867f48 
>   ambari-server/src/test/python/stacks/1.3.2/hooks/before-START/test_before_start.py 2663839 
>   ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 39374e8 
>   ambari-server/src/test/python/stacks/2.0.6/HDFS/test_namenode.py 8e94be3 
>   ambari-server/src/test/python/stacks/2.0.6/HDFS/test_snamenode.py 45b84bd 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py 98bc3a6 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py ef5c05f 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py a28a47e 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py f8ff29e 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_client.py 59ac114 
>   ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py be5c23e 
>   ambari-server/src/test/python/stacks/2.0.6/hooks/before-START/test_before_start.py 88448e6 
>   ambari-server/src/test/python/stacks/2.1.1/YARN/test_apptimelineserver.py fc1f547 
> 
> Diff: https://reviews.apache.org/r/18628/diff/
> 
> 
> Testing
> -------
> 
> security and default deployment on multinode cluster. deployment of mr, hdfs and "only client" components on separate hosts (default and secure cluster).
> 
> 
> Thanks,
> 
> Ivan Kozlov
> 
>


Re: Review Request 18628: Refactor temporary shared resource for HDFS and MR initialization

Posted by Ivan Kozlov <ik...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18628/
-----------------------------------------------------------

(Updated April 16, 2014, 3:14 p.m.)


Review request for Ambari and Andrew Onischuk.


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


Repository: ambari


Description
-------

Refactor (and remove if possible) the resource that has been created in a previous jira. All logic specific to MR should be moved to MR scripts. The same thing for HDFS.

Partially done. Diff for 1-st stack.


Diffs (updated)
-----

  ambari-agent/src/main/python/resource_management/libraries/script/hook.py 19b5204 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/hook.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/params.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/shared_initialization.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/templates/hadoop-env.sh.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py fa19ca3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py 6a37d02 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-RESTART/scripts/hook.py 7042602 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/hook.py 075a6b6 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/params.py 2db9b4b 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/shared_initialization.py c6cc432 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hadoop-env.sh.j2 76ac3f3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hdfs.conf.j2 d58a6f5 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/slaves.j2 4a9e713 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/taskcontroller.cfg.j2 3d5f4f2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/datanode.py 57fdb35 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_client.py 40bde89 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/namenode.py 2f26c98 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 98d536c 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/snamenode.py b2a3bd1 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/mapreduce.py 4b26814 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/params.py 1a77f95 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py 341d86f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py c7c0c70 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-RESTART/scripts/hook.py 7042602 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/hook.py 075a6b6 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py f1ef84a 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/shared_initialization.py f132c2e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/hdfs.conf.j2 d58a6f5 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/slaves.j2 4a9e713 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/taskcontroller.cfg.j2 3d5f4f2 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/datanode.py 57fdb35 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_client.py 6220326 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/journalnode.py a707e2e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/namenode.py 2179292 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py d55488d 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/snamenode.py b2a3bd1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/zkfc_slave.py f415f24 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/params.py 64e8f81 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/yarn.py 4c71566 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_datanode.py 6f51a8c 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_namenode.py 6efa948 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_snamenode.py 058c19a 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_client.py d581903 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_historyserver.py d22f4ba 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_jobtracker.py 255ba1a 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_tasktracker.py bab7f71 
  ambari-server/src/test/python/stacks/1.3.2/hooks/after-INSTALL/test_after_install.py PRE-CREATION 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-INSTALL/test_before_install.py 3867f48 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-START/test_before_start.py 1ef57aa 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 4ee2571 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_journalnode.py 380f0f8 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_namenode.py 9c659ea 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_snamenode.py fc96be5 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_zkfc.py d33e063 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/_test_hive_service_check.py PRE-CREATION 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_service_check.py 00c24d4 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py 25525c4 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py 95d5b48 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py e966b46 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py e5f38b7 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_client.py 25c72f6 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py 8559db0 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-START/test_before_start.py 7efb056 
  ambari-server/src/test/python/stacks/2.1/YARN/test_apptimelineserver.py 0a35276 

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


Testing
-------

security and default deployment on multinode cluster. deployment of mr, hdfs and "only client" components on separate hosts (default and secure cluster).


Thanks,

Ivan Kozlov


Re: Review Request 18628: Refactor temporary shared resource for HDFS and MR initialization

Posted by Ivan Kozlov <ik...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18628/
-----------------------------------------------------------

(Updated April 15, 2014, 4:54 p.m.)


Review request for Ambari and Andrew Onischuk.


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


Repository: ambari


Description
-------

Refactor (and remove if possible) the resource that has been created in a previous jira. All logic specific to MR should be moved to MR scripts. The same thing for HDFS.

Partially done. Diff for 1-st stack.


Diffs (updated)
-----

  ambari-agent/src/main/python/resource_management/libraries/script/hook.py 19b5204 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/hook.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/params.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/scripts/shared_initialization.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/after-INSTALL/templates/hadoop-env.sh.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/params.py fa19ca3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-INSTALL/scripts/shared_initialization.py 6a37d02 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-RESTART/scripts/hook.py 7042602 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/hook.py 4ee2214 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/params.py 2db9b4b 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/scripts/shared_initialization.py c5828d3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hadoop-env.sh.j2 76ac3f3 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/hdfs.conf.j2 d58a6f5 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/slaves.j2 4a9e713 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/hooks/before-START/templates/taskcontroller.cfg.j2 3d5f4f2 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/datanode.py 57fdb35 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/hdfs_client.py 40bde89 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/namenode.py 2f26c98 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/params.py 98d536c 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/scripts/snamenode.py b2a3bd1 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/mapreduce.py 94fe24e 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/scripts/params.py 1a77f95 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/MAPREDUCE/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/hook.py 51e5cd2 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/params.py 341d86f 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-INSTALL/scripts/shared_initialization.py c7c0c70 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-RESTART/scripts/hook.py 7042602 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/hook.py 4ee2214 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py f1ef84a 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/shared_initialization.py 3ffcc76 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/hdfs.conf.j2 d58a6f5 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/slaves.j2 4a9e713 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/templates/taskcontroller.cfg.j2 3d5f4f2 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/datanode.py 57fdb35 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs.py PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/hdfs_client.py 6220326 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/journalnode.py a707e2e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/namenode.py 2179292 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/params.py d55488d 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/snamenode.py b2a3bd1 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/scripts/zkfc_slave.py f415f24 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/hdfs.conf.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HDFS/package/templates/slaves.j2 PRE-CREATION 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/params.py 64e8f81 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/scripts/yarn.py 4c71566 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/YARN/package/templates/taskcontroller.cfg.j2 PRE-CREATION 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_datanode.py 6f51a8c 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_namenode.py 6efa948 
  ambari-server/src/test/python/stacks/1.3.2/HDFS/test_snamenode.py 058c19a 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_client.py 3606595 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_historyserver.py 7367b82 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_jobtracker.py faa27fd 
  ambari-server/src/test/python/stacks/1.3.2/MAPREDUCE/test_mapreduce_tasktracker.py d725c90 
  ambari-server/src/test/python/stacks/1.3.2/hooks/after-INSTALL/test_after_install.py PRE-CREATION 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-INSTALL/test_before_install.py 3867f48 
  ambari-server/src/test/python/stacks/1.3.2/hooks/before-START/test_before_start.py 2663839 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_datanode.py 4ee2571 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_journalnode.py 380f0f8 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_namenode.py 9c659ea 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_snamenode.py fc96be5 
  ambari-server/src/test/python/stacks/2.0.6/HDFS/test_zkfc.py d33e063 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/_test_hive_service_check.py PRE-CREATION 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_service_check.py 00c24d4 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py 25525c4 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py 95d5b48 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py e966b46 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py e5f38b7 
  ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_client.py 25c72f6 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-INSTALL/test_before_install.py 8559db0 
  ambari-server/src/test/python/stacks/2.0.6/hooks/before-START/test_before_start.py 88448e6 
  ambari-server/src/test/python/stacks/2.1/YARN/test_apptimelineserver.py 0a35276 

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


Testing
-------

security and default deployment on multinode cluster. deployment of mr, hdfs and "only client" components on separate hosts (default and secure cluster).


Thanks,

Ivan Kozlov