Repository: incubator-slider
Updated Branches:
  refs/heads/develop cc7a644ea -> 720bd19df


SLIDER-1233 Lost nodes should not contribute to container failures (fix a 
fun-test)


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

Branch: refs/heads/develop
Commit: 720bd19df9d2c0ad74cd536802a3c0028f7cfa96
Parents: cc7a644
Author: Gour Saha <gourks...@apache.org>
Authored: Wed Oct 11 17:25:23 2017 -0700
Committer: Gour Saha <gourks...@apache.org>
Committed: Wed Oct 11 17:25:23 2017 -0700

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


http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/720bd19d/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 e18a757..7e9ebf5 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
@@ -161,9 +161,10 @@ public class AppsUpgradeIT extends AgentCommandTestBase
         ])
 
     // verify
-    describe("COMMAND_LOGGER container failed count should reach 1")
-    expectFailedContainerCountReached(APPLICATION_NAME, COMMAND_LOGGER, 1,
-        CONTAINER_LAUNCH_TIMEOUT)
+    describe("COMMAND_LOGGER container failed count should remain 0, since "
+      + "container kills during upgrade is not counted as failures")
+    def failedCount = queryFailedCount(APPLICATION_NAME, COMMAND_LOGGER)
+    assert failedCount == 0
     describe("COMMAND_LOGGER container request count should reach 1")
     expectContainerRequestedCountReached(APPLICATION_NAME, COMMAND_LOGGER, 1,
         CONTAINER_LAUNCH_TIMEOUT)

Reply via email to