Closed Bug 1298602 Opened 4 years ago Closed 1 year ago

Intermittent ExpirationTracker.main | Value of: timeDiffMS < periodMS && aObj->mExpired

Categories

(Core :: XPCOM, defect, P3)

All
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox51 --- wontfix
firefox-esr68 --- fixed
firefox68 --- wontfix
firefox69 --- fixed
firefox70 --- fixed

People

(Reporter: aryx, Assigned: froydnj)

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell unknown])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1112970 +++

https://treeherder.mozilla.org/logviewer.html#?job_id=2660507&repo=autoland

07:37:53     INFO -  TEST-START | ExpirationTracker.main
07:37:54  WARNING -  TEST-UNEXPECTED-FAIL | ExpirationTracker.main | Value of: timeDiffMS < periodMS && aObj->mExpired
07:37:54     INFO -    Actual: false
07:37:54     INFO -  Expected: true @ /home/worker/workspace/build/src/xpcom/tests/gtest/TestExpirationTracker.cpp:125
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3

There are 24 total failures in the last 7 days on:

*debug: macosx1014-64
*opt: macosx1014-64-shippable

Recent failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=257704379&repo=autoland&lineNumber=752

16:10:20 INFO - TEST-START | ExpirationTracker.main
16:10:52 WARNING - TEST-UNEXPECTED-FAIL | ExpirationTracker.main | Value of: timeDiffMS < periodMS && aObj->mExpired
16:10:52 INFO - Actual: false
16:10:52 INFO - Expected: true @ /builds/worker/workspace/build/src/xpcom/tests/gtest/TestExpirationTracker.cpp:135
16:10:52 WARNING - TEST-UNEXPECTED-FAIL | ExpirationTracker.main | Value of: timeDiffMS < periodMS && aObj->mExpired
16:10:52 INFO - Actual: false
16:10:52 INFO - Expected: true @ /builds/worker/workspace/build/src/xpcom/tests/gtest/TestExpirationTracker.cpp:135
16:10:58 WARNING - TEST-UNEXPECTED-FAIL | ExpirationTracker.main | test completed (time: 38199ms)

Nathan can you assign someone or could you take a look at this failure?

Flags: needinfo?(nfroyd)
Whiteboard: [stockwell needswork:owner]

This condition is intermittently failing, but it's hard to tell from the
failure which part of the condition is failing. Split the condition
apart so ideally we'll get some better error messages. While we're
doing that, use a more explicit EXPECT_LT for the first half of the
condition for even better error messages.

I wrote a patch to make the assertion messages a little more debuggable from logs; it'll change the intermittent failure to something else, but we can just dup bugs back to this one if necessary.

Flags: needinfo?(nfroyd)
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/925326628d36
split condition in TestExpirationTracker; r=KrisWright
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.