Closed Bug 661784 Opened 13 years ago Closed 12 years ago

Intermittent timeout test_0083_error_patchApplyFailure_partial_complete.xul or test_0084_error_patchApplyFailure_verify_failed.xul | Test timed out. Maximum time allowed is 25 seconds

Categories

(Toolkit :: Application Update, defect)

Product:
Toolkit
Component:
Application Update
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla7

People

(Reporter: masayuki, Assigned: robert.strong.bugs)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 2 obsolete files)

patch - fix indentation
(deleted), patch
ehsan.akhgari
: review+
Details | Diff | Splinter Review
might be reated to bug 645607. http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1307091827.1307093164.25014.gz#err0 s: talos-r3-fed64-016 13314 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/mozapps/update/test/chrome/test_0083_error_patchApplyFailure_partial_complete.xul | Test timed out. Maximum time allowed is 25 seconds 13317 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/mozapps/update/test/chrome/test_0083_error_patchApplyFailure_partial_complete.xul | Checking if TESTS[2] test was performed... callback function name = defaultCallback, pageid = finished 13318 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/mozapps/update/test/chrome/test_0083_error_patchApplyFailure_partial_complete.xul | Found an existing Update Window from the current or a previous test... attempting to close it. PROCESS-CRASH | Main app process exited normally | application crashed (minidump found) Thread 0 (crashed)
OS: Windows 7 → Linux
Hardware: x86 → x86_64
Component: General → Application Update
QA Contact: general → application.update
Blocks: 438871
Attached patch patch rev1 (obsolete) (deleted) — — Splinter Review
This test never failed prior to changing the slow download interval from 1000 ms to 100 ms so I'm changing it back.
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #537879 - Flags: review?(dtownsend)
Attachment #537879 - Flags: review?(dtownsend) → review+
Robert, can this land?
Yes, I plan on landing it today with a couple of other patches
Pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/399c2fef0673 Leaving open until this is verified to have fixed this orange
This also happened on test_0084_error_patchApplyFailure_verify_failed.xul, etc in http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1307656910.1307658704.14651.gz (on a revision before comment 8)
Actually, what we usually do is close the orange bugs and later reopen them if they occur again.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
(In reply to comment #9) > This also happened on test_0084_error_patchApplyFailure_verify_failed.xul, > etc in > http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1307656910.1307658704. > 14651.gz (on a revision before comment 8) Makes sense. Both tests require the download for the mar file to take longer than it takes to normally download the test mar file.
Summary: Intermittent timeout /test_0083_error_patchApplyFailure_partial_complete.xul | Test timed out. Maximum time allowed is 25 seconds → Intermittent timeout test_0083_error_patchApplyFailure_partial_complete.xul or test_0084_error_patchApplyFailure_verify_failed.xul | Test timed out. Maximum time allowed is 25 seconds
Still failing... will investigate further
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Backed out the patch since it didn't help and might have made things worse http://hg.mozilla.org/mozilla-central/rev/9b43783ab8ec
Well, this at least pointed out that something strange is going on in that the patch caused this to happen much more often when it shouldn't have. If anything, it should have had the opposite affect.
Attached patch patch (obsolete) (deleted) — — Splinter Review
The reference to the timer appears to be getting lost... I verified this fixes this orange on Mac OS X.
Attachment #537879 - Attachment is obsolete: true
Attachment #538645 - Flags: review?(dtownsend)
Attached patch patch - fix indentation (deleted) — — Splinter Review
Attachment #538645 - Attachment is obsolete: true
Attachment #538645 - Flags: review?(dtownsend)
Attachment #538646 - Flags: review?(dtownsend)
Oh, of course! Another version of these issues.
Blocks: nsITimer-fail
Attachment #538646 - Flags: review?(dtownsend) → review+
http://hg.mozilla.org/mozilla-central/rev/a7e6be787ed5 Thanks Robert for tracking this down!
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Not sure why this bug was reopened and I'm quite certain this specific bug was caused by the timer scope and that it is fixed. Please file a new bug if this test is failing.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 12 years ago
Resolution: --- → FIXED
(In reply to Robert Strong [:rstrong] (do not email) from comment #44) > Not sure why this bug was reopened and I'm quite certain this specific bug > was caused by the timer scope and that it is fixed. Please file a new bug if > this test is failing. Because seemingly the same failure occurred again (see comment 43 and compare to the strings in previous comments).
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: