GitHub user hsaputra opened a pull request:
https://github.com/apache/incubator-twill/pull/56
(TWILL-142) Add longer time for timeout in EchoServerTestRun to check
whether the restart successful.
Bump up the timout for stopwatch to wait until all containers have been
restarted.
With new feature in TWILL-116 being committed, the EchoServerTestRun test
become bit flaky due to
sometimes the mini cluster could not allocate new containers in current
proposed timeout which is 30s.
Also fix logging for the test to log info when you get different set of
containers instead of error.
Run test with Travis without the change 5 times we got 2 error because of
container id same in EchoServerTestRun restart test.
With the fix, run 12 times Travis build and 100% NO error in the
EchoServerTestRun restart test.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/hsaputra/incubator-twill
increase_wait_for_restart_instances
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-twill/pull/56.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #56
----
commit a584d1c9b0b7980451df11f8bbe851d047d6a4cb
Author: hsaputra <[email protected]>
Date: 2015-07-15T22:59:05Z
(TWILL-142) Add longer time for timeout in EchoServerTestRun to check
whether the restart successful.
Bump up the timout for stopwatch to wait until all containers have been
restarted.
With new feature in TWILL-116 being committed, the EchoServerTestRun test
become bit flaky due to
sometimes the mini cluster could not allocate new containers in current
proposed timeout which is 30s.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
