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 2017/10/12 00:54:02 UTC
incubator-slider git commit: SLIDER-1233 Lost nodes should not
contribute to container failures (fix a fun-test)
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 <go...@apache.org>
Authored: Wed Oct 11 17:25:23 2017 -0700
Committer: Gour Saha <go...@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)