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.

Sign up for GitHub

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

Jump to bottom

[wasm] Bump chrome for testing - linux: 123.0.6312.122, windows: 123.0.6312.122 #101017

Closed
wants to merge 5 commits into from
Closed

[wasm] Bump chrome for testing - linux: 123.0.6312.122, windows: 123.0.6312.122 #101017

wants to merge 5 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

No description provided.

@github-actions github-actions bot added arch-wasm WebAssembly architecture area-Infrastructure-mono labels Apr 14, 2024
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Tagging subscribers to this area: @directhex
See info in area-owners.md if you want to be subscribed.

This was referenced Apr 14, 2024
Closed
Closed
@pavelsavara pavelsavara self-assigned this Apr 14, 2024
@pavelsavara pavelsavara added the os-browser Browser variant of arch-wasm label Apr 15, 2024
@pavelsavara
Copy link
Member

pavelsavara commented Apr 15, 2024 •
edited

It looks like chrome crash problem is gone, will give it yet another run just to be sure

pavelsavara
pavelsavara previously approved these changes Apr 15, 2024
View reviewed changes
@build-analysis build-analysis bot mentioned this pull request Apr 16, 2024
Closed
@pavelsavara
Copy link
Member

It looks like chrome crash problem is gone, will give it yet another run just to be sure

Unfortunately not. See Log

[23:47:56] info: [STRT] System.Text.Json.Serialization.Tests.NodeInteropTestsDynamic.NodesAsPocoProperties
[23:47:56] info: System.Text.Json.Serialization.Tests.NodeInteropTestsDynamic 0 ms
[23:47:56] fail: 
[23:47:56] fail: Unhandled Exception:
[23:47:56] fail: OutOfMemoryException
[23:47:56] fail: 
[23:47:56] fail: Unhandled Exception:
[23:47:56] fail: OutOfMemoryException
[23:48:28] info: Unable to evaluate script: tab crashed
[23:48:28] info: Unable to evaluate script: tab crashed

@pavelsavara pavelsavara added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Apr 16, 2024
@pavelsavara pavelsavara self-requested a review April 16, 2024 10:21
@pavelsavara pavelsavara dismissed their stale review April 16, 2024 10:22

still crashes

@build-analysis build-analysis bot mentioned this pull request Apr 16, 2024
Closed
@pavelsavara pavelsavara mentioned this pull request Apr 21, 2024
Closed
@pavelsavara
Copy link
Member

Closing in favor of #101345

@jkotas jkotas deleted the update-chrome-version-8677099424 branch April 27, 2024 22:48
@maraf maraf mentioned this pull request Apr 29, 2024
Closed
@build-analysis build-analysis bot mentioned this pull request May 2, 2024
Open
@ilonatommy ilonatommy mentioned this pull request May 6, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilonatommy ilonatommy Awaiting requested review from ilonatommy

@lewing lewing Awaiting requested review from lewing

@maraf maraf Awaiting requested review from maraf

@radical radical Awaiting requested review from radical

@pavelsavara pavelsavara Awaiting requested review from pavelsavara

Assignees

@pavelsavara pavelsavara

Labels
arch-wasm WebAssembly architecture area-Infrastructure-mono NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) os-browser Browser variant of arch-wasm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pavelsavara @lewing