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

(Updated Oct. 10, 2017, 4:42 p.m.)


Review request for Ambari, Dmitro Lisnichenko, Nate Cole, and Robert Levas.


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


Repository: ambari


Description
-------

Some stacks, such as HDP, do not support service-level upgrades for some 
services since they have hard coded dependencies with other services which 
might not be in the upgrade. 

For example, if YARN is configured to use Tez, then an upgrade of YARN or Tez 
must also include Tez or YARN respectively. This is because the path is hard 
coded inside of YARN to use its version and cannot be changed via any external 
environment variable.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/checks/AbstractCheckDescriptor.java
 d54e8a176a 
  
ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java
 314ab83718 
  
ambari-server/src/main/java/org/apache/ambari/server/checks/RequiredServicesInRepositoryCheck.java
 PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/checks/UpgradeCheck.java 
442bf0be12 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
 a2fb44199f 
  
ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryType.java 
954f705e9f 
  
ambari-server/src/main/java/org/apache/ambari/server/state/repository/VersionDefinitionXml.java
 c17f5e90f9 
  
ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_packages.json
 2db0a74195 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/AbstractCheckDescriptorTest.java
 092bd0bd88 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/AutoStartDisabledCheckTest.java
 671648ab71 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ClientRetryPropertyCheckTest.java
 206b4519c1 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ComponentExistsInRepoCheckTest.java
 ede1daacff 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ComponentsInstallationCheckTest.java
 4ae33f8bbf 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ConfigurationMergeCheckTest.java
 df7bd1fb70 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/DruidHighAvailabilityCheckTest.java
 e90931bc11 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/HiveMultipleMetastoreCheckTest.java
 7a1fca3161 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/HiveNotRollingWarningTest.java
 3b0c900991 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/HostsHeartbeatCheckTest.java
 daf2e4cb76 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/HostsMasterMaintenanceCheckTest.java
 8afa1eaf89 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/HostsRepositoryVersionCheckTest.java
 5539618c28 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/InstallPackagesCheckTest.java
 cdfbb0a790 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/KafkaKerberosCheckTest.java
 9a95c42327 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/MapReduce2JobHistoryStatePreservingCheckTest.java
 fc322b3d91 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/RangerAuditDbCheckTest.java
 ec91e483b3 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/RangerPasswordCheckTest.java
 f9569c3820 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/RangerSSLConfigCheckTest.java
 af016b01e0 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/RequiredServicesInRepositoryCheckTest.java
 PRE-CREATION 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/SecondaryNamenodeDeletedCheckTest.java
 5d0ca5f03d 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesMaintenanceModeCheckTest.java
 65776519c8 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesMapReduceDistributedCacheCheckTest.java
 a8569cc61f 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesNamenodeHighAvailabilityCheckTest.java
 c7e8fd63f1 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesNamenodeTruncateCheckTest.java
 b252ec56b0 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesTezDistributedCacheCheckTest.java
 96eefc1cb8 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesUpCheckTest.java
 7cf0a38b87 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/ServicesYarnWorkPreservingCheckTest.java
 331223d0e6 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/StormShutdownWarningTest.java
 1441256719 
  
ambari-server/src/test/java/org/apache/ambari/server/state/CheckHelperTest.java 
27b248f2a8 


Diff: https://reviews.apache.org/r/62871/diff/2/

Changes: https://reviews.apache.org/r/62871/diff/1-2/


Testing (updated)
-------

Manual testing of an install / upgrade with and without services which require 
dependencies. 

[INFO] Tests run: 4917, Failures: 0, Errors: 0, Skipped: 34
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 22:24 min
[INFO] Finished at: 2017-10-10T16:08:48-04:00
[INFO] Final Memory: 61M/876M
[INFO] ------------------------------------------------------------------------


Thanks,

Jonathan Hurley

Reply via email to