- Notifications
You must be signed in to change notification settings - Fork2.7k
Simplify Registry release pipeline#1204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
looks good!
250723d
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Summary
Registry Publication was failing due to insufficient permissions in the Docker Pipeline.
Option 1: increase permissions to
actions:write
Option 2: we just trigger on version release as prior.
Going withOption 2 just trigger on version release as its simpler, and is less of a security risk.
Docker image was still published, manual registry publish was run to resolve issue.
Pipeline with internal failure:https://github.com/github/github-mcp-server/actions/runs/18401464561/job/52431533739
Changes
v*
) instead of repository dispatch to avoid requiring elevated permissionsBenefits
✅Reliability: Registry publication only succeeds when Docker image is confirmed available
✅Security: No elevated workflow permissions required
✅Simplicity: Clean parallel execution with built-in coordination
✅Fail-safe: 5-minute timeout prevents hanging workflows