Skip to content

Abstractions Package - Pipeline Changes - #3628

Merged
paulmedynski merged 5 commits into
feat/azure-splitfrom
dev/paul/azure-split/abstractions-pipelines
Oct 16, 2025
Merged

Abstractions Package - Pipeline Changes#3628
paulmedynski merged 5 commits into
feat/azure-splitfrom
dev/paul/azure-split/abstractions-pipelines

Conversation

@paulmedynski

Copy link
Copy Markdown
Contributor

Overview

As part of the Azure split work, a new Abstractions package is being created that contains types shared between MDS and its future extensions (Azure will be the first). This PR contains the YAML pipeline changes for CI and Official builds.

This supercedes the original PRs #3471 and #3567.

This depends on PR #3626.

Issues

The first step towards #1108.

Testing

  • Manual inspection of all artifacts produced by the pipelines.

@paulmedynski paulmedynski added this to the 7.0-preview2 milestone Sep 23, 2025
@priyankatiwari08 priyankatiwari08 self-assigned this Sep 23, 2025
@paulmedynski
paulmedynski force-pushed the dev/paul/azure-split/abstractions-code branch from 96b8487 to 383a354 Compare September 23, 2025 20:42
@paulmedynski
paulmedynski force-pushed the dev/paul/azure-split/abstractions-pipelines branch from b02a214 to 49ec317 Compare September 24, 2025 13:46
@paulmedynski

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@codecov

codecov Bot commented Sep 25, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (feat/azure-split@2843bdd). Learn more about missing BASE report.

Additional details and impacted files
@@                 Coverage Diff                 @@
##             feat/azure-split    #3628   +/-   ##
===================================================
  Coverage                    ?   90.82%           
===================================================
  Files                       ?        6           
  Lines                       ?      316           
  Branches                    ?        0           
===================================================
  Hits                        ?      287           
  Misses                      ?       29           
  Partials                    ?        0           
Flag Coverage Δ
addons 90.82% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@paulmedynski paulmedynski changed the title [DRAFT] Abstractions Package - Pipeline Changes Abstractions Package - Pipeline Changes Sep 25, 2025
@paulmedynski
paulmedynski marked this pull request as ready for review September 25, 2025 15:29
@paulmedynski
paulmedynski requested a review from a team as a code owner September 25, 2025 15:29

@paulmedynski paulmedynski left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Commentary for reviewers, and some actions for myself.

Comment thread eng/pipelines/common/templates/jobs/build-signed-package-job.yml
Comment thread eng/pipelines/common/templates/jobs/ci-build-nugets-job.yml
Comment thread eng/pipelines/common/templates/jobs/ci-build-nugets-job.yml
Comment thread eng/pipelines/common/templates/jobs/ci-build-nugets-job.yml
Comment thread eng/pipelines/common/templates/jobs/ci-run-tests-job.yml
Comment thread eng/pipelines/libraries/ci-build-variables.yml
Comment thread eng/pipelines/libraries/common-variables.yml
Comment thread eng/pipelines/steps/compound-build-akv-step.yml
Comment thread eng/pipelines/steps/roslyn-analyzers-akv-step.yml
Comment thread eng/pipelines/dotnet-sqlclient-ci-core.yml
@paulmedynski

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@paulmedynski

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@paulmedynski

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@paulmedynski paulmedynski left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Commentary for reviewers.

Comment thread eng/pipelines/common/templates/jobs/ci-build-nugets-job.yml
Comment thread eng/pipelines/common/templates/jobs/ci-build-nugets-job.yml
Comment thread eng/pipelines/common/templates/jobs/build-signed-package-job.yml
Comment thread eng/pipelines/common/templates/jobs/validate-signed-package-job.yml
Comment thread eng/pipelines/common/templates/steps/ci-project-build-step.yml
Comment thread eng/pipelines/common/templates/steps/copy-dlls-for-test-step.yml
Comment thread build.proj
Base automatically changed from dev/paul/azure-split/abstractions-code to feat/azure-split September 30, 2025 18:11
@paulmedynski
paulmedynski force-pushed the dev/paul/azure-split/abstractions-pipelines branch 2 times, most recently from 75eb084 to 181aa0c Compare September 30, 2025 18:23
@mdaigle mdaigle self-assigned this Sep 30, 2025
mdaigle
mdaigle previously approved these changes Oct 1, 2025

@mdaigle mdaigle 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.

Overall, things look good to me. I think it would be helpful to have a walkthrough of a pipeline run to show us the different build artifacts, their version numbers, etc. so that we can see how it all fits together in the end.

Comment thread eng/pipelines/common/templates/jobs/build-signed-package-job.yml

@paulmedynski paulmedynski left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Commentary for reviewers.

Comment thread eng/pipelines/common/templates/steps/code-analyze-step.yml
Comment thread eng/pipelines/jobs/build-akv-official-job.yml
Comment thread eng/pipelines/steps/compound-build-akv-step.yml
This was referenced Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants