Skip to content

Fix daily CI OOM: build TFMs sequentially #1127

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.

Sign up for GitHub

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

Open
ramsessanchez wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix daily CI OOM: build TFMs sequentially #1127

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d2f07ff
Fix daily CI OOM: build TFMs sequentially and remove unnecessary Syst…
ramsessanchez Jun 17, 2026
f516dde
Switch daily CI to windows-2022-large agent
ramsessanchez Jun 18, 2026
39bc9a2
Switch daily CI to 1es-windows-ps-compute-m pool
ramsessanchez Jun 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 22 additions & 7 deletions .azure-pipelines/daily-ci-build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,7 @@ resources:
extends:
template: v1/1ES.Unofficial.PipelineTemplate.yml@1ESPipelineTemplates
parameters:
pool:
name: Azure-Pipelines-1ESPT-ExDShared
image: windows-latest
os: windows
pool: 1es-windows-ps-compute-m
Comment thread
ramsessanchez marked this conversation as resolved.
sdl:
sourceAnalysisPool:
name: Azure-Pipelines-1ESPT-ExDShared
Expand Down Expand Up @@ -74,10 +71,28 @@ extends:
displayName: Restore dependencies
workingDirectory: $(Build.SourcesDirectory)

- script: dotnet build Microsoft.Graph.Beta.sln --no-restore
displayName: Build SDK
# Build each TFM sequentially to avoid OOM kills on the build agent.
# Building all TFMs in parallel exhausts memory on standard runners.
- script: dotnet build src\Microsoft.Graph\Microsoft.Graph.Beta.csproj --no-restore -f netstandard2.0
displayName: Build SDK (netstandard2.0)
workingDirectory: $(Build.SourcesDirectory)

- script: dotnet test Microsoft.Graph.Beta.sln --no-build
- script: dotnet build src\Microsoft.Graph\Microsoft.Graph.Beta.csproj --no-restore -f netstandard2.1
displayName: Build SDK (netstandard2.1)
workingDirectory: $(Build.SourcesDirectory)

- script: dotnet build src\Microsoft.Graph\Microsoft.Graph.Beta.csproj --no-restore -f net8.0
displayName: Build SDK (net8.0)
workingDirectory: $(Build.SourcesDirectory)

- script: dotnet build src\Microsoft.Graph\Microsoft.Graph.Beta.csproj --no-restore -f net10.0
displayName: Build SDK (net10.0)
workingDirectory: $(Build.SourcesDirectory)

- script: dotnet build tests\Microsoft.Graph.DotnetCore.Test\Microsoft.Graph.DotnetCore.Test.csproj --no-restore
displayName: Build test project
workingDirectory: $(Build.SourcesDirectory)

- script: dotnet test tests\Microsoft.Graph.DotnetCore.Test\Microsoft.Graph.DotnetCore.Test.csproj --no-build
displayName: Run unit tests
workingDirectory: $(Build.SourcesDirectory)
5 changes: 1 addition & 4 deletions src/Microsoft.Graph/Microsoft.Graph.Beta.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,10 +85,7 @@
<PackageReference Include="Microsoft.Graph.Core" Version="4.*" />
<PackageReference Include="Microsoft.SourceLink.GitHub" Version="10.0.203" PrivateAssets="All" />
</ItemGroup>
<ItemGroup Condition=" '$(TargetFramework)' == 'netstandard2.0' ">
Comment thread
ramsessanchez marked this conversation as resolved.
<!-- System.Net.Http is required for netstandard2.0 compatibility; suppress NU1510 pruning warning from .NET 10+ SDK -->
<PackageReference Include="System.Net.Http" Version="4.3.4" NoWarn="NU1510" />
</ItemGroup>

<ItemGroup>
<Folder Include="Models\" />
</ItemGroup>
Expand Down
Loading