Skip to content

Conversation

MattBabbage
Copy link
Contributor

@MattBabbage MattBabbage commented Oct 10, 2025

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 with Option 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

  • Updated registry-releaser workflow: Added a 5-minute wait (with 30-second intervals) to check for Docker image availability before proceeding with MCP Registry publication
  • Simplified trigger approach: Uses version tag triggers (v*) instead of repository dispatch to avoid requiring elevated permissions

Benefits

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

@MattBabbage MattBabbage requested a review from a team as a code owner October 10, 2025 09:17
@Copilot Copilot AI review requested due to automatic review settings October 10, 2025 09:17
Copy link
Contributor

@Copilot Copilot AI left a 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.

@MattBabbage MattBabbage changed the title Link registry publication to Docker image availability Simplify Registry release pipeline Oct 10, 2025
Copy link
Contributor

@mattdholloway mattdholloway left a comment

Choose a reason for hiding this comment

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

looks good!

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.

3 participants