Skip to content

T8144: Updated trigger-docker-image-build workflow for pull_request_t… - #1094

Merged
c-po merged 1 commit into
vyos:currentfrom
kumvijaya:current
Jan 4, 2026
Merged

T8144: Updated trigger-docker-image-build workflow for pull_request_t…#1094
c-po merged 1 commit into
vyos:currentfrom
kumvijaya:current

Conversation

@kumvijaya

@kumvijaya kumvijaya commented Jan 4, 2026

Copy link
Copy Markdown
Contributor

…arget policy change

Change summary

This PR updates pull_request_target workflows to comply with GitHub's recent policy change announced on November 7, 2025 Actions: pull_request_target and environment branch protections changes.

Background
GitHub updated how pull_request_target workflows are evaluated. Previously, these workflows were evaluated using the PR's target branch. Now, they are always evaluated using the repository's default branch (current). This means branch-filtered workflows will only trigger reliably if they exist and match on the default branch.

Changes
Updated the workflows to include all supported branches in their pull_request_target trigger:

Since pull_request_target workflows are now evaluated from the default branch, the workflow file must exist unchanged on the default branch. To support branch-specific triggering across all maintained branches (current, circinus, sagitta), the branch filters must explicitly list all branches or be removed entirely.

These changes ensure that:
Workflows trigger correctly for PRs targeting any of the three supported branches
The workflow files are evaluated from the current branch as per GitHub's new requirement

Note:
As this workflow doesn't have any branch-specific logic (works as per the incoming target branch), no changes needed apart from this.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

https://vyos.dev/T8144

Related PR(s)

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@github-actions

github-actions Bot commented Jan 4, 2026

Copy link
Copy Markdown

👍
No issues in PR Title / Commit Title

@kumvijaya
kumvijaya requested a review from c-po January 4, 2026 10:37
@github-actions github-actions Bot added the current VyOS rolling release label Jan 4, 2026
@kumvijaya
kumvijaya enabled auto-merge January 4, 2026 10:38

@c-po c-po left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add LTS branches to pipeline

@c-po
c-po disabled auto-merge January 4, 2026 10:41
@c-po
c-po merged commit c289ef5 into vyos:current Jan 4, 2026
7 checks passed
@vyosbot vyosbot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Jan 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current VyOS rolling release mirror-completed

Development

Successfully merging this pull request may close these issues.

4 participants