Skip to content

{App Service} az webapp deploy: Add friendly name deploymentTag used to identify the deployment - #34134

Merged
Julie Zhu (yanzhudd) merged 3 commits into
Azure:devfrom
Shi1810:user/shikhajhadeploytag
Sep 28, 2026
Merged

Julie Zhu (yanzhudd) merged 3 commits into
Azure:devfrom
Shi1810:user/shikhajhadeploytag

Conversation

@Shi1810

@Shi1810 Shi1810 commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

🤖 PR Validation — ⚠️ Review suggested

Breaking Changes Tests
⚠️ None ️✔️ 130/130
⚠️AzureCLI-BreakingChangeTest
⚠️appservice
rule cmd_name rule_message suggest_message
⚠️ 1008 - ParaPropAdd webapp deploy cmd webapp deploy update parameter tag: added property options_deprecate_info=[{'target': '--tag', 'redirect': '--deploymentTag'}]
⚠️ 1010 - ParaPropUpdate webapp deploy cmd webapp deploy update parameter tag: updated property name from tag to deployment_tag
⚠️ 1010 - ParaPropUpdate webapp deploy cmd webapp deploy update parameter tag: updated property options from ['--tag'] to ['--deploymentTag', '--tag']

Related command

az webapp deploy

Description

This PR adds a capability to give friendly name to az webapp deploy commands via --deploymentTag to identify the deployment.

Testing Guide

az webapp deploy --resource-group shikhajhatestingrg --name shikhajha-securebuild-euap-2 --src-path ..live-secure-build-vulnerable.zip --type zip --async true --deploymentTag screenshot-live-test
image
image

History Notes

[App Service] az webapp deploy: Add --deploymentTag parameter to give friendly name to deployment


This checklist is used to make sure that common guidelines for a pull request are followed.

@Shi1810
Shi1810 requested a review from a team as a code owner September 25, 2026 11:17
Copilot AI lite review requested due to automatic review settings September 25, 2026 11:17
@azure-client-tools-bot-prd

Copy link
Copy Markdown

Hi Shi1810,
Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

@microsoft-github-policy-service microsoft-github-policy-service Bot added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Sep 25, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown
Contributor

Thank you for your contribution Shi1810! We will review the pull request and get back to you soon.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

One or more issues must be addressed before approval.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity

Open (1)
What changed in this PR

Adds a friendly deployment identifier to az webapp deploy for Linux web apps.

Changes:

  • Adds --deploymentTag.
  • Passes the value to OneDeploy requests.
  • Updates Windows-app warning coverage.
File Description
src/​azure-cli/​azure/​cli/​command_modules/​appservice/​tests/​latest/​test_webapp_commands_thru_mock.py Updated as part of this pull request.
src/​azure-cli/​azure/​cli/​command_modules/​appservice/​custom.py Updated as part of this pull request.
src/​azure-cli/​azure/​cli/​command_modules/​appservice/​_params.py Updated as part of this pull request.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/azure-cli/azure/cli/command_modules/appservice/_params.py Outdated
@yonzhan

Copy link
Copy Markdown
Collaborator

App Service

@jsntcy Yu Chen (jsntcy) added the Request X Engineering Agent Request X Engineering Agent testing and review label Sep 28, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown
Contributor

🔔 Routing this PR to @Azure/act-observability-squad.

@yanzhudd Julie Zhu (yanzhudd) changed the title [App Service] az webapp deploy: Add friendly name deploymentTag used to identify the deployment {App Service} az webapp deploy: Add friendly name deploymentTag used to identify the deployment Sep 28, 2026
@x-engineering-agent

Copy link
Copy Markdown
Contributor

Live test results — azdev test --live --series (changed test files only)

✅ PASS

Selectors: appservice.test_scm_url_includes_encoded_deployment_tag appservice.test_webapp_deploy_ignores_deployment_tag_for_windows_webapp appservice.test_arm_body_includes_deployment_tag (module)
PR head ref: user/shikhajhadeploytag
PR head sha: f0738ff8e0055e6f2c24fb1585f4999a1fff4cdf
PR base ref: dev
New test files in PR: false

Changed test files run
src/azure-cli/azure/cli/command_modules/appservice/tests/latest/test_webapp_commands_thru_mock.py

Workflow run: https://github.com/Azure/issue-sentinel/actions/runs/36377050576

Last 80 lines of azdev output

=============
| Run Tests |
=============


test index found: /home/runner/.azdev/env_config/home/runner/work/issue-sentinel/issue-sentinel/.venv/test_index/latest.json

Test on modules: appservice.test_scm_url_includes_encoded_deployment_tag, appservice.test_webapp_deploy_ignores_deployment_tag_for_windows_webapp, appservice.test_arm_body_includes_deployment_tag

WARNING: RUNNING TESTS LIVE
The tests are set to run against current profile "latest"
============================= test session starts ==============================
platform linux -- Python 3.12.14, pytest-9.1.1, pluggy-1.6.0 -- /home/runner/work/issue-sentinel/issue-sentinel/.venv/bin/python
cachedir: .pytest_cache
rootdir: /home/runner/work/issue-sentinel/issue-sentinel/azure-cli/src/azure-cli
plugins: platformdirs-4.12.0, forked-1.7.5, xdist-3.8.0
collecting ... collected 3 items

azure-cli/src/azure-cli/azure/cli/command_modules/appservice/tests/latest/test_webapp_commands_thru_mock.py::TestOneDeployTag::test_scm_url_includes_encoded_deployment_tag PASSED [ 33%]
azure-cli/src/azure-cli/azure/cli/command_modules/appservice/tests/latest/test_webapp_commands_thru_mock.py::TestOneDeployTag::test_webapp_deploy_ignores_deployment_tag_for_windows_webapp PASSED [ 66%]
azure-cli/src/azure-cli/azure/cli/command_modules/appservice/tests/latest/test_webapp_commands_thru_mock.py::TestOneDeployTag::test_arm_body_includes_deployment_tag PASSED [100%]

- generated xml file: /home/runner/work/issue-sentinel/issue-sentinel/test-output/results.xml -
============================== 3 passed in 0.65s ===============================

Posted by the X Engineering Agent live-test workflow.

@yanzhudd
Julie Zhu (yanzhudd) merged commit 304a51b into Azure:dev Sep 28, 2026
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

act-observability-squad Auto-Assign Auto assign by bot customer-reported Issues that are reported by GitHub users external to the Azure organization. Request X Engineering Agent Request X Engineering Agent testing and review Web Apps az webapp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants