Skip to content

[ci] enable PR checker trigger for release branches#382

Merged
yxieca merged 1 commit into
sonic-net:masterfrom
yijingyan2:enable-PR-checker-release-branch
Jun 16, 2026
Merged

[ci] enable PR checker trigger for release branches#382
yxieca merged 1 commit into
sonic-net:masterfrom
yijingyan2:enable-PR-checker-release-branch

Conversation

@yijingyan2

@yijingyan2 yijingyan2 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Approach

What is the motivation for this PR?

The Azure Pipelines PR checker for sonic-dash-ha only triggered on PRs targeting the master branch. PRs raised against release branches (e.g. 202605) did not run the PR validation pipeline, leaving release-branch changes unverified.

How did you do it?

Added the 202??? branch pattern to the pr.branches.include list so the PR checker also triggers for release branches.

Signed-off-by: Yijing Yan <yijingyan@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

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

Correct and complete. Adding 202??? to both trigger: and pr: validates release-branch builds and PRs — the 202605 branch exists here, so these were previously unverified.

Checked:

  • The 202??? glob is right — ? matches a single character in Azure DevOps branch filters, so it matches 202605, 202505, etc., consistent with the rest of sonic-net.
  • Good that both trigger: and pr: were updated.
  • Azure pipeline is green.

One small thing: the PR description says "The Azure Pipelines PR checker for sonic-dash-ha…" — looks like a copy-paste from the sibling PR; this repo is sonic-snmpagent. Worth fixing the body, not blocking.

LGTM.

@yxieca yxieca merged commit 40b78f9 into sonic-net:master Jun 16, 2026
6 checks passed
@mssonicbld

Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202605: #384

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants