Reduce Azure SQL test scope in PR pipelines - #4609
Draft
saurabh500 wants to merge 2 commits into
Draft
Conversation
Run only connectivity and core bulk-copy manual tests against Azure SQL Database in PR pipelines while preserving full scheduled CI coverage. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: a97820b3-c016-4be6-9a0e-37f7e1ef3088
Contributor
There was a problem hiding this comment.
Pull request overview
Focuses Azure SQL manual-test coverage in PR pipelines while preserving full scheduled CI coverage.
Changes:
- Restricts Azure SQL PR tests to set 2 connectivity and BulkCopy tests.
- Adds shared regular and flaky test filters.
- Propagates filters through reusable test templates.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
eng/pipelines/sqlclient-pr-project-ref-pipeline.yml |
Enables focused Azure testing. |
eng/pipelines/sqlclient-pr-package-ref-pipeline.yml |
Enables focused Azure testing. |
eng/pipelines/pr/stages/test-stages.yml |
Applies the focused filter and set 2. |
eng/pipelines/pr/sqlclient-pr-pipeline.yml |
Imports shared filter variables. |
eng/pipelines/pr/jobs/test-sqlclientmanual-job.yml |
Adds manual-test filter support. |
eng/pipelines/dotnet-sqlclient-ci-core.yml |
Conditionally narrows Azure matrices. |
eng/pipelines/common/variables/azure-sql-test-filters.yml |
Defines regular and flaky filters. |
eng/pipelines/common/templates/steps/run-all-tests-step.yml |
Applies filters to manual-test runs. |
eng/pipelines/common/templates/stages/ci-run-tests-stage.yml |
Propagates configuration filters. |
eng/pipelines/common/templates/jobs/ci-run-tests-job.yml |
Forwards filters to test steps. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Use Windows-compatible double quotes so VSTest receives the intended category expression instead of treating apostrophes as part of the property name. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: a97820b3-c016-4be6-9a0e-37f7e1ef3088
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Limit Azure SQL Database manual-test legs in all three PR pipelines to:
SQL/ConnectivityTestsManualTests/BulkCopysuiteThe required
sqlclient-prpipeline now applies the focused filter to its Azure job. The Package- and Project-reference PR pipelines additionally reduce their Azure SQL matrix from test sets 1/2/3 to set 2, where all selected tests live, while retaining both regular and flaky runs.Scheduled CI, local SQL Server, LocalDB, named-instance, unit, functional, Always Encrypted, and Azure extension-package coverage are unchanged. Each reference PR pipeline's Azure SQL matrix drops from 51 jobs to 17, removing 68 jobs across the two pipelines.
Issues
N/A
Testing
sqlclient-pr,PR-SqlClient-Package, andPR-SqlClient-Projectfrom this branch.Guidelines
Please review the contribution guidelines before submitting a pull request: