----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53251/#review154097 -----------------------------------------------------------
ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.5.xml (line 849) <https://reviews.apache.org/r/53251/#comment223602> Did a lot of this cleanup since these were optional anyway. ambari-server/src/main/resources/upgrade-pack.xsd (line 129) <https://reviews.apache.org/r/53251/#comment223601> xs:NMTOKEN is more restrictive than normal xs:string types. It disallows empty strings and funky characters. ambari-server/src/main/resources/upgrade-pack.xsd (lines 142 - 152) <https://reviews.apache.org/r/53251/#comment223600> Noticed that XSD for execute-stage was missing, so I added it. - Jonathan Hurley On Oct. 28, 2016, 12:01 a.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53251/ > ----------------------------------------------------------- > > (Updated Oct. 28, 2016, 12:01 a.m.) > > > Review request for Ambari, Nate Cole and Robert Levas. > > > Bugs: AMBARI-18726 > https://issues.apache.org/jira/browse/AMBARI-18726 > > > Repository: ambari > > > Description > ------- > > Add a way to compute whether an execute-stage should be included, other than > the simple service/component check. > > For example some stage should only be executed if the cluster has Kerberos > enabled. This can be determined using the {{cluster-env/security_enabled}} > flag; or (better yet) the {{Cluster}}'s {{securityType}} property, which > would be set to "KERBEROS". > > Currently the only filter is based on service and component, which does not > work in the case of Kerberos since Kerberos can be enabled without needing to > install the KERBEROS service or the KERBEROS_CLIENT component. This will > happen if the Kerberos identities are managed manually rather than > configuring Ambari to do it by integrating with a supported KDC. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > d83aaa2 > > ambari-server/src/main/java/org/apache/ambari/server/state/SecurityType.java > dc52c4e > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeContext.java > 2f616e7 > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java > 3a2dc89 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java > 80bb26c > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Condition.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ConditionProcessor.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Grouping.java > be1f469 > > ambari-server/src/main/resources/stacks/HDP/2.1/upgrades/nonrolling-upgrade-2.3.xml > e3bb29d > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.2.xml > 87afdbc > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.3.xml > e082f72 > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.4.xml > ed81582 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml > 3cc0a65 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml > e8e00f0 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.4.xml > 790e5dc > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.3.xml > 01e0601 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.4.xml > 2925b64 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.5.xml > 266c71b > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml > 8b00893 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.4.xml > 6cc1106 > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.5.xml > b0b78a0 > > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.4.xml > d1e1059 > > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.5.xml > 9021761 > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.4.xml > 390f18e > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml > 6af7767 > > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.5.xml > 5e3561a > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/upgrade-2.5.xml > b02cbad > ambari-server/src/main/resources/upgrade-pack.xsd b02d941 > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java > d644a09 > > ambari-server/src/test/java/org/apache/ambari/server/state/stack/upgrade/StageWrapperBuilderTest.java > 267932c > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_direction.xml > 92fd0b2 > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_nonrolling_new_stack.xml > 789df2e > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test.xml > 5e02b15 > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_checks.xml > c6d8bbe > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_nonrolling.xml > 36f8062 > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_partial.xml > 5d8ef01 > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_to_new_stack.xml > 91770fb > ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test.xml > 021c73a > > ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test_15388.xml > d1dc62d > > ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test_checks.xml > 273f619 > > ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test_conditions.xml > PRE-CREATION > > ambari-server/src/test/resources/stacks_with_upgrade_cycle/HDP/2.2.0/upgrades/upgrade_test_15388.xml > 905def5 > > Diff: https://reviews.apache.org/r/53251/diff/ > > > Testing > ------- > > PENDING > > > Thanks, > > Jonathan Hurley > >
