1445838 - Increase timeout for code coverage (ccov) tasks.

Status: RESOLVED FIXED

(Testing :: Code Coverage, enhancement)

Description

[Greg Mierzwinski [:sparky]]

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 1

[Greg Mierzwinski [:sparky]]

Attachment: Bug 1445838 - Increase task timeout on code coverage (ccov) test tasks.

Increases the max-run-time for all test tasks on ccov builds.

Review commit: https://reviewboard.mozilla.org/r/227944/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/227944/

Comment 2

[Marco Castelluccio [:marco]]

We probably want to only increase the timeout for the test suites that actually need it. Let's see what Joel thinks.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

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)

Comment 4

[Greg Mierzwinski [:sparky]]

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 5

[Greg Mierzwinski [:sparky]]

Comment on attachment 8959051 [details]
Bug 1445838 - Increase task timeout on code coverage (ccov) test tasks.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/227944/diff/1-2/

Comment 6

[Greg Mierzwinski [:sparky]]

Here is the try run for this change: https://treeherder.mozilla.org/#/jobs?repo=try&revision=77937065345df45568479f70d893c83416a66968

Comment 7

[Greg Mierzwinski [:sparky]]

Comment on attachment 8959051 [details]
Bug 1445838 - Increase task timeout on code coverage (ccov) test tasks.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/227944/diff/2-3/

Comment 8

[Greg Mierzwinski [:sparky]]

Treeherder link for the last review request: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f6735f196a5c785ba23f2eeb2bba8e3a5b27cdd1

Comment 9

[Marco Castelluccio [:marco]]

Comment on attachment 8959051 [details]
Bug 1445838 - Increase task timeout on code coverage (ccov) test tasks.

https://reviewboard.mozilla.org/r/227944/#review234106

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e11efca51a53
Increase task timeout on code coverage (ccov) test tasks. r=marco

Comment 11

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/e11efca51a53