----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62871/#review187579 -----------------------------------------------------------
Ship it! Ship It! - Robert Levas On Oct. 10, 2017, 3:44 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62871/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2017, 3:44 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 > ----- > > > 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/CompatibleServicesInUpgradeCheck.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 > > > Diff: https://reviews.apache.org/r/62871/diff/1/ > > > Testing > ------- > > Manual testing of an install / upgrade with and without services which > require dependencies. > > UNIT TESTS PENDING... > > > Thanks, > > Jonathan Hurley > >
