This is an archive of the discontinued LLVM Phabricator instance.

Bump the default timeout in dosep.py to 10m.
ClosedPublic

Authored by sivachandra on May 8 2015, 3:53 PM.

Details

Summary

This seems to be sufficient to get the tests taking longer than the
previous timeout of 5m to run to completion on Android to pass instead
of timing out.

Diff Detail

Event Timeline

sivachandra updated this revision to Diff 25378.May 8 2015, 3:53 PM
sivachandra retitled this revision from to Bump the default timeout in dosep.py to 10m..
sivachandra updated this object.
sivachandra edited the test plan for this revision. (Show Details)
sivachandra added a reviewer: chaoren.
sivachandra added a subscriber: Unknown Object (MLST).
chaoren added inline comments.May 8 2015, 3:55 PM
test/dosep.py
50–51

Please update comment too.

chaoren added inline comments.May 8 2015, 3:58 PM
test/dosep.py
6

Please update this.

8

Ditto.

50–51

Actually, this special case should probably just be removed.

156

And this.

158

Here too.

sivachandra updated this revision to Diff 25379.May 8 2015, 3:58 PM

Remove the special case for TestConcurrentEvents.

sivachandra updated this revision to Diff 25381.May 8 2015, 4:06 PM

Update comments.

chaoren accepted this revision.May 8 2015, 4:07 PM
chaoren edited edge metadata.
This revision is now accepted and ready to land.May 8 2015, 4:07 PM
sivachandra closed this revision.May 8 2015, 4:12 PM