Skip to content

fix(ci): daily CI pipeline configuration#3136

Merged
ramsessanchez merged 6 commits into
mainfrom
fix/daily-ci-pipeline-config
Jun 19, 2026
Merged

fix(ci): daily CI pipeline configuration#3136
ramsessanchez merged 6 commits into
mainfrom
fix/daily-ci-pipeline-config

Conversation

@ramsessanchez

@ramsessanchez ramsessanchez commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes the daily CI pipeline configuration to match changes from microsoftgraph/msgraph-sdk-dotnet-core#1063.

Changes

  • Added missing \image\ property to the 1ES pool config (required by 1ES Pipeline Templates)
  • Switched pool OS to Windows to align with the release pipeline
  • Added Azure Artifacts feed for NuGet restore to bypass network isolation on the \Azure-Pipelines-1ESPT-ExDShared\ pool
    • \NuGetAuthenticate@1\ step for feed authentication
    • Inline
      uget.config\ generation using the \NUGET_FEED_URI\ pipeline variable (set via ADO UI)
  • Installed both .NET 8 and .NET 10 SDKs to support all test target frameworks
Microsoft Reviewers: Open in CodeFlow

- Add missing image property to 1ES pool config
- Switch pool OS to Windows to align with release pipeline
- Add Azure Artifacts feed for NuGet restore to bypass network isolation
- Install both .NET 8 and .NET 10 SDKs for all test target frameworks

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ramsessanchez ramsessanchez requested a review from a team as a code owner June 5, 2026 22:27
Copilot AI review requested due to automatic review settings June 5, 2026 22:27

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

This PR updates the daily Azure Pipelines CI definition to align with upstream pipeline template requirements and the repo’s multi-targeting needs (including net10.0), while adapting to the 1ES ExDShared pool constraints.

Changes:

  • Updated the 1ES pool configuration to run on Windows and added the required image property.
  • Installed .NET 8 and .NET 10 SDKs in the pipeline to support all target frameworks.
  • Added Azure Artifacts authentication and generated a NuGet config for restore under network isolation.

Comment thread .azure-pipelines/daily-ci-build.yml Outdated
Comment thread .azure-pipelines/daily-ci-build.yml
ramsessanchez and others added 5 commits June 5, 2026 16:07
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
The daily CI build was failing due to out-of-memory when building all
4 TFMs in parallel. This matches the fix applied in msgraph-beta-sdk-dotnet PR #1127.

Changes:
- Build each TFM sequentially instead of solution-wide parallel build
- Build and test the test project explicitly by path

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Switch from the default Azure-Pipelines-1ESPT-ExDShared pool to
1es-windows-ps-compute-m which has more memory, matching the update
in msgraph-beta-sdk-dotnet PR #1127.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ramsessanchez ramsessanchez merged commit b6f4b45 into main Jun 19, 2026
10 checks passed
@ramsessanchez ramsessanchez deleted the fix/daily-ci-pipeline-config branch June 19, 2026 22:22
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