Closed Bug 1759903 Opened 3 years ago Closed 3 years ago

Perma "domstreams" config dom/serviceworkers/test/test_serviceworker_interfaces.html | false: ReadableStreamDefaultController should be defined on self

Categories

(Core :: DOM: Streams, defect, P5)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1757240
Tracking Status
firefox-esr91 --- unaffected
firefox98 --- unaffected
firefox99 --- unaffected
firefox100 --- affected

People

(Reporter: intermittent-bug-filer, Unassigned)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

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


[task 2022-03-16T18:21:47.056Z] 18:21:47     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | true: NetworkInformation should  NOT be defined on self 
[task 2022-03-16T18:21:47.057Z] 18:21:47     INFO - Buffered messages finished
[task 2022-03-16T18:21:47.058Z] 18:21:47     INFO - TEST-UNEXPECTED-FAIL | dom/serviceworkers/test/test_serviceworker_interfaces.html | false: ReadableStreamDefaultController should  be defined on self 
[task 2022-03-16T18:21:47.058Z] 18:21:47     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:417:16
[task 2022-03-16T18:21:47.059Z] 18:21:47     INFO -     setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2022-03-16T18:21:47.059Z] 18:21:47     INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-03-16T18:21:47.060Z] 18:21:47     INFO - TEST-UNEXPECTED-FAIL | dom/serviceworkers/test/test_serviceworker_interfaces.html | false: ReadableStreamDefaultReader should  be defined on self 
[task 2022-03-16T18:21:47.061Z] 18:21:47     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:417:16
[task 2022-03-16T18:21:47.061Z] 18:21:47     INFO -     setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2022-03-16T18:21:47.062Z] 18:21:47     INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-03-16T18:21:47.063Z] 18:21:47     INFO - TEST-UNEXPECTED-FAIL | dom/serviceworkers/test/test_serviceworker_interfaces.html | 2 === 0: The following interface(s) are not enumerated: ReadableStreamDefaultController, ReadableStreamDefaultReader 
[task 2022-03-16T18:21:47.063Z] 18:21:47     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:417:16
[task 2022-03-16T18:21:47.063Z] 18:21:47     INFO -     setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2022-03-16T18:21:47.064Z] 18:21:47     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | true: Bogus pref annotation for Module 

Hello Matthew, can you please take a look at this?
This new fail might be related to this one too.

Flags: needinfo?(mgaudet)

Can I re-title this bug without messing up your system?

This bug is the result of JS Streams (which has different behaviour) running the same tests as DOM Streams.

JS Streams is our backup position should we discover a show-stopping bug potentially requiring that we not ship DOM streams to release. This is increasingly looking like an unlikely outcome.

I'd sort of like to mark this as a accepted perma-fail in the tier 2 job if that's possible. Otherwise, I'll have to figure out if we can make this test conditional on build configuration.

Flags: needinfo?(mgaudet) → needinfo?(ctuns)
Component: DOM: Service Workers → DOM: Streams

(In reply to Matthew Gaudet (he/him) [:mgaudet] from comment #2)

Can I re-title this bug without messing up your system?

This bug is the result of JS Streams (which has different behaviour) running the same tests as DOM Streams.

JS Streams is our backup position should we discover a show-stopping bug potentially requiring that we not ship DOM streams to release. This is increasingly looking like an unlikely outcome.

I'd sort of like to mark this as a accepted perma-fail in the tier 2 job if that's possible. Otherwise, I'll have to figure out if we can make this test conditional on build configuration.

Sure, no problem.

Flags: needinfo?(ctuns)

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

Has Regression Range: --- → yes

This should not fail permanently. It sounds like this config will continue to run in CI for some time. Could you DOM_STREAMS as property to AppConstants, add it as helper data to https://searchfox.org/mozilla-central/source/dom/serviceworkers/test/test_serviceworker_interfaces.html and set ReadableStreamDefaultController conditional in https://searchfox.org/mozilla-central/source/dom/serviceworkers/test/test_serviceworker_interfaces.js based on this?

Flags: needinfo?(mgaudet)
Summary: Perma [tier 2] dom/serviceworkers/test/test_serviceworker_interfaces.html | false: ReadableStreamDefaultController should be defined on self → Perma "domstreams" config dom/serviceworkers/test/test_serviceworker_interfaces.html | false: ReadableStreamDefaultController should be defined on self

I was thinking on this, and I think the actual route to be pursued right now is turning off these builds in Bug 1757240, which I will do shortly.

Flags: needinfo?(mgaudet)

There are also these failures
dom/tests/mochitest/general/test_interfaces.html | ReadableStreamDefaultController should be defined on 'window' scope - log here: https://treeherder.mozilla.org/logviewer?job_id=371493119&repo=mozilla-central

These should be fixed by bug Bug 1757240, which is on autoland.

This looks to be resolved, as expected, by Bug 1757240

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.