Skip to content

T8390: Support manual workflow dispatch for package rebuild workflow#109

Merged
c-po merged 2 commits intovyos:currentfrom
kumvijaya:current
Mar 17, 2026
Merged

T8390: Support manual workflow dispatch for package rebuild workflow#109
c-po merged 2 commits intovyos:currentfrom
kumvijaya:current

Conversation

@kumvijaya
Copy link
Copy Markdown
Contributor

@kumvijaya kumvijaya commented Mar 16, 2026

Change Summary

Enable manual workflow triggering via workflow_dispatch in addition to automatic PR merge triggers. Updated the job condition to run when either a pull request is merged OR the workflow is manually dispatched, providing more flexibility for package rebuilds.

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)
  • Other (please describe):

Related Task(s)

https://vyos.dev/T8390

Related PR(s)

Proposed changes

How to test

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

@c-po
Copy link
Copy Markdown
Member

c-po commented Mar 17, 2026

This looks AI generated/assisted. I wo der why the headline states remove, but its oviously an extension / new conditional path.

Copy link
Copy Markdown
Member

@sever-sever sever-sever left a comment

Choose a reason for hiding this comment

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

Allow manual start of the workflow

@kumvijaya
Copy link
Copy Markdown
Contributor Author

kumvijaya commented Mar 17, 2026

This looks AI generated/assisted. I wo der why the headline states remove, but its oviously an extension / new conditional path.

@c-po my bad, inititally it was remove only (yes, took ai assistance to do that writeup),.
then updated as per the discussion in slack, updated the PR info now

@kumvijaya kumvijaya changed the title T8390: Remove PR merge requirement from workflow to allow manual dispatch T8390: Support manual workflow dispatch for package rebuild workflow Mar 17, 2026
@c-po c-po merged commit af440e4 into vyos:current Mar 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants