Skip to content

Fix Out of memory with source gen tests#88266

Merged
tarekgh merged 2 commits into
dotnet:mainfrom
tarekgh:FixSourceGenTestAllocations
Jul 1, 2023
Merged

Fix Out of memory with source gen tests#88266
tarekgh merged 2 commits into
dotnet:mainfrom
tarekgh:FixSourceGenTestAllocations

Conversation

@tarekgh

@tarekgh tarekgh commented Jun 30, 2023

Copy link
Copy Markdown
Member

Fixes #87934

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 30, 2023
@ghost ghost assigned tarekgh Jun 30, 2023
@tarekgh

tarekgh commented Jun 30, 2023

Copy link
Copy Markdown
Member Author

/azp run runtime-coreclr libraries-jitstress

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@BruceForstall

Copy link
Copy Markdown
Contributor

Related: #87934

@tarekgh

tarekgh commented Jun 30, 2023

Copy link
Copy Markdown
Member Author

/azp run runtime-coreclr libraries-jitstress

@tarekgh
tarekgh marked this pull request as ready for review June 30, 2023 22:40
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@tarekgh

tarekgh commented Jun 30, 2023

Copy link
Copy Markdown
Member Author

@CyrusNajmabadi could you please have a look?

CC @jaredpar

@ghost

ghost commented Jun 30, 2023

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/area-extensions-logging
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #87934

Author: tarekgh
Assignees: tarekgh
Labels:

area-Extensions-Logging, needs-area-label

Milestone: -

@jkotas jkotas added source-generator Indicates an issue with a source generator feature and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jun 30, 2023
@tarekgh tarekgh added this to the 8.0.0 milestone Jun 30, 2023
@tarekgh

tarekgh commented Jul 1, 2023

Copy link
Copy Markdown
Member Author

The unrelated failure in the jitstress is tracked by #88224

@tarekgh
tarekgh merged commit 8b7b5ef into dotnet:main Jul 1, 2023
@tarekgh
tarekgh deleted the FixSourceGenTestAllocations branch July 1, 2023 00:52
string? extraFile = null)
{
Project proj = CreateTestProject(references);
using var workspace = CreateTestWorkspace();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For my education, OOM's were triggered by AdhocWorkspace instances not being disposed?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

that is right

@ghost ghost locked as resolved and limited conversation to collaborators Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Extensions-Logging source-generator Indicates an issue with a source generator feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test failure: Microsoft.Extensions.Logging.Generators.Tests.LoggerMessageGeneratorParserTests.NeedlessExceptionInMessage

5 participants