Skip to content

[browser] default WasmDebugLevel to 0 for tests#104361

Merged
pavelsavara merged 5 commits intodotnet:mainfrom
pavelsavara:WasmDebugLevel_0
Jul 4, 2024
Merged

[browser] default WasmDebugLevel to 0 for tests#104361
pavelsavara merged 5 commits intodotnet:mainfrom
pavelsavara:WasmDebugLevel_0

Conversation

@pavelsavara
Copy link
Member

@pavelsavara pavelsavara commented Jul 3, 2024 •
edited
Loading

  • default WasmDebugLevel to 0 for tests on CI
  • print size of linear memory at the end of each test
  • reduce timeouts

Fixes #103829
Fixes #103246

@pavelsavara pavelsavara added arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript os-browser Browser variant of arch-wasm labels Jul 3, 2024
@pavelsavara pavelsavara added this to the 9.0.0 milestone Jul 3, 2024
@pavelsavara pavelsavara self-assigned this Jul 3, 2024
@pavelsavara pavelsavara requested a review from lewing as a code owner July 3, 2024 12:28
@dotnet-policy-service
Copy link
Contributor

dotnet-policy-service bot commented Jul 3, 2024

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

@pavelsavara pavelsavara requested review from ilonatommy and maraf July 3, 2024 12:29
@pavelsavara pavelsavara requested a review from akoeplinger as a code owner July 3, 2024 12:41
ilonatommy
ilonatommy reviewed Jul 3, 2024
View reviewed changes
akoeplinger
akoeplinger reviewed Jul 3, 2024
View reviewed changes
@pavelsavara
Copy link
Member Author

pavelsavara commented Jul 3, 2024

/azp run runtime-wasm

@azure-pipelines
Copy link

azure-pipelines bot commented Jul 3, 2024

Azure Pipelines successfully started running 1 pipeline(s).

@pavelsavara pavelsavara changed the title [browser] default WasmDebugLevel to 0 for tests on CI [browser] default WasmDebugLevel to 0 for tests Jul 3, 2024
This was referenced Jul 3, 2024
Closed
Open
Open
Closed
Closed
@pavelsavara
Copy link
Member Author

pavelsavara commented Jul 4, 2024

/ba-g filled #104419 and #104418

@pavelsavara pavelsavara merged commit dd6840d into dotnet:main Jul 4, 2024
@pavelsavara pavelsavara deleted the WasmDebugLevel_0 branch July 4, 2024 08:55
@build-analysis build-analysis bot mentioned this pull request Jul 4, 2024
Closed
This was referenced Jul 4, 2024
Merged
Closed
@build-analysis build-analysis bot mentioned this pull request Jul 4, 2024
Closed
maraf added a commit to maraf/runtime that referenced this pull request Jul 12, 2024
@maraf
[browser] default WasmDebugLevel to 0 for tests (dotnet#104361)
f18a052
@maraf maraf mentioned this pull request Jul 15, 2024
Merged
@github-actions github-actions bot locked and limited conversation to collaborators Aug 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@akoeplinger akoeplinger akoeplinger left review comments

@ilonatommy ilonatommy ilonatommy approved these changes

@lewing lewing Awaiting requested review from lewing lewing is a code owner

@maraf maraf Awaiting requested review from maraf

Assignees

@pavelsavara pavelsavara

Labels

arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript os-browser Browser variant of arch-wasm

Projects

None yet

Milestone

9.0.0

Development

Successfully merging this pull request may close these issues.

[browser] System.Memory.Tests is taking too long on FF [browser][ST] Wasm tests timeout

3 participants

@pavelsavara @akoeplinger @ilonatommy