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

Fix the log contention in correctness legs #72263

Merged
merged 6 commits into from
Feb 26, 2024
Merged

Conversation

jaredpar
Copy link
Member

The PS1 scripts are not using ExitWithExitCode thus they are not properly killing the compiler server and msbuild hence the logs are unreadable.

The PS1 scripts are not using `ExitWithExitCode` thus they are not
properly killing the compiler server and msbuild hence the logs are
unreadable.
@jaredpar jaredpar requested a review from a team as a code owner February 24, 2024 15:35
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Feb 24, 2024
@jaredpar
Copy link
Member Author

@dotnet/roslyn-infrastructure this fixes the publish log contention issue

@jaredpar jaredpar merged commit 28580f7 into dotnet:main Feb 26, 2024
30 checks passed
@jaredpar jaredpar deleted the logs branch February 26, 2024 03:13
@jjonescz jjonescz added this to the 17.10 P2 milestone Feb 27, 2024
jaredpar added a commit to jaredpar/roslyn that referenced this pull request Mar 6, 2024
This fixes another log contention issue we were seeing in CI. This was
not caught in dotnet#72263 because the syntax generation only ran
conditionally hence masked this problem.
jaredpar added a commit that referenced this pull request Mar 7, 2024
This fixes another log contention issue we were seeing in CI. This was
not caught in #72263 because the syntax generation only ran
conditionally hence masked this problem.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants