Closed Bug 1860848 Opened 8 months ago Closed 8 months ago

Turn webtransport/echo-large-bidirectional-streams.https.html into any.js

Categories

(Core :: Networking: HTTP, task, P2)

task

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: saschanaz, Assigned: saschanaz)

References

Details

(Whiteboard: [necko-triaged], [wptsync upstream])

Attachments

(1 file)

It doesn't have to be main-thread-only test, and having it as any.js helps me understand why it fails with my patch.

Severity: -- → N/A
Priority: -- → P2
Whiteboard: [necko-triaged]
Pushed by krosylight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cd1c053db69c
Turn echo-large-bidirectional-streams into any.js r=necko-reviewers,kershaw
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/42807 for changes under testing/web-platform/tests
Whiteboard: [necko-triaged] → [necko-triaged], [wptsync upstream]

Backed out for causing failures on echo-large-bidirectional-streams.https.any.html

[task 2023-10-27T14:43:51.981Z] 14:43:51     INFO - TEST-START | /webtransport/echo-large-bidirectional-streams.https.any.html
[task 2023-10-27T14:43:51.990Z] 14:43:51     INFO - Closing window f1975a1c-02ab-4f7a-b292-f62b67dde854
[task 2023-10-27T14:44:12.710Z] 14:44:12     INFO - PID 19320 | JavaScript error: , line 0: WebTransportError: WebTransport connection rejected
[task 2023-10-27T14:44:12.711Z] 14:44:12     INFO - PID 19320 | JavaScript error: , line 0: WebTransportError: WebTransport connection rejected
[task 2023-10-27T14:44:23.038Z] 14:44:23     INFO - 
[task 2023-10-27T14:44:23.038Z] 14:44:23     INFO - TEST-UNEXPECTED-TIMEOUT | /webtransport/echo-large-bidirectional-streams.https.any.html | Ensure large bidirectional streams does not cause race condition - Test timed out
[task 2023-10-27T14:44:23.039Z] 14:44:23     INFO - TEST-UNEXPECTED-TIMEOUT | /webtransport/echo-large-bidirectional-streams.https.any.html | expected OK
[task 2023-10-27T14:44:23.039Z] 14:44:23     INFO - TEST-INFO took 31060ms
Flags: needinfo?(krosylight)

I'm sure that's bug 1829481 which is an existing issue, can we land it if it's not permanent?

Flags: needinfo?(krosylight) → needinfo?(ncsoregi)

It will take about 40 minutes more to have some results, in the meantime the trees are closed due to some backlog that occurred.

If need be, we will reland it.

Flags: needinfo?(ncsoregi)

Hmm, it failed three times in serial, that's at least close to perma. 🤔 Looks like it's failing specifically on tsan opt jobs.

Randell, any idea why would that be?

Flags: needinfo?(rjesup)

For now I'll adjust the meta file to cover that, which already allows tsan failure: https://searchfox.org/mozilla-central/source/testing/web-platform/meta/webtransport/echo-large-bidirectional-streams.https.html.ini

But it's suspicious and shouldn't be silently ignored.

Pushed by krosylight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aff07dd12684
Turn echo-large-bidirectional-streams into any.js r=necko-reviewers,kershaw
Upstream PR merged by moz-wptsync-bot
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/42825 for changes under testing/web-platform/tests
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
Upstream PR merged by moz-wptsync-bot
See Also: → 1862184

Investigate TSAN failures

Flags: needinfo?(rjesup)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: