Intermittent toolkit/components/downloads/test/unit/test_DownloadLegacy.js | single tracking bug
Categories
(Toolkit :: Downloads API, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox111 | --- | fixed |
People
(Reporter: jmaher, Assigned: mak)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure, intermittent-testcase, Whiteboard: [stockwell unknown])
Attachments
(2 files)
Updated•3 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 42•2 years ago
|
||
Update:
There have been 34 failures within the last 7 days:
- 4 failures on Linux 18.04 x64 WebRender asan opt
- 8 failures on Linux 18.04 x64 WebRender opt
- 15 failures on Linux 18.04 x64 WebRender Shippable opt
- 1 failure on Windows 10 x86 2004 WebRender debug
- 5 failures on Windows 10 x64 2004 WebRender debug/opt
- 1 failure on Windows 10 x64 2004 WebRender Shippable opt
Recent failure log: https://treeherder.mozilla.org/logviewer?job_id=402331532&repo=autoland&lineNumber=7276
[task 2023-01-13T15:14:38.552Z] 15:14:38 INFO - TEST-START | toolkit/components/downloads/test/unit/test_DownloadLegacy.js
[task 2023-01-13T15:14:39.001Z] 15:14:39 WARNING - TEST-UNEXPECTED-FAIL | toolkit/components/downloads/test/unit/test_DownloadLegacy.js | xpcshell return code: 0
[task 2023-01-13T15:14:39.003Z] 15:14:39 INFO - TEST-INFO took 448ms
[task 2023-01-13T15:14:39.004Z] 15:14:39 INFO - >>>>>>>
[task 2023-01-13T15:14:39.005Z] 15:14:39 INFO - (xpcshell/head.js) | test MAIN run_test pending (1)
[task 2023-01-13T15:14:39.005Z] 15:14:39 INFO - (xpcshell/head.js) | test run_next_test 0 pending (2)
[task 2023-01-13T15:14:39.005Z] 15:14:39 INFO - (xpcshell/head.js) | test MAIN run_test finished (2)
[task 2023-01-13T15:14:39.006Z] 15:14:39 INFO - running event loop
[task 2023-01-13T15:14:39.006Z] 15:14:39 INFO - toolkit/components/downloads/test/unit/test_DownloadLegacy.js | Starting setup test_common_initialize
[task 2023-01-13T15:14:39.008Z] 15:14:39 INFO - (xpcshell/head.js) | test test_common_initialize pending (2)
[task 2023-01-13T15:14:39.009Z] 15:14:39 INFO - (xpcshell/head.js) | test run_next_test 0 finished (2)
[task 2023-01-13T15:14:39.009Z] 15:14:39 INFO - (xpcshell/head.js) | test run_next_test 1 pending (2)
[task 2023-01-13T15:14:39.009Z] 15:14:39 INFO - (xpcshell/head.js) | test test_common_initialize finished (2)
[task 2023-01-13T15:14:39.009Z] 15:14:39 INFO - "CONSOLE_MESSAGE: (info) No chrome package registered for chrome://branding/locale/brand.properties"
[task 2023-01-13T15:14:39.010Z] 15:14:39 INFO - toolkit/components/downloads/test/unit/test_DownloadLegacy.js | Starting test_basic
[task 2023-01-13T15:14:39.010Z] 15:14:39 INFO - (xpcshell/head.js) | test test_basic pending (2)
[task 2023-01-13T15:14:39.011Z] 15:14:39 INFO - TEST-PASS | toolkit/components/downloads/test/unit/test_DownloadLegacy.js | test_basic - [test_basic : 45] Sanity check the temporary file doesn't exist. - true == true
Marco, any chance you could help assign this to someone?
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 44•2 years ago
|
||
the DownloadTarget .size property is zero, that means either it was not updated yet (though the code seems to properly await for refresh()) or something removed the file. I cannot reproduce locally, I'll push a test on Try checking the file with IOUtils and hopefully that may tell us something, otherwise I'll have to add a bunch of logging to check the calls to refresh.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 47•2 years ago
|
||
There have been 35 total failures in the last 7 days, recent failure log.
Affected platforms are:
- linux1804-64-asan-qr
- linux1804-64-qr
- linux1804-64-shippable-qr
- windows10-32-2004-qr
- windows10-64-2004-qr
Comment hidden (Intermittent Failures Robot) |
Comment 49•2 years ago
|
||
Marco, any updates on this? It's on the disable recommended list with 149 total failures in last 30 days.
Updated•2 years ago
|
Assignee | ||
Comment 50•2 years ago
|
||
Unfortunately I don't have news yet.
I may try to workaround the problem until we have a better understanding of the issue. It would be a pity to disable this test that is covering most of the user download path. Let me try and see if it helps at all.
Assignee | ||
Comment 51•2 years ago
|
||
The test is often seeing downloadTaget.size = 0, that means refresh() was either
not invoked or not properly awaited for. For now just add an artificial refresh
to avoid disabling the test.
A proper fix should be investigated in Bug 1814364
Updated•2 years ago
|
Assignee | ||
Comment 52•2 years ago
|
||
I'm setting leave-open because I noticed a much less frequent failure related to Places annotations in the logs, that's something to investigate yet, though it's not that common as the size one afaict.
Comment 53•2 years ago
|
||
Comment 54•2 years ago
|
||
bugherder |
Updated•2 years ago
|
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 56•2 years ago
|
||
The downloads code is not awaiting for history storing data, thus a previous
sub-test may still be in the process of storing its annotations.
Assignee | ||
Comment 57•2 years ago
•
|
||
I attached a second patch to cover the history annotations failure too, so we can ideally close this once it lands, since the .size problem will be covered by Bug 1814364.
Assignee | ||
Updated•2 years ago
|
Comment 58•2 years ago
|
||
Comment 59•2 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Description
•