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)