Merge lp://qastaging/~robertcarr/mir/fix-1227683 into lp://qastaging/mir
Status: | Merged |
---|---|
Approved by: | Daniel van Vugt |
Approved revision: | no longer in the source branch. |
Merged at revision: | 1178 |
Proposed branch: | lp://qastaging/~robertcarr/mir/fix-1227683 |
Merge into: | lp://qastaging/mir |
Diff against target: |
11 lines (+1/-1) 1 file modified
tests/acceptance-tests/test_client_input.cpp (+1/-1) |
To merge this branch: | bzr merge lp://qastaging/~robertcarr/mir/fix-1227683 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Daniel van Vugt | Approve | ||
Kevin DuBois | Pending | ||
Review via email: mp+193168@code.qastaging.launchpad.net |
This proposal supersedes a proposal from 2013-10-28.
Commit message
test_client_
(LP: #1227683)
Description of the change
Started investigating bug 1227683, the hidden clients test failure.
I was able to reproduce it quite consistently today (about 3/10 valgrind runs ><). My system is quite slow today somehow, I guess exaggerating the effect.
Through use of SESSION_
Started investigating and found that the timeout fixed in this branch was still down at 2 seconds!
Bumped it to 60. Was able to run a 100 repetition run under valgrind with no failures.
PASSED: Continuous integration, rev:1171 jenkins. qa.ubuntu. com/job/ mir-ci/ 1609/ jenkins. qa.ubuntu. com/job/ mir-android- trusty- i386-build/ 31 jenkins. qa.ubuntu. com/job/ mir-clang- trusty- amd64-build/ 30 jenkins. qa.ubuntu. com/job/ mir-trusty- amd64-ci/ 7 jenkins. qa.ubuntu. com/job/ mir-trusty- amd64-ci/ 7/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ mir-trusty- armhf-ci/ 7 jenkins. qa.ubuntu. com/job/ mir-trusty- armhf-ci/ 7/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: 10.97.0. 26:8080/ job/mir- ci/1609/ rebuild
http://