Closed
Bug 866293
Opened 11 years ago
Closed 11 years ago
Intermittent test_osfile_async.xul | system_shutdown: Log message observervation promise was rejected as expected | system_shutdown: Log messages observervation promise was rejected
Categories
(Toolkit Graveyard :: OS.File, defect)
Tracking
(firefox25 wontfix, firefox26 fixed, firefox-esr24 wontfix)
RESOLVED
FIXED
mozilla26
People
(Reporter: RyanVM, Assigned: yzen)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file, 1 obsolete file)
1.57 KB,
patch
|
Yoric
:
review+
|
Details | Diff | Splinter Review |
Regression from bug 862570. https://tbpl.mozilla.org/php/getParsedLog.php?id=22295695&tree=Mozilla-Inbound Rev3 WINNT 5.1 mozilla-inbound debug test mochitest-other on 2013-04-26 12:42:58 PDT for push 089bfd3486f5 slave: talos-r3-xp-060 13:01:00 INFO - 15338 INFO TEST-INFO | chrome://mochitests/content/chrome/toolkit/components/osfile/tests/mochi/test_osfile_async.xul | system_shutdown: waitObservation timeout exceeded. 13:01:00 INFO - 15339 INFO TEST-INFO | chrome://mochitests/content/chrome/toolkit/components/osfile/tests/mochi/test_osfile_async.xul | system_shutdown: waitObservation timeout exceeded. 13:01:00 INFO - OS Agent Received message {"fun":"SET_DEBUG","args":[false],"id":243} 13:01:00 INFO - OS Agent Calling method SET_DEBUG 13:01:00 INFO - 15340 INFO TEST-INFO | chrome://mochitests/content/chrome/toolkit/components/osfile/tests/mochi/test_osfile_async.xul | system_shutdown: Unregistered listener for resource chrome\toolkit\components\osfile\tests\mochi\main_test_osfile_async.js 13:01:00 INFO - 15341 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/components/osfile/tests/mochi/test_osfile_async.xul | system_shutdown: Log message observervation promise was rejected as expected. 13:01:00 INFO - OS Agent Method SET_DEBUG succeeded 13:01:00 INFO - OS Agent Sending positive reply undefined id is 246
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Comment 2•11 years ago
|
||
BTW, nice typo in that error message. Might want to fix that up while you're fixing this failures :)
Summary: Intermittent test_osfile_async.xul | system_shutdown: Log message observervation promise was rejected as expected. → Intermittent test_osfile_async.xul | system_shutdown: Log message observervation promise was rejected as expected | system_shutdown: Log messages observervation promise was rejected
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → yura.zenevich
Assignee | ||
Comment 3•11 years ago
|
||
It looks like the timeout might be a little too small. I could not reproduce this issue on my machine. Do you think we should increase it slightly? Perhaps half a second, if that's not too much? Thanks.
Flags: needinfo?(dteller)
Comment 4•11 years ago
|
||
It's worth trying. So far, the two failures are under Windows, so you might wish to: - increase timeout; - measure and log how much time it actually takes to receive the message, for future reference; - try this through the Windows TryServer.
Flags: needinfo?(dteller)
Assignee | ||
Comment 5•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=9f35d83cb1bf
Attachment #743156 -
Flags: review?(dteller)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 7•11 years ago
|
||
The try does not seem to have the issue. Let me know if there's anything else I should add/change?
Flags: needinfo?(dteller)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 9•11 years ago
|
||
Comment on attachment 743156 [details] [diff] [review] Patch for 866293. Review of attachment 743156 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me. If the issue arises again, we should have some information from the additional logs (using Try Runs that succeed, that is) that would help us adjust the timeout. An alternative would be to push the timeout to something extreme, say, 20 seconds, but that would probably be overkill.
Attachment #743156 -
Flags: review?(dteller) → review+
Flags: needinfo?(dteller)
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Reporter | ||
Comment 10•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d2348909462c
Keywords: checkin-needed
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Comment 13•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d2348909462c
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Comment 17•11 years ago
|
||
Not as frequent as it used to be, but still happening...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 19•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=69b7542be19d
Attachment #754542 -
Flags: review?(dteller)
Updated•11 years ago
|
Attachment #754542 -
Flags: review?(dteller) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #743156 -
Attachment is obsolete: true
Assignee | ||
Updated•11 years ago
|
Keywords: intermittent-failure → checkin-needed
Reporter | ||
Updated•11 years ago
|
Keywords: intermittent-failure
Assignee | ||
Comment 20•11 years ago
|
||
Apologies for removing the flag by accident.
Reporter | ||
Comment 21•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ffea3a88169e
Keywords: checkin-needed
Reporter | ||
Comment 22•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ffea3a88169e
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Target Milestone: mozilla23 → mozilla24
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
yzen, could you take a look at this?
Flags: needinfo?(yura.zenevich)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 39•11 years ago
|
||
As per Yoric's message: http://krijnhoetmer.nl/irc-logs/developers/20130910#l-3281
Flags: needinfo?(yura.zenevich)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
This should have be fixed by bug 913899. Please reopen if necessary.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 44•11 years ago
|
||
Possible to uplift to 25?
status-firefox25:
--- → affected
status-firefox26:
--- → fixed
status-firefox-esr24:
--- → affected
Target Milestone: mozilla24 → mozilla26
The patch that fixes the issue as a side-effect is quite experimental and changes part of the shutdown of Firefox. I would rather avoid uplifting it at least until we are reasonably certain that we have not increased the number of shutdown freezes.
Reporter | ||
Comment 46•11 years ago
|
||
OK, seems very unlikely that it'll get beta approval then. If I hit this on beta/esr24, I will just disable the test.
Anyway, the test only ensures that some warning is displayed as expected. This is useful only for Firefox developers and add-on developers.
Updated•11 months ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•