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


Ship it!




Ship It!

- Dmitro Lisnichenko


On Jan. 23, 2017, 5:33 p.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55845/
> -----------------------------------------------------------
> 
> (Updated Jan. 23, 2017, 5:33 p.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko, Nate Cole, and Robert Levas.
> 
> 
> Bugs: AMBARI-19677
>     https://issues.apache.org/jira/browse/AMBARI-19677
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The {{extlib}} directory of Storm, which is managed by Ambari WRT things like 
> Atlas hook JARs, must be updated during an upgrade to point to the correct 
> versions of the JAR files.
> 
> # Deploy HDP-2.5.0.0 cluster with Ambari-2.5.0.0 (2.5.0.0-694) - secure 
> cluster via Blueprint
> # Perform HOU to 2.5.3.0 and let it complete
> # Run Storm service check
> 
> 
> During the upgrade, the pointers are changed to `current` of Atlas which 
> still points to the old JARs:
> 
> ```
> [root@r6-mams-ambari-re2-7 storm]# ls -lahrt  
> /usr/hdp/2.5.3.0-38/storm/extlib/
> total 16K
> drwxr-xr-x. 12 root root 4.0K Jan 20 06:03 ..
> lrwxrwxrwx.  1 root root   81 Jan 20 07:08 
> storm-bridge-shim-0.7.0.2.5.0.0-1245.jar -> 
> /usr/hdp/current/atlas-server/hook/storm/storm-bridge-shim-0.7.0.2.5.0.0-1245.jar
> lrwxrwxrwx.  1 root root   88 Jan 20 07:08 
> atlas-plugin-classloader-0.7.0.2.5.0.0-1245.jar -> 
> /usr/hdp/current/atlas-server/hook/storm/atlas-plugin-classloader-0.7.0.2.5.0.0-1245.jar
> drwxr-xr-x.  2 root root 4.0K Jan 20 07:08 .
> ```
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/setup_atlas_hook.py
>  eeae4bc 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
>  c74d046 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/status_params.py
>  1fd1b9b 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
>  70caaea 
>   
> ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/params_linux.py
>  d667d9f 
>   
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py
>  137f29a 
>   ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py 
> e97759b 
>   ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py bbaef78 
> 
> Diff: https://reviews.apache.org/r/55845/diff/
> 
> 
> Testing
> -------
> 
> - Performed an upgrade and downgrade with Storm/Atlas and verified that the 
> symlinks are correctly created on upgrade and reset properly on downgrade. 
> - Also, verified that a restart after the upgrade has completed correctly 
> resets the pointers to `current`:
> 2017-01-23 14:40:49,970 - Atlas Server is present on this host, will symlink 
> jars inside of /usr/hdp/current/storm-nimbus/extlib to 
> /usr/hdp/current/atlas-server/hook/storm if not already done.
> ```
> 2017-01-23 14:40:49,970 - 
> Link['/usr/hdp/current/storm-nimbus/extlib/atlas-plugin-classloader-0.7.0.2.5.2.0-67.jar']
>  {'to': 
> '/usr/hdp/current/atlas-server/hook/storm/atlas-plugin-classloader-0.7.0.2.5.2.0-67.jar'}
> 2017-01-23 14:40:49,971 - 
> Link['/usr/hdp/current/storm-nimbus/extlib/atlas-plugin-classloader-0.7.0.2.5.2.0-67.jar']
>  replacing old symlink to 
> /usr/hdp/2.5.2.0-67/atlas/hook/storm/atlas-plugin-classloader-0.7.0.2.5.2.0-67.jar
> 2017-01-23 14:40:49,973 - Creating symbolic 
> Link['/usr/hdp/current/storm-nimbus/extlib/atlas-plugin-classloader-0.7.0.2.5.2.0-67.jar']
>  to 
> /usr/hdp/current/atlas-server/hook/storm/atlas-plugin-classloader-0.7.0.2.5.2.0-67.jar
> 2017-01-23 14:40:49,973 - 
> Link['/usr/hdp/current/storm-nimbus/extlib/storm-bridge-shim-0.7.0.2.5.2.0-67.jar']
>  {'to': '/usr/hdp/current/atlas-server/hook/storm/storm-bridge-shim-
> ```
> 
> ----------------------------------------------------------------------
> Total run:1159
> Total errors:0
> Total failures:0
> OK
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 54.824 s
> [INFO] Finished at: 2017-01-23T09:57:20-05:00
> [INFO] Final Memory: 19M/245M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>

Reply via email to