Closed Bug 1739197 Opened 3 years ago Closed 2 years ago

Intermittent PacerTestIntLongDuplication.Past | Expected equality of these values:

Categories

(Core :: WebRTC: Audio/Video, defect, P2)

defect

Tracking

()

RESOLVED FIXED
99 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox93 --- unaffected
firefox94 --- unaffected
firefox95 --- unaffected
firefox96 --- wontfix
firefox97 --- wontfix
firefox98 --- wontfix
firefox99 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: nika)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(1 file)

Filed by: abutkovits [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=356892494&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Xmj6A5laSGCD4-2Qu0R4ow/runs/0/artifacts/public/logs/live_backing.log


[task 2021-11-03T16:32:33.831Z] 16:32:33     INFO -  TEST-START | PacerTestIntLongDuplication.Past
[task 2021-11-03T16:32:43.732Z] 16:32:43  WARNING -  TEST-UNEXPECTED-FAIL | PacerTestIntLongDuplication.Past | Expected equality of these values:
[task 2021-11-03T16:32:43.732Z] 16:32:43     INFO -    time - now
[task 2021-11-03T16:32:43.733Z] 16:32:43     INFO -      Which is: 9900 ms
[task 2021-11-03T16:32:43.733Z] 16:32:43     INFO -    -d1
[task 2021-11-03T16:32:43.733Z] 16:32:43     INFO -      Which is: -100 ms @ /builds/worker/checkouts/gecko/dom/media/gtest/TestPacer.cpp:72
[task 2021-11-03T16:32:43.733Z] 16:32:43  WARNING -  TEST-UNEXPECTED-FAIL | PacerTestIntLongDuplication.Past | test completed (time: 9902ms)
Has Regression Range: --- → yes
Assignee: nobody → apehrson

My changes in bug 1751948 apparently caused this test to start permafailing, so I looked into the issue. I think I have a fix, so stealing the bug.

Assignee: apehrson → nika
Blocks: 1751948
Priority: P5 → P2

Without this change, it was possible for the event dispatched in the
"Past" test case to be fully dispatched and notified on the background
thread pool before the call to WaitFor() connected a listener for the
event. This would lead to a test failure due to the observed
notification being the duplicated event fired 10 seconds after the
previous event was supposed to dispatch.

Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/559391cb798c
Ensure listeners are connected before enqueueing events in TestPacer, r=bwc
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: