Closed
Bug 1445838
Opened 7 years ago
Closed 7 years ago
Increase timeout for code coverage (ccov) tasks.
Categories
(Testing :: Code Coverage, enhancement)
Testing
Code Coverage
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: sparky, Assigned: sparky)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
Currently, many of the tasks are failing because the timeout is too low: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=0d81c80876dd09536f78d1158e1e6ff78f9ad226&filter-searchStr=win%20ccov
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Blocks: code-coverage
Comment 2•7 years ago
|
||
We probably want to only increase the timeout for the test suites that actually need it. Let's see what Joel thinks.
Flags: needinfo?(jmaher)
Comment 3•7 years ago
|
||
ideally we will only do this on the failing tests like jittest, jstest, web-platform-tests. If a transform makes that too hard, I could get behind a longer timeout- although 10800 is 3 hours, and I would like to think we wouldn't have anything longer than 2 hours (wpt is hitting the 2 hour limit)
Flags: needinfo?(jmaher)
Assignee | ||
Comment 4•7 years ago
|
||
Ok, devtools also needs it's timeout increased. Wpt has to go above two hours because, as you say, it hit that limit.
I'll add them to the test task yamls.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•7 years ago
|
||
Here is the try run for this change: https://treeherder.mozilla.org/#/jobs?repo=try&revision=77937065345df45568479f70d893c83416a66968
Comment hidden (mozreview-request) |
Assignee | ||
Comment 8•7 years ago
|
||
Treeherder link for the last review request: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f6735f196a5c785ba23f2eeb2bba8e3a5b27cdd1
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8959051 [details]
Bug 1445838 - Increase task timeout on code coverage (ccov) test tasks.
https://reviewboard.mozilla.org/r/227944/#review234106
Attachment #8959051 -
Flags: review?(mcastelluccio) → review+
Comment 10•7 years ago
|
||
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e11efca51a53
Increase task timeout on code coverage (ccov) test tasks. r=marco
Comment 11•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•6 years ago
|
Assignee: nobody → gmierz2
You need to log in
before you can comment on or make changes to this bug.
Description
•