Skip to content

Conversation

@YuliiaKovalova
Copy link
Member

@YuliiaKovalova YuliiaKovalova commented Aug 22, 2025

Attempt 2 for #12126

The previous one was reverted due to problems with exp pipelines.
Now it got resolved with some tweaks around package validation in VS tests.

validated in scope of https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/663724

Copilot AI review requested due to automatic review settings August 22, 2025 11:37
@YuliiaKovalova YuliiaKovalova requested a review from a team as a code owner August 22, 2025 11:37
Copy link
Contributor

Copilot AI left a comment

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 splits the internal Azure Pipeline configuration to separate experimental/performance builds from production builds. The change addresses previous pipeline failures in experimental builds by creating dedicated pipeline configurations with different validation settings.

Key changes:

  • Creates a new dedicated experimental/performance pipeline with reduced validation requirements
  • Removes experimental branch triggers from the main production pipeline
  • Adds signing configuration overrides to handle NGEN validation issues in experimental VS insertions

Reviewed Changes

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

Show a summary per file
File Description
azure-pipelines/vs-insertion-experimental.yml Updates pipeline references from 'MSBuild' to 'MSBuildExpPerf' to consume the new experimental pipeline
azure-pipelines/.vsts-dotnet-exp-perf.yml Creates new experimental/performance pipeline with simplified validation settings
azure-pipelines/.vsts-dotnet-build-jobs.yml Extracts common build job definitions into reusable template with parameterized features
Directory.Build.targets Adds signing configuration overrides for test signing to prevent NGEN validation issues
.vsts-dotnet.yml Removes experimental branch triggers and refactors to use the new build jobs template
Comments suppressed due to low confidence (1)

@YuliiaKovalova YuliiaKovalova enabled auto-merge (squash) August 22, 2025 13:58
@YuliiaKovalova YuliiaKovalova merged commit b4a341d into main Aug 22, 2025
9 checks passed
@YuliiaKovalova YuliiaKovalova deleted the dev/ykovalova/split_prod_exp_branches branch August 22, 2025 14:38
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.

4 participants