----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59534/#review175972 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On May 24, 2017, 2:30 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59534/ > ----------------------------------------------------------- > > (Updated May 24, 2017, 2:30 p.m.) > > > Review request for Ambari and Nate Cole. > > > Bugs: AMBARI-21114 > https://issues.apache.org/jira/browse/AMBARI-21114 > > > Repository: ambari > > > Description > ------- > > The following unit tests are failing in the patch upgrade feature branch: > > ``` > Failed tests: > > StackUpgradeConfigurationMergeTest.testMergedConfigurationsDoNotAddExplicitelyRemovedProperties:261 > Nothing captured yet > UpgradeResourceProviderTest.testCreateResourcesWithAutoSkipAll:430 > expected:<[Updating configuration zookeeper-newconfig]> but was:<[Skipping > Configuration Task hdp_2_1_1_zookeeper_new_config_type]> > UpgradeResourceProviderTest.testCreateResourcesWithAutoSkipFailures:333 > expected:<[Updating configuration zookeeper-newconfig]> but was:<[Skipping > Configuration Task hdp_2_1_1_zookeeper_new_config_type]> > > UpgradeResourceProviderTest.testCreateResourcesWithAutoSkipManualVerification:383 > expected:<[Updating configuration zookeeper-newconfig]> but was:<[Skipping > Configuration Task hdp_2_1_1_zookeeper_new_config_type]> > > UpgradeResourceProviderTest.testUpdateSkipFailures:1328->testCreateResourcesWithAutoSkipFailures:333 > expected:<[Updating configuration zookeeper-newconfig]> but was:<[Skipping > Configuration Task hdp_2_1_1_zookeeper_new_config_type]> > UpgradeResourceProviderTest.testUpgradeHistory:1622 Implement me! > UpgradeActionTest.testExpressUpgradeUpdateDesiredStackAction:455 null > UpgradeActionTest.testFinalizeDowngradeAcrossStacks:742 > expected:<[COMPLET]ED> but was:<[FAIL]ED> > > UpgradeActionTest.testFinalizeDowngradeWhenDidNotFinalizePreviousUpgrade:535 > expected:<[FAIL]ED> but was:<[COMPLET]ED> > UpgradeActionTest.testFinalizeUpgradeAcrossStacks:665 > expected:<[COMPLET]ED> but was:<[FAIL]ED> > UpgradeActionTest.testFinalizeUpgradeWithClusterStateInconsistencies:832 > expected:<[COMPLET]ED> but was:<[FAIL]ED> > UpgradeHelperTest.testCompleteUpgradeOrchestration:377 expected:<[Service > Check Zk]> but was:<[Completing ZooKeeper1 Server2 on 192.168.64.1]> > UpgradeHelperTest.testPartialUpgradeOrchestration:312 expected:<3> but > was:<0> > UpgradeHelperTest.testServiceCheckUpgradeStages:1073 expected:<6> but > was:<7> > > Tests in error: > UpgradeResourceProviderTest.testCreatePartialDowngrade:649 » NoSuchElement > UpgradeResourceProviderTest.testMergeConfigurations:1201 » IllegalState > missin... > > ComponentVersionCheckActionTest.testMixedComponentVersions:341->createNewRepoVersion:273 > » Ambari > > UpgradeActionTest.testFinalizeUpgrade:548->createUpgradeClusterTargetRepo:303 > » Ambari > > UpgradeActionTest.testFinalizeWithHostsAlreadyCurrent:589->createUpgradeClusterTargetRepo:303 > » Ambari > UpgradeHelperTest.testHostGroupingOrchestration:2133 » IllegalState missing > be... > UpgradeHelperTest.testOrchestrationNoServerSideOnDowngrade:1851 » > IllegalState > UpgradeHelperTest.testResolverBadJmx:1713 » IllegalState missing behavior > defi... > UpgradeHelperTest.testResolverCaseInsensitive:1644 » IllegalState missing > beha... > UpgradeHelperTest.testResolverWithFailedUpgrade:1569 » IllegalState missing > be... > UpgradeHelperTest.testUpgradeOrchestrationWithHostsInMM:465 » IllegalState > mis... > ``` > > > Diffs > ----- > > ambari-server/docs/api/generated/index.html 1771346e3a > ambari-server/docs/api/generated/swagger.json f7dbb6bd0e > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java > 04773bcfb3 > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java > f306d69e58 > > > Diff: https://reviews.apache.org/r/59534/diff/1/ > > > Testing > ------- > > > Thanks, > > Jonathan Hurley > >
