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


Ship it!





ambari-common/src/main/python/resource_management/core/providers/package/yumrpm.py
Line 39 (original), 39 (patched)
<https://reviews.apache.org/r/60809/#comment255439>

    If you rebase, I think this was already committed.


- Jonathan Hurley


On July 12, 2017, 1:23 p.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60809/
> -----------------------------------------------------------
> 
> (Updated July 12, 2017, 1:23 p.m.)
> 
> 
> Review request for Ambari, Andrii Tkach, Dmytro Grinenko, Jonathan Hurley, 
> and Sid Wagle.
> 
> 
> Bugs: AMBARI-21455
>     https://issues.apache.org/jira/browse/AMBARI-21455
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Remove services that are not needed in BigInsights stack such as Titan and 
> SYSTEMML, and all references to JNBG, R4ML.
> 
> Also, fix a bug when calling force_remove_packages.py in the "Remove IOP 
> Select" action because the path to "rpm" should not be fully qualified.
> 
> ```
> ExecutionFailed: Execution of '/usr/bin/rpm -e --nodeps iop-select' returned 
> 127. /var/lib/ambari-agent/ambari-sudo.sh: line 50: /usr/bin/rpm: No such 
> file or directory
> ```
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/core/providers/package/yumrpm.py
>  064b504 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2.5/role_command_order.json
>  dc4811b 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2.5/services/stack_advisor.py
>  8883f57 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/role_command_order.json
>  5ee4b32 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SYSTEMML/metainfo.xml
>  b73e31e 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SYSTEMML/package/scripts/__init__.py
>  5561e10 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SYSTEMML/package/scripts/params.py
>  dd7e46c 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SYSTEMML/package/scripts/service_check.py
>  c15b907 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/SYSTEMML/package/scripts/systemml_client.py
>  2d45b68 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/configuration/titan-env.xml
>  4f80ea1 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/configuration/titan-hbase-solr.xml
>  dd45141 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/configuration/titan-log4j.xml
>  f61a479 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/kerberos.json
>  ccabbf0 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/metainfo.xml
>  73f4635 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/package/files/titanSmoke.groovy
>  79438be 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/package/scripts/params.py
>  3cb7aef 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/package/scripts/service_check.py
>  3c011a1 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/package/scripts/titan.py
>  fd94c82 
>   
> ambari-server/src/main/resources/stacks/BigInsights/4.2/services/TITAN/package/scripts/titan_client.py
>  d54ccee 
> 
> 
> Diff: https://reviews.apache.org/r/60809/diff/1/
> 
> 
> Testing
> -------
> 
> Verified on local cluster.
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>

Reply via email to