Closed Bug 1798716 Opened 1 year ago Closed 1 year ago

Intermittent netwerk/test/browser/browser_103_redirect_from_server.js | single tracking bug

Categories

(Core :: Networking: HTTP, defect, P3)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1783686
Tracking Status
firefox-esr102 --- unaffected
firefox106 --- unaffected
firefox107 --- unaffected
firefox108 --- wontfix
firefox109 --- wontfix
firefox110 --- affected

People

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

References

(Blocks 1 open bug, Regression)

Details

(Keywords: intermittent-failure, intermittent-testcase, regression, Whiteboard: [necko-triaged])

Filed by: imoraru [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=395259526&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/WmEG89f6QNG9k7LGxBotBQ/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/WmEG89f6QNG9k7LGxBotBQ/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1


[task 2022-11-02T14:54:44.588Z] 14:54:44     INFO - TEST-PASS | netwerk/test/browser/browser_103_redirect_from_server.js | test_eh_before_redirect: unexpected amount of normal request made expected at most 1 ({"hinted":1,"normal":0}), got 0 ({"hinted":1,"normal":0}) - true == true - 
[task 2022-11-02T14:54:44.589Z] 14:54:44     INFO - Leaving test bound test_eh_before_redirect
[task 2022-11-02T14:54:44.590Z] 14:54:44     INFO - Entering test bound test_redirect_eh_different_tld
[task 2022-11-02T14:54:44.591Z] 14:54:44     INFO - Buffered messages finished
[task 2022-11-02T14:54:44.597Z] 14:54:44     INFO - TEST-UNEXPECTED-FAIL | netwerk/test/browser/browser_103_redirect_from_server.js | test_redirect_eh_before_different_tld: unexpected amount of hinted request made expected at most 0 ({"hinted":0,"normal":0}), got 1 ({"hinted":1,"normal":0}) - false == true - JS frame :: resource://testing-common/early_hint_preload_test_helper.jsm :: lax_request_count_checking :: line 26
[task 2022-11-02T14:54:44.598Z] 14:54:44     INFO - Stack trace:
[task 2022-11-02T14:54:44.599Z] 14:54:44     INFO - resource://testing-common/early_hint_preload_test_helper.jsm:lax_request_count_checking:26
[task 2022-11-02T14:54:44.600Z] 14:54:44     INFO - chrome://mochitests/content/browser/netwerk/test/browser/browser_103_redirect_from_server.js:test_hint_completion_on_redirect:40
[task 2022-11-02T14:54:44.601Z] 14:54:44     INFO - TEST-PASS | netwerk/test/browser/browser_103_redirect_from_server.js | test_redirect_eh_before_different_tld: unexpected amount of normal request made expected at most 0 ({"hinted":0,"normal":0}), got 0 ({"hinted":1,"normal":0}) - true == true - 
[task 2022-11-02T14:54:44.602Z] 14:54:44     INFO - Leaving test bound test_redirect_eh_different_tld

:mleclair, since you are the author of the regressor, bug 1783683, could you take a look?

For more information, please visit auto_nag documentation.

Flags: needinfo?(mleclair)

Set release status flags based on info from the regressing bug 1783683

Summary: Intermittent TV netwerk/test/browser/browser_103_redirect_from_server.js | single tracking bug → Intermittent netwerk/test/browser/browser_103_redirect_from_server.js | single tracking bug

Set release status flags based on info from the regressing bug 1783683

Hi Mark, I hope it's OK if we assign this to you.

Assignee: nobody → mleclair
Blocks: earlyhints
Status: NEW → ASSIGNED
Priority: P5 → P3
Whiteboard: [necko-triaged]

(In reply to Valentin Gosu [:valentin] (he/him) from comment #7)

Hi Mark, I hope it's OK if we assign this to you.

Yes, that's fine. I talked with Manuel, and he mentioned two cases:

"1. The test case doesn't use the preloaded object (doesn't display the image) and therefore doesn't wait for the request to complete -> it might go into the next test case.
2. We cancel the early hint request when we get the redirect, but it could already be sent to the server (inherently racy, we can't fix that, we can only allow it to happen)"

It seems that he's also working on a few extra redirect test cases + a cancel notification to stop the EH on redirects. So we should maybe disable this for now?

I'm not sure how to do that though.

Flags: needinfo?(mleclair)
Blocks: 1802501
No longer blocks: earlyhints

This will be fixed with Bug 1783686.

Assignee: mleclair → manuel

Fixed by Bug 1783686

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Duplicate of bug: 1783686
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.