Skip to content

Reapply "devops(docker): split browser layers and use zstd compressio…#41232

Open
KRRT7 wants to merge 2 commits into
microsoft:mainfrom
KRRT7:retrying
Open

Reapply "devops(docker): split browser layers and use zstd compressio…#41232
KRRT7 wants to merge 2 commits into
microsoft:mainfrom
KRRT7:retrying

Conversation

@KRRT7

@KRRT7 KRRT7 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

…n for faster pulls (#40702)" (#41081)

This reverts commit 67930e8.

@KRRT7

KRRT7 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

@dgozman ^^

@dcrousso dcrousso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

it doesnt look like anything has changed since #40702 to fix the issues that resulted in it being rolled out in #41081

@KRRT7

KRRT7 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

@dcrousso I tagged them so that they'd trigger the GHAs, I have no idea what you guys saw, I've been using it on my linux / macOS machines with no issues, I'd need to trigger the issue for me to be able to debug

edit: sorry, could've been clearer about my intentions

@KRRT7

KRRT7 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

as per #41081 (comment) without the logs, best I can hope for is for it to get triggered in your CI for me to be able to analyze.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@KRRT7 KRRT7 requested a review from dcrousso June 10, 2026 22:10
@KRRT7

KRRT7 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

hmm, ok, I think I know what's up.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [firefox] › mcp/http.spec.ts:103 › http transport browser lifecycle (isolated) @mcp-ubuntu-latest-firefox

7278 passed, 1119 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

6 flaky ⚠️ [chromium-library] › library/popup.spec.ts:261 › should not throw when click closes popup `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:645 › screencast › should capture full viewport `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:434 › should produce network events, routing, and annotations for Service Worker (advanced) `@chromium-ubuntu-22.04-node22`
⚠️ [webkit-library] › library/trace-viewer.spec.ts:2405 › should preserve custom doctype `@webkit-ubuntu-22.04-node20`
⚠️ [webkit-page] › page/page-set-input-files.spec.ts:38 › should upload a folder `@webkit-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:816 › should update state on subsequent run `@windows-latest-node22`

39503 passed, 771 skipped


Merge workflow run.

@dgozman

dgozman commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

@KRRT7 Seems to be still failing.

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