Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-14 Thread Tom Beerbower


> On June 13, 2016, 8:29 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml,
> >  line 238
> > 
> >
> > Does this need to change for existing clusters?
> > What about the different stacks?
> > I don't believe ATLAS is even available on HDP 2.2

This export was added earlier for HDP 2.5 so it should not be present in any 
existing clusters.


- Tom


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


On June 13, 2016, 5:51 p.m., Tom Beerbower wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48655/
> ---
> 
> (Updated June 13, 2016, 5:51 p.m.)
> 
> 
> Review request for Ambari, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-17203
> https://issues.apache.org/jira/browse/AMBARI-17203
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Atlas atlas-application.properties configuration is required by the Atlas 
> hooks (Falcon, Storm, Sqoop and Hive).
> Currently the atlas configuration is not being pushed to all the hook hosts 
> which could lead to issues if any config is changed. The Atlas configuration 
> changes needs to be pushed to all dependent hosts.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py
>  441f0da 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/setup_atlas_falcon.py
>  67077c4 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml
>  c20523d 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py
>  fea0635 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/setup_atlas_hive.py
>  e78190f 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py
>  c154a16 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/setup_atlas_sqoop.py
>  d18d820 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
>  eadbd4a 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/setup_atlas_storm.py
>  6c3e91f 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/metainfo.xml 
> 1998131 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/metainfo.xml 
> ac1f9e7 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/SQOOP/metainfo.xml 
> eb67d63 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/metainfo.xml 
> 3faadc0 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 1bcc09e 
> 
> Diff: https://reviews.apache.org/r/48655/diff/
> 
> 
> Testing
> ---
> 
> Manual test deploy Atlas with all hook integration services on multi-host 
> cluster.  Verify Atlas configuration locations.  Update Atlas configuration.  
> Verify dependant service restart suggestion from Ambari.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>



Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Alejandro Fernandez

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




ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml
 (line 238)


Does this need to change for existing clusters?
What about the different stacks?
I don't believe ATLAS is even available on HDP 2.2


- Alejandro Fernandez


On June 13, 2016, 5:51 p.m., Tom Beerbower wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48655/
> ---
> 
> (Updated June 13, 2016, 5:51 p.m.)
> 
> 
> Review request for Ambari, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-17203
> https://issues.apache.org/jira/browse/AMBARI-17203
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Atlas atlas-application.properties configuration is required by the Atlas 
> hooks (Falcon, Storm, Sqoop and Hive).
> Currently the atlas configuration is not being pushed to all the hook hosts 
> which could lead to issues if any config is changed. The Atlas configuration 
> changes needs to be pushed to all dependent hosts.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py
>  441f0da 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/setup_atlas_falcon.py
>  67077c4 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml
>  c20523d 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py
>  fea0635 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/setup_atlas_hive.py
>  e78190f 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py
>  c154a16 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/setup_atlas_sqoop.py
>  d18d820 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
>  eadbd4a 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/setup_atlas_storm.py
>  6c3e91f 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/metainfo.xml 
> 1998131 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/metainfo.xml 
> ac1f9e7 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/SQOOP/metainfo.xml 
> eb67d63 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/metainfo.xml 
> 3faadc0 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 1bcc09e 
> 
> Diff: https://reviews.apache.org/r/48655/diff/
> 
> 
> Testing
> ---
> 
> Manual test deploy Atlas with all hook integration services on multi-host 
> cluster.  Verify Atlas configuration locations.  Update Atlas configuration.  
> Verify dependant service restart suggestion from Ambari.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>



Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread John Speidel

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


Ship it!




Ship It!

- John Speidel


On June 13, 2016, 5:51 p.m., Tom Beerbower wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48655/
> ---
> 
> (Updated June 13, 2016, 5:51 p.m.)
> 
> 
> Review request for Ambari, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-17203
> https://issues.apache.org/jira/browse/AMBARI-17203
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Atlas atlas-application.properties configuration is required by the Atlas 
> hooks (Falcon, Storm, Sqoop and Hive).
> Currently the atlas configuration is not being pushed to all the hook hosts 
> which could lead to issues if any config is changed. The Atlas configuration 
> changes needs to be pushed to all dependent hosts.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py
>  441f0da 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/setup_atlas_falcon.py
>  67077c4 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml
>  c20523d 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py
>  fea0635 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/setup_atlas_hive.py
>  e78190f 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py
>  c154a16 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/setup_atlas_sqoop.py
>  d18d820 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
>  eadbd4a 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/setup_atlas_storm.py
>  6c3e91f 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/metainfo.xml 
> 1998131 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/metainfo.xml 
> ac1f9e7 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/SQOOP/metainfo.xml 
> eb67d63 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/metainfo.xml 
> 3faadc0 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 1bcc09e 
> 
> Diff: https://reviews.apache.org/r/48655/diff/
> 
> 
> Testing
> ---
> 
> Manual test deploy Atlas with all hook integration services on multi-host 
> cluster.  Verify Atlas configuration locations.  Update Atlas configuration.  
> Verify dependant service restart suggestion from Ambari.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>



Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Nate Cole

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


Ship it!




Ship It!

- Nate Cole


On June 13, 2016, 1:51 p.m., Tom Beerbower wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48655/
> ---
> 
> (Updated June 13, 2016, 1:51 p.m.)
> 
> 
> Review request for Ambari, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-17203
> https://issues.apache.org/jira/browse/AMBARI-17203
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Atlas atlas-application.properties configuration is required by the Atlas 
> hooks (Falcon, Storm, Sqoop and Hive).
> Currently the atlas configuration is not being pushed to all the hook hosts 
> which could lead to issues if any config is changed. The Atlas configuration 
> changes needs to be pushed to all dependent hosts.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py
>  441f0da 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/setup_atlas_falcon.py
>  67077c4 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml
>  c20523d 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py
>  fea0635 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/setup_atlas_hive.py
>  e78190f 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py
>  c154a16 
>   
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/setup_atlas_sqoop.py
>  d18d820 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
>  eadbd4a 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/setup_atlas_storm.py
>  6c3e91f 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/metainfo.xml 
> 1998131 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/metainfo.xml 
> ac1f9e7 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/SQOOP/metainfo.xml 
> eb67d63 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/metainfo.xml 
> 3faadc0 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 1bcc09e 
> 
> Diff: https://reviews.apache.org/r/48655/diff/
> 
> 
> Testing
> ---
> 
> Manual test deploy Atlas with all hook integration services on multi-host 
> cluster.  Verify Atlas configuration locations.  Update Atlas configuration.  
> Verify dependant service restart suggestion from Ambari.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>



Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Tom Beerbower

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

Review request for Ambari, John Speidel and Nate Cole.


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


Repository: ambari


Description
---

Atlas atlas-application.properties configuration is required by the Atlas hooks 
(Falcon, Storm, Sqoop and Hive).
Currently the atlas configuration is not being pushed to all the hook hosts 
which could lead to issues if any config is changed. The Atlas configuration 
changes needs to be pushed to all dependent hosts.


Diffs
-

  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py
 441f0da 
  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/setup_atlas_falcon.py
 67077c4 
  
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml
 c20523d 
  
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py
 fea0635 
  
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/setup_atlas_hive.py
 e78190f 
  
ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params_linux.py
 c154a16 
  
ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/setup_atlas_sqoop.py
 d18d820 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
 eadbd4a 
  
ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/setup_atlas_storm.py
 6c3e91f 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/metainfo.xml 
1998131 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/metainfo.xml 
ac1f9e7 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/SQOOP/metainfo.xml 
eb67d63 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/STORM/metainfo.xml 
3faadc0 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
1bcc09e 

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


Testing
---

Manual test deploy Atlas with all hook integration services on multi-host 
cluster.  Verify Atlas configuration locations.  Update Atlas configuration.  
Verify dependant service restart suggestion from Ambari.

mvn clean test


Thanks,

Tom Beerbower