Skip to content

chore(release): use shared mcp-server-deploy reusable workflow - #98

Merged
asachs01 merged 1 commit into
mainfrom
chore/use-reusable-deploy-workflow
May 13, 2026
Merged

chore(release): use shared mcp-server-deploy reusable workflow#98
asachs01 merged 1 commit into
mainfrom
chore/use-reusable-deploy-workflow

Conversation

@asachs01

Copy link
Copy Markdown
Collaborator

Replaces this repo's inline deploy job with a call to the new canonical reusable workflow shipped in wyre-technology/.github#11.

Why

PR #97 fixed the what for autotask — targets gwp-autotask, deploys by digest. But the same fix is needed in 7 sibling *-mcp repos, all of which currently deploy to orphaned ACAs via :latest. Copy-pasting the same 40-line shell block 8 times is how this bug got here in the first place.

The new mcp-server-deploy.yml reusable workflow puts the deploy logic in one place. This PR is the reference call site; identical conversions for halopsa-mcp, datto-rmm-mcp, itglue-mcp, huntress-mcp, liongard-mcp, domotz-mcp, and cipp-mcp follow.

Diff shape

  • Inline deploy job (40+ lines of shell, env vars, az CLI invocation) → 9-line uses: call
  • Pinned to merge SHA c2d6fc89211757997102d100115d02997d4e3521 (not @main) so upstream changes to the reusable workflow can't silently alter this repo's pipeline
  • secrets: inherit forwards AZURE_CLIENT_ID / AZURE_TENANT_ID / AZURE_SUBSCRIPTION_ID from the caller's environment

Behavior parity

Identical to what just shipped in PR #97 and v2.25.2:

  • Targets gwp-autotask (not mcpgw-prod-autotask)
  • Image reference: ghcr.io/wyre-technology/autotask-mcp@<digest> (not :latest)
  • IMAGE_VERSION env on the deployed revision: v<version>-sha-<short>-<runId>
  • Hard-fails if digest input is empty or not sha256:-prefixed

Test plan

  • YAML parses
  • Next push-to-main release should produce a deploy log identical in shape to v2.25.2 (which I already verified runs correctly today)
  • Verify the gateway tool catalog stays healthy after that deploy

Replaces the inline deploy job with a call to the canonical
wyre-technology/.github@c2d6fc8.
Pinned to the merge SHA, not :main, so unreviewed upstream changes
can't silently alter this repo's release pipeline.

This is the reference call site — the same conversion is being applied
to halopsa-mcp, datto-rmm-mcp, itglue-mcp, huntress-mcp, liongard-mcp,
domotz-mcp, and cipp-mcp in parallel PRs. After full rollout, the
gwp-<vendor> + digest deploy logic lives in exactly one place across
the fleet.

Net diff: -41 +9 lines (inline shell deleted, replaced with `uses:` +
typed inputs).

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@asachs01
asachs01 merged commit ee7fc14 into main May 13, 2026
8 checks passed
@asachs01
asachs01 deleted the chore/use-reusable-deploy-workflow branch May 13, 2026 19:34
@github-project-automation github-project-automation Bot moved this from Todo to Done in MSP Claude Plugins May 13, 2026
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