Add App Service deployment slot routing based on deployment history by Copilot · Pull Request #6627 · Azure/azure-dev

Copilot · 2026-01-28T00:58:46Z

App Service Deployment Slots Support - Phase 1 Foundation

This PR implements the foundational support for Azure App Service deployment slots as outlined in issue #6526. It focuses on intelligent deployment routing based on deployment history and available slots.

🎯 What This PR Does

Adds automatic slot detection and smart deployment logic to azd deploy for App Service targets:

First deployment: Deploys to main app + all slots (ensures consistency)
Single slot scenario: Automatically deploys to that slot
Multiple slots: Prompts user to select target (including main app)
No slots: Deploys to main app only (existing behavior)

📝 Changes Made

Core API Methods (`cli/azd/pkg/azapi/webapp.go`)

HasAppServiceDeployments() - Checks deployment history via Azure Web Apps API
GetAppServiceSlots() - Retrieves available deployment slots
DeployAppServiceSlotZip() - Deploys zip package to specific slot

Service Target Updates (`cli/azd/pkg/project/service_target_appservice.go`)

Updated appServiceTarget to include console for user interaction
Modified Deploy() to support multi-target deployment
Added determineDeploymentTargets() with intelligent routing logic
Comprehensive error handling with target-specific messages
Support for AZD_DEPLOY_{SERVICE}_SLOT_NAME environment variable to skip prompts
Deployment messages mirror Azure Portal terminology:
- No slots: "Uploading deployment package"
- Main app with slots: "Uploading deployment package to production slot"
- Named slots: "Uploading deployment package to slot 'name'"

Test Coverage

Unit tests (cli/azd/pkg/azapi/webapp_slots_test.go):
- HasAppServiceDeployments (with/without deployments)
- GetAppServiceSlots (with/without slots)
- DeployAppServiceSlotZip (success/not found scenarios)
Functional test (cli/azd/test/functional/deploy_test.go):
- Test_CLI_Deploy_SlotDeployment - End-to-end test of slot deployment workflow
- Uses serviceHealthProber struct with session-aware retry delays
- Context and testing.T passed as method parameters (not stored in struct)
- Reuses existing cleanupRg function for resource cleanup

✅ Quality Assurance

🔐 Security

No security vulnerabilities found (CodeQL scan)
Proper nil pointer checks prevent potential panics
Safe resource cleanup prevents file handle leaks

📚 Documentation

Added detailed documentation explaining the deployment strategy:

Rationale for deploying to all slots on first deployment
Explanation of single vs. multiple slot scenarios
User experience considerations for slot selection
Environment variable for skipping prompts in CI/CD scenarios

🚀 What's Next (Future Phases)

This PR implements Phase 1 foundation. Future enhancements:

Phase 2: CLI flags (--to, --traffic, --status)
Phase 3: Traffic management and slot swapping
Phase 4: Slot-specific configuration support

🔗 Related Issue

Closes #6526 (Phase 1)

Original prompt

let's use this to fix #6526. Create a PR with the current changes and the context of this conversation.
Do not make changes to /extension folder

[Chronological Review: The conversation began with the user requesting the flow path for running `azd deploy` for a host of `appservice`, specifically looking for a command that calls a manager to invoke the app service implementation with `zipdeploy`. The user then inquired about the logic behind `GetServiceTarget` and how resources are identified for deployment. Following this, the user asked about which `serviceTarget` has a custom resolver and how it operates. The user then shifted focus to updating the app service to utilize a custom resolver, requesting the implementation details for this change. The conversation evolved to discuss deploying app service to slots, with specific conditions for deployment based on previous deployments and slot availability. The user clarified the deployment logic, emphasizing that if there are no previous deployments, the app should be deployed to both the main web app and any slots. The user also requested to create a PR to fix a specific GitHub issue without modifying the `/extension` folder.]

[Intent Mapping:

"find the flow path for running azd deploy for a host=appservice." - User wants to understand the deployment command structure.
"What's the logic behind GetServiceTarget?" - User seeks clarity on resource identification for deployment.
"what serviceTarget has custom resolver?" - User is interested in understanding custom resolvers.
"Can we update appservice to use a custom resolver as well?" - User wants to implement a custom resolver for app service.
"let's directly update the Deploy() implementation for appservice." - User decides to modify the deployment logic instead of using a custom resolver.
"what happens in case of no previous deployments + multiple slots?" - User seeks clarification on deployment behavior in specific scenarios.
"let's do number 2. Deploy to main and all slots." - User confirms the desired deployment strategy.
"let's use this to fix App Service Deployment Slots Support - Phase 1 #6526." - User requests to create a PR with the changes discussed.]

[Technical Inventory:

azd deploy: Command for deploying applications.
GetServiceTarget: Function for identifying deployment resources.
Custom resolver: A mechanism to customize deployment logic.
App service slots: Deployment targets for Azure App Services.
GitHub issue App Service Deployment Slots Support - Phase 1 #6526: A specific issue related to the discussed changes.]

[Code Archaeology:

Deploy(): The main function to be modified for app service deployment logic.
New methods to be added:
- Method to check for previous deployments.
- Method to retrieve web app slots.]

[Progress Assessment:

Completed Tasks: User has outlined the deployment logic and confirmed the approach.
Partially Complete Work: Implementation details for the Deploy() method and PR creation are pending.]

[Context Validation: All necessary context for continuing the implementation of the Deploy() method and creating the PR is captured.]

[Recent Commands Analysis:

Last Agent Commands: User requested to create a PR with the current changes and context of the conversation, specifying not to modify the /extension folder.
Tool Results Summary: No specific tool results were returned as the focus was on planning and outlining changes.
Pre-Summary State: The agent was actively discussing the deployment logic for app service and preparing to create a PR.
Operation Context: The commands were executed to finalize the deployment strategy and address a specific GitHub issue.]

1. Conversation Overview: - Primary Objectives: - "find the flow path for running azd deploy for a host=appservice." - "What's the logic behind GetServiceTarget?" - "Can we update appservice to use a custom resolver as well?" - "let's use this to fix https://github.com//issues/6526." - Session Context: The conversation flowed from understanding deployment commands to discussing the logic behind deployment strategies and ultimately deciding on modifications to the `Deploy()` method for app service. - User Intent Evolution: The user shifted from exploring custom resolvers to directly modifying the deployment logic based on deployment history and slot availability.

Technical Foundation:

azd: Azure Developer CLI for deploying applications.
GetServiceTarget: Function for determining the target service for deployment.
Custom resolver: A feature to allow custom logic in deployment processes.
App service slots: Deployment targets that allow for staging and testing.

Codebase Status:

Deploy():
- Purpose: Main function for deploying the app service.
- Current State: Logic to be modified to handle deployment based on previous deployments and slot availability.
- Key Code Segments: N...

Created from VS Code.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

… and fix file handle cleanup Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

cli/azd/extensions/microsoft.azd.concurx/go.sum

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

cli/azd/test/functional/deploy_test.go

cli/azd/test/recording/recording.go

cli/azd/test/functional/deploy_test.go

…sion-based retry delay Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

cli/azd/test/functional/deploy_test.go

…meters instead Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

vhvb1989 · 2026-01-29T22:56:31Z

/azp run azure-dev - cli

azure-pipelines · 2026-01-29T22:56:56Z

Azure Pipelines successfully started running 1 pipeline(s).

cli/azd/pkg/project/service_target_appservice.go

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

weikanglim

Looks good, @vhvb1989! I appreciate the effort towards landing this cleanly and in a thoughtful manner. Kudos on making the e2e test readable and demonstrating the overall feature here.

vhvb1989 · 2026-01-31T00:38:53Z

/check-enforcer evaluate

vhvb1989 added 6 commits January 23, 2026 22:54

fix security issue with playwright/test 1.49.1 (#6592)

026aa97

Merge branch 'main' of https://github.com/Azure/azure-dev

cb20d72

Merge branch 'main' of https://github.com/Azure/azure-dev

6335069

Merge branch 'main' of https://github.com/Azure/azure-dev

fb08f19

Merge branch 'main' of https://github.com/Azure/azure-dev

c71904a

Checkpoint from VS Code for cloud agent session

eaf2514

Copilot AI assigned Copilot and vhvb1989 Jan 28, 2026

Copilot started work on behalf of vhvb1989 January 28, 2026 00:59 View session

Copilot AI and others added 3 commits January 28, 2026 01:06

Add unit tests for app service slot deployment methods

990499e

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Address code review feedback: add nil checks, improve error messages,…

bf679af

… and fix file handle cleanup Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Final implementation of App Service slots deployment - Phase 1

1809645

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Fix app service deployment logic with custom resolver~~ Add App Service deployment slot routing based on deployment history Jan 28, 2026

Copilot AI requested a review from vhvb1989 January 28, 2026 01:18

Copilot finished work on behalf of vhvb1989 January 28, 2026 01:18

vhvb1989 reviewed Jan 28, 2026

View reviewed changes

cli/azd/extensions/microsoft.azd.concurx/go.sum Show resolved Hide resolved

Copilot started work on behalf of vhvb1989 January 28, 2026 18:26 View session

Remove unintended changes to extension files

3edac61

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Copilot AI requested a review from vhvb1989 January 28, 2026 18:31

Copilot finished work on behalf of vhvb1989 January 28, 2026 18:31

support env var to skip prompt in multi-slot scenario

c363e98

vhvb1989 marked this pull request as ready for review January 28, 2026 19:21

vhvb1989 requested review from hemarina and wbreza as code owners January 28, 2026 19:21

Copilot AI review requested due to automatic review settings January 28, 2026 19:21

vhvb1989 requested review from JeffreyCA, rajeshkamal5050, tg-msft and weikanglim as code owners January 28, 2026 19:21

Copilot started reviewing on behalf of vhvb1989 January 28, 2026 19:21 View session

Update cli/azd/pkg/project/service_target_appservice.go

77e9ce2

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

weikanglim reviewed Jan 29, 2026

View reviewed changes

cli/azd/test/functional/deploy_test.go Outdated Show resolved Hide resolved

weikanglim reviewed Jan 29, 2026

View reviewed changes

cli/azd/test/recording/recording.go Outdated Show resolved Hide resolved

weikanglim reviewed Jan 29, 2026

View reviewed changes

cli/azd/test/functional/deploy_test.go Outdated Show resolved Hide resolved

endpoints

32e4f57

Copilot started work on behalf of vhvb1989 January 29, 2026 22:28 View session

Refactor test health probe to use serviceHealthProber struct with ses…

653de87

…sion-based retry delay Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Copilot finished work on behalf of vhvb1989 January 29, 2026 22:34

vhvb1989 reviewed Jan 29, 2026

View reviewed changes

cli/azd/test/functional/deploy_test.go Outdated Show resolved Hide resolved

Copilot started work on behalf of vhvb1989 January 29, 2026 22:38 View session

Remove ctx and t from serviceHealthProber struct, pass as method para…

3e42526

…meters instead Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Copilot finished work on behalf of vhvb1989 January 29, 2026 22:42

Copilot AI requested a review from vhvb1989 January 29, 2026 22:42

Copilot started work on behalf of vhvb1989 January 29, 2026 22:48 View session

Replace cleanupResourceGroup with existing cleanupRg function

e335b77

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Copilot finished work on behalf of vhvb1989 January 29, 2026 22:53

recording

e47582c

weikanglim reviewed Jan 30, 2026

View reviewed changes

cli/azd/pkg/project/service_target_appservice.go Outdated Show resolved Hide resolved

Copilot started work on behalf of vhvb1989 January 30, 2026 01:27 View session

Update deployment messages to mirror Azure Portal terminology

4065615

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>

Copilot finished work on behalf of vhvb1989 January 30, 2026 01:33

weikanglim approved these changes Jan 30, 2026

View reviewed changes

vhvb1989 enabled auto-merge (squash) January 30, 2026 21:45

vhvb1989 merged commit 79cfd1b into main Jan 31, 2026
7 checks passed

vhvb1989 deleted the copilot/vscode-mkxbgxd1-01dk branch January 31, 2026 00:45

This was referenced Feb 2, 2026

Create changelog for azd core version 1.24.0 #6680

Closed

Release azd core v1.24.0 #6681

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add App Service deployment slot routing based on deployment history#6627

Add App Service deployment slot routing based on deployment history#6627
vhvb1989 merged 20 commits intomainfrom
copilot/vscode-mkxbgxd1-01dk

Copilot AI commented Jan 28, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

vhvb1989 commented Jan 29, 2026

Uh oh!

azure-pipelines bot commented Jan 29, 2026

Uh oh!

Uh oh!

weikanglim left a comment

Uh oh!

vhvb1989 commented Jan 31, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

Copilot AI commented Jan 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

App Service Deployment Slots Support - Phase 1 Foundation

🎯 What This PR Does

📝 Changes Made

Core API Methods (cli/azd/pkg/azapi/webapp.go)

Service Target Updates (cli/azd/pkg/project/service_target_appservice.go)

Test Coverage

✅ Quality Assurance

🔐 Security

📚 Documentation

🚀 What's Next (Future Phases)

🔗 Related Issue

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

vhvb1989 commented Jan 29, 2026

Uh oh!

azure-pipelines bot commented Jan 29, 2026

Uh oh!

Uh oh!

weikanglim left a comment

Choose a reason for hiding this comment

Uh oh!

vhvb1989 commented Jan 31, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Copilot AI commented Jan 28, 2026 •

edited

Loading

Core API Methods (`cli/azd/pkg/azapi/webapp.go`)

Service Target Updates (`cli/azd/pkg/project/service_target_appservice.go`)