You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@slider.apache.org by go...@apache.org on 2015/06/14 20:02:45 UTC

incubator-slider git commit: SLIDER-787 App Upgrade/Reconfig support in Slider (comment tricky assert)

Repository: incubator-slider
Updated Branches:
  refs/heads/develop 9900d7b31 -> 3ee3d1ea3


SLIDER-787 App Upgrade/Reconfig support in Slider (comment tricky assert)


Project: http://git-wip-us.apache.org/repos/asf/incubator-slider/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-slider/commit/3ee3d1ea
Tree: http://git-wip-us.apache.org/repos/asf/incubator-slider/tree/3ee3d1ea
Diff: http://git-wip-us.apache.org/repos/asf/incubator-slider/diff/3ee3d1ea

Branch: refs/heads/develop
Commit: 3ee3d1ea373c79becfe29f3c8ac7220284c3439f
Parents: 9900d7b
Author: Gour Saha <go...@apache.org>
Authored: Sun Jun 14 11:00:58 2015 -0700
Committer: Gour Saha <go...@apache.org>
Committed: Sun Jun 14 11:00:58 2015 -0700

----------------------------------------------------------------------
 .../org/apache/slider/funtest/lifecycle/AppsUpgradeIT.groovy   | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/3ee3d1ea/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/AppsUpgradeIT.groovy
----------------------------------------------------------------------
diff --git a/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/AppsUpgradeIT.groovy b/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/AppsUpgradeIT.groovy
index db79351..9fc8e6a 100644
--- a/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/AppsUpgradeIT.groovy
+++ b/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/AppsUpgradeIT.groovy
@@ -115,8 +115,10 @@ public class AppsUpgradeIT extends AgentCommandTestBase
             ARG_FORCE
         ])
 
-    describe("Check Slider AM goes down and then comes back up")
-    ensureApplicationNotInState(APPLICATION_NAME, YarnApplicationState.RUNNING)
+    // This is a very time constrained check, so disabling it. Catching the
+    // app not in RUNNING state after AM restart, is like racing with RM.
+//    describe("Check Slider AM goes down and then comes back up")
+//    ensureApplicationNotInState(APPLICATION_NAME, YarnApplicationState.RUNNING)
     // Spin till the Slider AM is back up. Note: COMMAND_LOGGER
     // container count goes down to 1 here (due to spec change)
     ensureYarnApplicationIsUp(appId)