Closed Bug 1809105 Opened 2 years ago Closed 1 year ago

Increase TIMEOUT_MULTIPLIER for CDP browser chrome tests for ASAN, TSAN and CCOV builds as well

Categories

(Remote Protocol :: CDP, defect, P1)

defect
Points:
1

Tracking

(firefox111 fixed)

RESOLVED FIXED
111 Branch
Tracking Status
firefox111 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

(Depends on 1 open bug)

Details

(Whiteboard: [webdriver:m6])

Attachments

(1 file)

Right now we only have the multiplier of 4 active for debug builds, but we should have at least the same value for ASAN and TSAN builds given that those are slow as well.

Blocks: 1641872
Points: --- → 1
Priority: -- → P2
Whiteboard: [webdriver:backlog]

Actually also for CCOV jobs the timeout should be enhanced so that we cover all slowly running builds.

There are interesting results:
https://treeherder.mozilla.org/jobs?repo=try&revision=518affdcf6696701faa26ad716388a95729129af

While tsan seems to fail for browser_setCacheDisabled.js (bug 1775791) all the CCOV tests perma fail in browser_lifecycleEvent.js (bug 1734312). Most likely we will have to fix those dependencies first.

Depends on: 1775791, 1734312
Summary: Increase TIMEOUT_MULTIPLIER for CDP browser chrome tests for ASAN/TSAN builds as well → Increase TIMEOUT_MULTIPLIER for CDP browser chrome tests for ASAN, TSAN and CCOV builds as well
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Priority: P2 → P1
Whiteboard: [webdriver:backlog] → [webdriver:m6]
Depends on: 1810395

Ideally the Mochitest harness should provide a default multiplier per build type. I filed bug 1810395 for that. For now we should just go ahead and extend the timeout for the specific test as all the other tests are doing.

Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff5fb2cd7411
[cdp] Increase TIMEOUT_MULTIPLIER for browser chrome tests for debug, ASAN/TSAN, and CCOV builds. r=webdriver-reviewers,Sasha,jdescottes
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: