Skip to content

Remove runtime Helix MSBuild parallelism override - #131831

Open
jkoritzinsky with Copilot wants to merge 3 commits into
mainfrom
copilot/remove-msbuild-parallelism
Open

Remove runtime Helix MSBuild parallelism override#131831
jkoritzinsky with Copilot wants to merge 3 commits into
mainfrom
copilot/remove-msbuild-parallelism

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

main PR N/A

Description

Runtime test Helix submission no longer needs to force high MSBuild parallelism now that the Helix Job Monitor is enabled. This PR removes the runtime send-to-Helix /maxcpucount:55 override and its stale rationale, allowing the shared default /maxcpucount behavior to apply.

  • Removed the conditional msbuildParallelism: '/maxcpucount:55' override from run-test-job.yml.
  • Kept the existing shared send-to-Helix template default unchanged.

Customer Impact

No product runtime impact. This simplifies CI infrastructure behavior for runtime test Helix submissions.

Regression

No. This intentionally changes CI submission parallelism after Helix Job Monitor enablement.

Testing

Not included in this description.

Risk

Low. The change is limited to runtime test pipeline YAML and falls back to the existing shared send-to-Helix default.

Package authoring no longer needed in .NET 9

IMPORTANT: Starting with .NET 9, you no longer need to edit a NuGet package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older versions.

Copilot AI review requested due to automatic review settings August 4, 2026 21:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
16 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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

Co-authored-by: jkoritzinsky <1571408+jkoritzinsky@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 4, 2026 21:28
Copilot AI changed the title [WIP] Remove msbuild parallelism after Helix Job Monitor enablement Remove runtime Helix MSBuild parallelism override Aug 4, 2026
Copilot AI requested a review from jkoritzinsky August 4, 2026 21:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Co-authored-by: jkoritzinsky <1571408+jkoritzinsky@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 4, 2026 21:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@jkoritzinsky
jkoritzinsky marked this pull request as ready for review August 4, 2026 21:54
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 5 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Remove msbuild parallelism after Helix Job Monitor enablement

3 participants