Closed Bug 1979540 Opened 6 months ago Closed 5 months ago

Perma [tier 2] testing/firefox-ui/tests/functional/safebrowsing/test_warning_pages.py TestSafeBrowsingWarningPages.test_warning_pages | marionette_driver.errors.TimeoutException: Timed out after 300.0 seconds

Categories

(Testing :: Firefox UI Tests, defect, P5)

defect

Tracking

(firefox-esr128 unaffected, firefox-esr140 unaffected, firefox141 unaffected, firefox142 unaffected, firefox143 fixed, firefox144 fixed)

RESOLVED FIXED
144 Branch
Tracking Status
firefox-esr128 --- unaffected
firefox-esr140 --- unaffected
firefox141 --- unaffected
firefox142 --- unaffected
firefox143 --- fixed
firefox144 --- fixed

People

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

References

(Regression)

Details

(Keywords: intermittent-failure, regression, Whiteboard: [webdriver:m17][webdriver:external])

Attachments

(4 files)

Filed by: abutkovits [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=519644121&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/G0GZ6rYoQNeB27oO-0qgFg/runs/0/artifacts/public/logs/live_backing.log


[task 2025-07-26T23:24:45.700+00:00] 23:24:45     INFO -  1753572285700	Marionette	DEBUG	3 -> [0,3066,"Marionette:SetContext",{"value":"content"}]
[task 2025-07-26T23:24:45.700+00:00] 23:24:45     INFO -  1753572285700	Marionette	DEBUG	3 <- [1,3066,null,{"value":null}]
[task 2025-07-26T23:24:45.798+00:00] 23:24:45     INFO - TEST-UNEXPECTED-ERROR | testing/firefox-ui/tests/functional/safebrowsing/test_warning_pages.py TestSafeBrowsingWarningPages.test_warning_pages | marionette_driver.errors.TimeoutException: Timed out after 300.0 seconds
[task 2025-07-26T23:24:45.798+00:00] 23:24:45     INFO - Traceback (most recent call last):
[task 2025-07-26T23:24:45.798+00:00] 23:24:45     INFO -   File "/task_175357140997779/build/venv/lib/python3.10/site-packages/marionette_harness/marionette_test/testcases.py", line 193, in run
[task 2025-07-26T23:24:45.798+00:00] 23:24:45     INFO -     testMethod()
[task 2025-07-26T23:24:45.798+00:00] 23:24:45     INFO -   File "/task_175357140997779/build/tests/firefox-ui/tests/testing/firefox-ui/tests/functional/safebrowsing/test_warning_pages.py", line 63, in test_warning_pages
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -     self.check_report_link(unsafe_page)
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -   File "/task_175357140997779/build/tests/firefox-ui/tests/testing/firefox-ui/tests/functional/safebrowsing/test_warning_pages.py", line 111, in check_report_link
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -     Wait(self.marionette, timeout=self.marionette.timeout.page_load).until(
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -   File "/task_175357140997779/build/venv/lib/python3.10/site-packages/marionette_driver/wait.py", line 152, in until
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -     raise errors.TimeoutException(
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO - TEST-INFO took 311647ms
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -  1753572285763	Marionette	DEBUG	3 -> [0,3067,"Marionette:GetContext",{}]
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -  1753572285764	Marionette	DEBUG	3 <- [1,3067,null,{"value":"content"}]
[task 2025-07-26T23:24:45.799+00:00] 23:24:45     INFO -  1753572285764	Marionette	DEBUG	3 -> [0,3068,"Marionette:SetContext",{"value":"chrome"}]

:smayya, since you are the author of the regressor, bug 1972350, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(smayya)

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

There are 518 failures in the last 7 days.

Whimboo, can you please check and assign this bug to someone since smayya haven't answered and it's failing pretty frequent?

Flags: needinfo?(hskupin)
Assignee: nobody → smolnar
Status: NEW → ASSIGNED

Until a fix is in place, I've made a disable patch

Assignee: smolnar → nobody
Status: ASSIGNED → NEW
Keywords: leave-open
Pushed by smolnar@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/ba0d099de158 https://hg.mozilla.org/integration/autoland/rev/48e1ad373139 Disable 'test_warning_pages.py' for frequent failures. r=webdriver-reviewers,Sasha DONTBUILD

Hey Sunil, I've taken another look to provide more information. So looks like we hit:

[Parent 93097, Socket Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x804B005D (NS_ERROR_LOCAL_NETWORK_ACCESS_DENIED): file /builds/worker/checkouts/gecko/netwerk/protocol/http/nsHttpConnectionMgr.cpp:1591

and navigation to "http://127.0.0.1:55946/support.html?topic=phishing-malware" times out.
That seems to be happening when we:

Looking at the bug 1972350, not sure if we should set now some extra permissions. Could you help us here?

Whiteboard: [stockwell disabled]
Pushed by csabou@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/fef66082b5c3 https://hg.mozilla.org/integration/autoland/rev/abb6213d5f8c Disable 'test_warning_pages.py' on linux 22.04 wayland for frequent failures. r=intermittent-reviewers,jmaher DONTBUILD
Flags: needinfo?(smayya)

Hey Sunil,
I've seen that you removed needinfo, did you not find anything?

Flags: needinfo?(smayer97)
Flags: needinfo?(smayer97) → needinfo?(smayya)

Hey Alexandra,

I've seen that you removed needinfo, did you not find anything?
Sorry for missing this, it was by accident.

We have introduced LNA feature recently and any navigation to localhost (127.0.0.1) would prompt user for permission.
My guess is that we are experiencing timeout because the request is waiting on the user prompt.
Can you please confirm this?

I did not reproduce the issue locally, please let me know if you want me to have a look at it further.

Flags: needinfo?(smayya) → needinfo?(aborovova)

I don't see the prompt :( The navigation is just pending. I've attached the screenshot from the test in case I'm missing something.
In case you want to run the test locally, not sure if you've seen that the test is disabled now. To enable it again, you need to remove this metadata: https://searchfox.org/mozilla-central/source/testing/firefox-ui/tests/functional/safebrowsing/manifest.toml#15-21.

Flags: needinfo?(aborovova) → needinfo?(smayya)
Flags: needinfo?(hskupin)
Keywords: test-disabled
Assignee: nobody → smayya
Flags: needinfo?(smayya)

Hi Sunil, I wanted to check if you have an update for us regarding a potential fix for this perma test failure. Thanks!

Flags: needinfo?(smayya)

Hey henrik, sorry for the delay, I have posted the patch for review.

Flags: needinfo?(smayya) → needinfo?(hskupin)
Attachment #9509015 - Attachment description: WIP: Bug 1979540 - grant local network access for test_warning_pages.py tests → Bug 1979540 - grant local network access for test_warning_pages.py tests. r=whimboo
Pushed by smayya@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/57af4aff54e3 https://hg.mozilla.org/integration/autoland/rev/3db0e0dc53a8 grant local network access for test_warning_pages.py tests. r=webdriver-reviewers,whimboo

Sunil, it would be good to get this fix uplifted to beta so that we can run the test there as well. Mind doing an uplift request? Thanks!

Status: NEW → RESOLVED
Closed: 5 months ago
Flags: needinfo?(hskupin) → needinfo?(smayya)
Resolution: --- → FIXED
Whiteboard: [stockwell disabled] → [webdriver:m17][webdriver:external]
Target Milestone: --- → 144 Branch

Comment on attachment 9504208 [details]
Bug 1979540 - Disable 'test_warning_pages.py' for frequent failures. r=#intermittent-reviewers

Beta/Release Uplift Approval Request

  • User impact if declined/Reason for urgency: UI safe browsing tests wont be run for beta..
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Changes are modifications to test
  • String changes made/needed:
  • Is Android affected?: No
Flags: needinfo?(smayya)
Attachment #9504208 - Flags: approval-mozilla-beta?

Sunil, as it looks like you requested the uplift on the wrong patch. Can you please double check? Thanks.

Flags: needinfo?(smayya)
Flags: needinfo?(smayya)
Attachment #9504208 - Flags: approval-mozilla-beta?

Comment on attachment 9509015 [details]
Bug 1979540 - grant local network access for test_warning_pages.py tests. r=whimboo

Beta/Release Uplift Approval Request

  • User impact if declined/Reason for urgency: UI tests for safebrowsing wont be run for beta
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This patch only modifies test files
  • String changes made/needed:
  • Is Android affected?: No
Attachment #9509015 - Flags: approval-mozilla-beta?

(In reply to Henrik Skupin [:whimboo][⌚️UTC+2] from comment #30)

Sunil, as it looks like you requested the uplift on the wrong patch. Can you please double check? Thanks.

My Bad. Thanks for flagging this.

Comment on attachment 9509015 [details]
Bug 1979540 - grant local network access for test_warning_pages.py tests. r=whimboo

Approved for 143.0b5.

Attachment #9509015 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: