Skip to content

[ci] Only run dnceng-public pipeline for fork PRs#11153

Merged
jonathanpeppers merged 1 commit intomainfrom
jonathanpeppers/dev-peppers-optimize-public-ci-triggers
Apr 17, 2026
Merged

[ci] Only run dnceng-public pipeline for fork PRs#11153
jonathanpeppers merged 1 commit intomainfrom
jonathanpeppers/dev-peppers-optimize-public-ci-triggers

Conversation

@jonathanpeppers
Copy link
Copy Markdown
Member

Description

The dnceng-public pipeline (azure-pipelines-public.yaml) currently runs CI
builds on pushes to main, release/*, and feature/*, and also triggers for
all PRs (not just forks). This duplicates work already handled by the internal
DevDiv pipeline, wasting public pool resources.

This PR restricts the dnceng-public pipeline to only run for fork PRs:

  • Remove CI trigger -- set trigger: none so pushes to main/release/feature
    no longer trigger builds on dnceng-public.
  • Add fork-only condition to all stages -- every stage now checks
    eq(variables['System.PullRequest.IsFork'], 'True') so non-fork PRs that
    still trigger the pipeline will skip all stages immediately.

The pr: trigger cannot filter by fork status in YAML, so non-fork PRs will
still trigger the pipeline, but no stages will execute.

  • Useful description of why the change is necessary.
  • N/A -- no issue linked
  • N/A -- CI pipeline config change, no unit tests applicable

Remove CI triggers (pushes to main/release/feature) from the public
pipeline since those are already handled by the internal DevDiv pipeline.

Add fork-only condition to all stages so non-fork PRs skip immediately,
freeing up dnceng-public pool usage.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 17, 2026 21:05
Copy link
Copy Markdown
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

Updates the public Azure DevOps pipeline configuration to avoid duplicating CI work already covered by the internal DevDiv pipeline, conserving public build pool capacity by ensuring the public pipeline only executes for fork-based PRs.

Changes:

  • Disabled branch CI triggers in the public pipeline via trigger: none.
  • Added fork-only stage conditions using System.PullRequest.IsFork so stages skip for non-fork PR validations.

@jonathanpeppers
Copy link
Copy Markdown
Member Author

Seems to work:

image

@jonathanpeppers jonathanpeppers merged commit 39301e9 into main Apr 17, 2026
6 of 7 checks passed
@jonathanpeppers jonathanpeppers deleted the jonathanpeppers/dev-peppers-optimize-public-ci-triggers branch April 17, 2026 21:16
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.

2 participants