Skip to content

Fix daily CI OOM: build TFMs sequentially#1127

Open
ramsessanchez wants to merge 3 commits into
mainfrom
fix/ci-oom-large-agent
Open

Fix daily CI OOM: build TFMs sequentially#1127
ramsessanchez wants to merge 3 commits into
mainfrom
fix/ci-oom-large-agent

Conversation

@ramsessanchez

@ramsessanchez ramsessanchez commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

The daily CI build was failing due to out-of-memory (95%+ memory usage) causing the build to be canceled after ~54 minutes.

Changes

  • Build each TFM sequentially in the daily CI pipeline to avoid OOM kills, matching the approach already used in pipelines/ci-build.yml
  • Build and test the test project explicitly by path instead of solution-wide --no-build
  • Remove unnecessary System.Net.Http 4.3.4 PackageReference that was triggering NU1510 warnings on .NET 10+ SDK

Root Cause

The daily CI was running dotnet build Microsoft.Graph.Beta.sln --no-restore which builds all 4 TFMs in parallel, exhausting memory on the standard hosted agent. The official CI pipeline (pipelines/ci-build.yml) already solved this by building each TFM one at a time.

Microsoft Reviewers: Open in CodeFlow

ramsessanchez and others added 3 commits June 17, 2026 16:40
…em.Net.Http ref

- Build each target framework sequentially in daily CI to avoid OOM kills,
  matching the approach used in pipelines/ci-build.yml
- Revert agent pool back to windows-latest (sequential builds fit in memory)
- Build and test the test project explicitly by path
- Remove unnecessary System.Net.Http 4.3.4 PackageReference (NU1510 warning)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sequential TFM builds alone are insufficient — even a single TFM compilation
exhausts memory on the standard windows-latest agent (95%+ usage, canceled
after 30 min). The larger agent provides enough RAM for this SDK.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use the same compute pool as pipelines/ci-build.yml, which has sufficient
memory for compiling this large SDK.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ramsessanchez ramsessanchez marked this pull request as ready for review June 18, 2026 23:49
@ramsessanchez ramsessanchez requested a review from a team as a code owner June 18, 2026 23:49
Comment thread src/Microsoft.Graph/Microsoft.Graph.Beta.csproj
Comment thread .azure-pipelines/daily-ci-build.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants