Skip to content

Move the glue trigger to a separate job after all builds are done#321

Merged
cmmarslender merged 2 commits into
mainfrom
trigger-separate-job
Feb 23, 2026
Merged

Move the glue trigger to a separate job after all builds are done#321
cmmarslender merged 2 commits into
mainfrom
trigger-separate-job

Conversation

@cmmarslender
Copy link
Copy Markdown
Member

@cmmarslender cmmarslender commented Feb 23, 2026

So we can be sure both the arm64 and amd64 debs are done at the time of trigger.


Note

Low Risk
CI-only change that reorders when an external release notification is sent; risk is limited to potentially delaying or skipping the repo-update trigger if the new job or its dependencies are misconfigured.

Overview
Moves the release-time Glue trigger out of the build matrix and into a dedicated trigger-repo-update job that runs only on release events and needs: [build], ensuring the repo update is triggered only after all build variants (e.g., amd64/arm64 .debs) have finished.

The new job performs a shallow checkout, obtains a JWT via Chia-Network/actions/github/jwt@main, then calls Chia-Network/actions/github/glue@main with the release tag payload for the chiavdf project.

Written by Cursor Bugbot for commit 22a3c08. This will update automatically on new commits. Configure here.

… we can be sure both the arm64 and amd64 debs are done at the time of trigger
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread .github/workflows/build-packages.yml
Copy link
Copy Markdown
Member

@hoffmang9 hoffmang9 left a comment

Choose a reason for hiding this comment

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

Should we change the yaml name?

@cmmarslender
Copy link
Copy Markdown
Member Author

Should we change the yaml name?

I think we need to change the mac intel runner too - let me make a few more updates...

Copy link
Copy Markdown
Member

@hoffmang9 hoffmang9 left a comment

Choose a reason for hiding this comment

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

lgtm

@cmmarslender cmmarslender merged commit a477d96 into main Feb 23, 2026
132 of 137 checks passed
@cmmarslender cmmarslender deleted the trigger-separate-job branch February 23, 2026 23:58
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