Closed Bug 1391532 Opened 8 years ago Closed 8 years ago

Intermittent /service-workers/service-worker/registration-updateviacache.https.html | register-with-updateViaCache-all-then-none - assert_not_equals: Imported script should have updated got disallowed value 1503029624.95

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

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

References

(Blocks 1 open bug)

Details

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

Attachments

(1 file)

HoPang, are these related to bug 1353636?
Flags: needinfo?(bhsu)
I believe it has something to do with that :( Let me investigate this. At this moment, I am very curious about the 15 failures last week. They all happened on osx, does that lead to anything helpful to solve this bug?
Assignee: nobody → bhsu
Flags: needinfo?(bhsu)
The mac-specific failures are probably just a timing issue. Do you have rr setup locally at all? Maybe this would reproduce under rr chaos mode.
Thanks, let me find a mac machine and setup an rr on it.
Oh, I just find out that we don't have rr for osx now, I am now falling back to my Linux...
Yea, I meant maybe rr chaos on Linux would trigger similar race condition we see on Mac. Sorry for my confusing comment.
this has 55 failures since August 18th: https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1391532&startday=2017-08-17&endday=2017-08-24&tree=all I ran across this and was excited to see work was being done on this already! Thanks for jumping on it.
Whiteboard: [stockwell needswork]
Hi Ben, and sorry again for not being able to reproduce this. After thinking this case over, I humbly think the root cause is the precision of the timestamp is too low which is 10^-2 sec in the testcase. By applying this patch, we can make the precision reach to 10^-6 sec. Since it does no harm to anything, I suggest we land this patch and checko how it would help.
Attachment #8902632 - Flags: review?(bkelly)
Attachment #8902632 - Flags: review?(bkelly) → review+
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/6c8a5a8f95aa Preserve the precision of a timestamp when converting it to string. r=bkelly
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Whiteboard: [stockwell needswork:owner] → [stockwell fixed:logic]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: