[7.0.3 Cherry-pick] Move Kerberos pipeline to ci/ and rename to sqlclient-ci-kerberos-pipeline.yml#4471
Conversation
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
dae7ad2 to
a6a5037
Compare
There was a problem hiding this comment.
Pull request overview
Cherry-picks the Kerberos Azure DevOps pipeline relocation/rename into release/7.0 for the 7.0.3 servicing release, aligning the Kerberos pipeline with the repository’s eng/pipelines/ci/ organization and naming conventions.
Changes:
- Moved Kerberos pipeline assets under
eng/pipelines/ci/kerberos/and renamed the main pipeline file tosqlclient-ci-kerberos-pipeline.yml. - Updated template references inside the Kerberos pipeline to the new
ci/kerberospaths. - Updated PR validation pipeline path filters to exclude the new Kerberos pipeline location.
Reviewed changes
Copilot reviewed 3 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| eng/pipelines/sqlclient-pr-project-ref-pipeline.yml | Updates PR path exclusion to reflect the new Kerberos pipeline location under eng/pipelines/ci/. |
| eng/pipelines/sqlclient-pr-package-ref-pipeline.yml | Updates PR path exclusion to reflect the new Kerberos pipeline location under eng/pipelines/ci/. |
| eng/pipelines/ci/kerberos/sqlclient-ci-kerberos-pipeline.yml | Renames/moves Kerberos pipeline and updates internal template references to the new folder. |
| eng/pipelines/ci/kerberos/linux-init-step.yml | Adds the Linux Kerberos domain-join + TGT acquisition step template under the new location. |
| eng/pipelines/ci/kerberos/linux-cleanup-step.yml | Adds the Linux Kerberos cleanup step template under the new location. |
| eng/pipelines/ci/kerberos/build-and-test-steps.yml | Adds shared build/test/publish steps template for Kerberos jobs under the new location. |
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## release/7.0 #4471 +/- ##
===============================================
- Coverage 72.70% 65.67% -7.04%
===============================================
Files 280 275 -5
Lines 43090 65950 +22860
===============================================
+ Hits 31329 43310 +11981
- Misses 11761 22640 +10879
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary
Cherry-pick of #4468 (
188c71ed0d4422ab4141f1079443fd909ebfcee7) intorelease/7.0for the 7.0.3 servicing release. The original cherry-pick failed due to merge conflicts, which have now been resolved manually.Moves the Kerberos test pipeline under the
ci/directory and renames the main pipeline file to follow theci/naming convention (matchingci/stress/sqlclient-ci-stress-pipeline.yml).Changes
eng/pipelines/kerberos/→eng/pipelines/ci/kerberos/(folder with all templates, preserving history)sqlclient-kerberos.yml→sqlclient-ci-kerberos-pipeline.yml/eng/pipelines/ci/kerberos/...sqlclient-pr-package-ref-pipeline.ymlandsqlclient-pr-project-ref-pipeline.ymlConflict resolution
The conflict was in the PR trigger path filters of
sqlclient-pr-package-ref-pipeline.ymlandsqlclient-pr-project-ref-pipeline.yml. Onrelease/7.0these files exclude the Kerberos path explicitly, so instead of removing the entry (as done onmain), the excluded path was updated fromeng/pipelines/kerberos/*toeng/pipelines/ci/*to match the new pipeline location.