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

Disable Linux arm64 llvm full aot runtime tests #60576

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

steveisok
Copy link
Member

Addresses #60234

Suspicion is that we're hitting some kind of OOM error within docker. Unblocks CI while we investigate.

Addresses dotnet#60234

Suspicion is that we're hitting some kind of OOM error within docker.  Unblocks CI while we investigate.
Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

Thanks!

@hoyosjs
Copy link
Member

hoyosjs commented Oct 19, 2021

@steveisok @lewing This still doesn't address the WASM part which is ~1/3 of the hits. Given that one is not so much test but a build leg, should this be prioritized? Or should we merge this to improve the hit count.

@akoeplinger
Copy link
Member

I think we should merge this given it fixes at least some builds.

@akoeplinger akoeplinger merged commit f4ba253 into dotnet:main Oct 19, 2021
steveisok pushed a commit to steveisok/runtime that referenced this pull request Oct 21, 2021
Missed excluding this configuration in dotnet#60576
akoeplinger pushed a commit that referenced this pull request Oct 22, 2021
Missed excluding this configuration in #60576

Co-authored-by: Steve Pfister <[email protected]>
@ghost ghost locked as resolved and limited conversation to collaborators Nov 18, 2021
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.

4 participants