Skip to content

Skip post-deploy trigger sync for self-hosted MCP deployments#5035

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-deploy-python-mcp-project
Draft

Skip post-deploy trigger sync for self-hosted MCP deployments#5035
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-deploy-python-mcp-project

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 20, 2026

Deployments of self-hosted Python MCP projects to Premium/App Service Plan apps using Secrets auth were failing after zip deploy with host runtime ServiceUnavailable. The failure path came from shared post-deploy trigger synchronization, not from package upload.

  • Deployment flow: disable trigger sync for MCP projects

    • Added DisableSyncTriggersForMcpExecuteStep to explicitly set context.syncTriggersPostDeploy = false when context.isMcpProject is true.
    • Priority is set to run before the shared PostDeploySyncTriggersExecuteStep, preventing the host-runtime sync call for MCP deploys.
  • Wiring in deploy pipeline

    • Updated deploy.ts to include the new step in innerDeploy(...) optional steps, ahead of remote MCP server registration.
  • Coverage

    • Added focused unit tests for the new step:
      • executes only for MCP projects
      • clears syncTriggersPostDeploy when executed
// deploy.ts
await innerDeploy(site, deployFsPath, deployContext, [
  new DisableSyncTriggersForMcpExecuteStep(),
  new CreateRemoteMcpServerExecuteStep()
]);

// DisableSyncTriggersForMcpExecuteStep.ts
public async execute(context: T): Promise<void> {
  context.syncTriggersPostDeploy = false;
}

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • aka.ms
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node scripts/pretest.mjs (dns block)
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI requested review from Copilot and removed request for Copilot May 20, 2026 18:21
Copilot AI changed the title [WIP] Fix deployment issue for self-hosted Python MCP project Skip post-deploy trigger sync for self-hosted MCP deployments May 20, 2026
Copilot AI requested a review from nturinski May 20, 2026 18:23
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.

Fail to deploy self-hosted Python MCP project to Premium/App Service Plan function app with "Secrets" authentication type

2 participants