-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60992/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez and Sid Wagle.
Bugs: AMBARI-21530
https://issues.apache.org/jira/browse/AMBARI-21530
Repository: ambari
Description
-------
During an upgrade from BI 4.2 to HDP 2.6, some service checks were failing.
This is because the service checks were having their hooks/service folders
overwritten by some of the scheduler framework. At the time of orchestration,
the cluster desired ID was still BI but the effective ID used for the upgrade
was HDP (which was being clobbered)
Exception on running YARN service check:
```
Traceback (most recent call last):
File
"/var/lib/ambari-agent/cache/stacks/BigInsights/4.2/services/YARN/package/scripts/service_check.py",
line 91, in <module>
ServiceCheck().execute()
File
"/usr/lib/python2.6/site-packages/resource_management/libraries/script/script.py",
line 329, in execute
method(env)
File
"/var/lib/ambari-agent/cache/stacks/BigInsights/4.2/services/YARN/package/scripts/service_check.py",
line 54, in service_check
user=params.smokeuser,
File "/usr/lib/python2.6/site-packages/resource_management/core/shell.py",
line 72, in inner
```
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
2eca76cd8d
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
0b47ff1184
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/historyserver.py
7405a4e017
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
5f400a3692
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/service_check.py
a36e8ccce1
ambari-server/src/test/python/TestStackFeature.py 230734c9c7
Diff: https://reviews.apache.org/r/60992/diff/1/
Testing
-------
PENDING...
Thanks,
Jonathan Hurley
