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

[wasm] Don't build samples in parallel on CI #90470

Merged
merged 1 commit into from
Aug 12, 2023

Conversation

radical
Copy link
Member

@radical radical commented Aug 12, 2023

The build fails when building samples. The wasm samples are built in parallel, and are all relinking (because publish'ed in Release). This is likely the container running out of resources. So, this PR is an attempt to not build these in parallel, and hopefully avoid hitting the limits.

Prompted by failures described in dotnet/dnceng#450

@radical radical added the arch-wasm WebAssembly architecture label Aug 12, 2023
@ghost ghost assigned radical Aug 12, 2023
@ghost
Copy link

ghost commented Aug 12, 2023

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

Issue Details

Prompted by failures described in dotnet/dnceng#450

Author: radical
Assignees: -
Labels:

arch-wasm

Milestone: -

@ghost
Copy link

ghost commented Aug 12, 2023

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

Issue Details

Prompted by failures described in dotnet/dnceng#450

Author: radical
Assignees: radical
Labels:

arch-wasm, area-Infrastructure-libraries, area-Infrastructure-mono

Milestone: -

@lewing lewing merged commit f515ab0 into dotnet:main Aug 12, 2023
7 of 13 checks passed
@radical radical requested review from lewing and removed request for brickgeorge August 12, 2023 23:26
@radical radical deleted the wasm-samples-serial branch August 12, 2023 23:26
@ghost ghost locked as resolved and limited conversation to collaborators Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants