Skip to content

test: reach the trace viewer over IPv4 loopback under WSL - #42733

Merged
Yury Semikhatsky (yury-s) merged 1 commit into
microsoft:mainfrom
csaavedra:fix-trace-viewer-wsl-ipv4-loopback
Sep 16, 2026
Merged

Yury Semikhatsky (yury-s) merged 1 commit into
microsoft:mainfrom
csaavedra:fix-trace-viewer-wsl-ipv4-loopback

Conversation

@csaavedra

Copy link
Copy Markdown
Contributor

Two tests load the trace viewer inside the browser under test rather than in the fixture's own Chromium page: "should notify parent when ready to receive messages" and "should render blob trace received from opener". On the webkit-wsl channel the browser runs in the WSL guest and reaches the host over mirrored networking, which mirrors the IPv4 loopback but not the host's ::1 -- and show-trace binds whatever localhost resolves to, which is ::1 on Windows. The viewer is unreachable from the guest, so both have timed out on every webkit-wsl run since #42451 added them.

Pin the viewer to 127.0.0.1 for that channel, as playwright-client.spec.ts already does for browser.bind().

Two tests load the trace viewer inside the browser under test rather than in
the fixture's own Chromium page: "should notify parent when ready to receive
messages" and "should render blob trace received from opener". On the
webkit-wsl channel the browser runs in the WSL guest and reaches the host over
mirrored networking, which mirrors the IPv4 loopback but not the host's ::1 --
and show-trace binds whatever localhost resolves to, which is ::1 on Windows.
The viewer is unreachable from the guest, so both have timed out on every
webkit-wsl run since microsoft#42451 added them.

Pin the viewer to 127.0.0.1 for that channel, as playwright-client.spec.ts
already does for browser.bind().
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

8564 passed, 1446 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

1 failed
❌ [firefox-library] › library/browsercontext-storage-state.spec.ts:453 › should work when service worker is intefering and the origin is not open @firefox-ubuntu-22.04-node20

4 flaky ⚠️ [chromium-library] › library/video.spec.ts:699 › screencast › should capture full viewport on hidpi `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:736 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node24`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`

51723 passed, 1247 skipped


Merge workflow run.

@yury-s Yury Semikhatsky (yury-s) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There were a couple related reports, e.g. #41180 and a fix for vscode extension microsoft/playwright-vscode#769. We can probably resolve it by tweaking node ip resolution to prefer ipv4. This is fine to fix the tests but we need a solution for our users too.

@csaavedra

Claudio Saavedra (csaavedra) commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

Please merge since I don't think I have permissions.

@yury-s
Yury Semikhatsky (yury-s) merged commit 6480b58 into microsoft:main Sep 16, 2026
44 of 45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants