Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update infrastructure metadata for webtransport-h3.https.sub.any.js #39345

Merged

Conversation

jonathan-j-lee
Copy link
Contributor

These tests are timing out on Firefox macOS and blocking unrelated PRs (e.g., #39196, #39201, #39227):

/infrastructure/server/webtransport-h3.https.sub.any.serviceworker.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.serviceworker.html
/infrastructure/server/webtransport-h3.https.sub.any.worker.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.worker.html
/infrastructure/server/webtransport-h3.https.sub.any.sharedworker.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.sharedworker.html
/infrastructure/server/webtransport-h3.https.sub.any.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.html

Disable them for now. Also, remove the .any.window.html section, which is redundant with .any.html.

These tests are timing out on Firefox macOS and blocking unrelated PRs
(e.g., web-platform-tests#39196, web-platform-tests#39201, web-platform-tests#39227):

```
/infrastructure/server/webtransport-h3.https.sub.any.serviceworker.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.serviceworker.html
/infrastructure/server/webtransport-h3.https.sub.any.worker.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.worker.html
/infrastructure/server/webtransport-h3.https.sub.any.sharedworker.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.sharedworker.html
/infrastructure/server/webtransport-h3.https.sub.any.html
  TIMEOUT WebTransport server should be running and should handle a bidirectional stream - Test timed out
  TIMEOUT /infrastructure/server/webtransport-h3.https.sub.any.html
```

Disable them for now. Also, remove the `.any.window.html` section, which
is redundant with `.any.html`.
@jonathan-j-lee jonathan-j-lee marked this pull request as ready for review April 3, 2023 22:50
@wpt-pr-bot wpt-pr-bot requested a review from jgraham April 3, 2023 22:50
@jonathan-j-lee jonathan-j-lee changed the title Update infrastructure metadata Update infrastructure metadata for webtransport-h3.https.sub.any.js Apr 3, 2023
@gsnedders
Copy link
Member

This is (another) fix for #39309.

@gsnedders
Copy link
Member

(Closing this in favour of earlier, near-identical PRs)

@gsnedders gsnedders closed this Apr 4, 2023
@gsnedders gsnedders reopened this Apr 4, 2023
@gsnedders
Copy link
Member

Actually, the others tried to get this working by getting Web Transport working on Firefox; let's land this because it just updates the expectations to what Nightly now has.

@bashi
Copy link
Member

bashi commented Apr 4, 2023

+1 to land this first. It will unblock my pending CL (#39201)

@gsnedders gsnedders merged commit b8e8fc9 into web-platform-tests:master Apr 4, 2023
@jonathan-j-lee jonathan-j-lee deleted the firefox-h3-timeout branch April 4, 2023 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants