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


Ship it!




Ship It!

- Sid Wagle


On July 13, 2017, 11:16 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60855/
> -----------------------------------------------------------
> 
> (Updated July 13, 2017, 11:16 p.m.)
> 
> 
> Review request for Ambari, Andrii Tkach, Dmytro Grinenko, Jonathan Hurley, 
> and Sid Wagle.
> 
> 
> Bugs: AMBARI-21463
>     https://issues.apache.org/jira/browse/AMBARI-21463
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Oozie Server restart failed due to this: Unable to copy 
> /usr/share/HDP-oozie/ext-2.2.zip because it does not exist
> 
> Doesn't look like HDP rpms created this path:
> ```
> [root@sid-test-2 ~]# ls -l /var/lib/oozie/ext-2.2.zip
> -rwxr-xr-x. 1 oozie hadoop 6800612 Jul  5 18:03 /var/lib/oozie/ext-2.2.zip
> [root@sid-test-2 ~]# ls -l /usr/hdp/2.6.1.0-129/oozie/libext/ext-2.2.zip
> -rw-r--r--. 1 oozie hadoop 6800612 Jul  6 16:36 
> /usr/hdp/2.6.1.0-129/oozie/libext/ext-2.2.zip
> ```
> 
> The ext2js rpm seems to come from IOPUtils:
> ```
> [root@sid-test-2 oozie]# yum list | grep extjs
> extjs.noarch                              2.2_IBM_2-1                
> @IOP-UTILS-1.3
> 
> [root@sid-test-2 oozie]# rpm -qa | grep extjs
> extjs-2.2_IBM_2-1.noarch
> ```
> 
> We should swap the source from
> /usr/share/HDP-oozie/ext-2.2.zip
> to
> /usr/share/BIGINSIGHTS-oozie/ext-2.2.zip
> 
> since the latter does exist.
> 
> Also, restarting Oozie Clients during EU is failing because stack_tools.py is 
> missing the "get_stack_name" function in the __all__ variable.
> Lastly, disable showing the BigInsights stack by default in the UI.
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/stack_tools.py
>  830598b 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
>  0c38b0b 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server.py
>  2cf3313 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py
>  a1f3336 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
>  0c30c78 
>   ambari-server/src/main/resources/stacks/BigInsights/4.2.5/metainfo.xml 
> 3cf364e 
> 
> 
> Diff: https://reviews.apache.org/r/60855/diff/1/
> 
> 
> Testing
> -------
> 
> Verified on local cluster.
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>

Reply via email to