Fix template path in quarantine pipeline - #14937
Merged
Merged
Conversation
Template paths resolve relative to the referencing file. quarantine.yml is already inside azure-pipelines/, so the azure-pipelines/ prefix resolved to azure-pipelines/azure-pipelines/ and the pipeline failed at YAML compile time on every run since dotnet#14371. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
AR-May
marked this pull request as ready for review
September 4, 2026 12:27
AR-May
enabled auto-merge (squash)
September 4, 2026 12:29
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The change is a straightforward correction of YAML template paths and appears consistent across all affected jobs with no functional risk beyond restoring the pipeline parsing.
Pull request overview
Fixes YAML template includes in the Azure DevOps quarantine pipeline by updating template: paths to be relative to azure-pipelines/quarantine.yml, restoring the pipeline’s ability to parse and run.
Changes:
- Update the
check-documentation-only-change.ymltemplate reference in each quarantine job (Windows/Linux/macOS) to use a relative path.
File summaries
| File | Description |
|---|---|
| azure-pipelines/quarantine.yml | Fix template include paths so YAML parsing succeeds and the quarantine pipeline can run again. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Contributor
|
pipelines still fail, |
VolPlita
reviewed
Sep 4, 2026
The quarantine definition fetches with depth 1, but check-documentation-only-change.yml runs `git diff HEAD HEAD~1`, which fails with "unknown revision" because the parent commit was never fetched. The step then fails the job before the build runs. msbuild-pr is unaffected because its definition sets fetchDepth 0. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0518e79a-a652-4869-aa2e-5dc82c738c0b
Member
Author
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
VolPlita
approved these changes
Sep 7, 2026
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.
The quarantine pipeline has not run since #14371 (~50 days!). It fails
while parsing YAML.
Cause:
template:paths resolve relative to the file that contains them.quarantine.ymlis already inazure-pipelines/, so the extra prefix pointed toa folder that does not exist.