Skip to content

{Compute} Fix vmss extension set bug#33755

Merged
yanzhudd merged 1 commit into
Azure:devfrom
william051200:fix-set-vmss-extension
Jul 21, 2026
Merged

{Compute} Fix vmss extension set bug#33755
yanzhudd merged 1 commit into
Azure:devfrom
william051200:fix-set-vmss-extension

Conversation

@william051200

@william051200 william051200 commented Jul 21, 2026

Copy link
Copy Markdown
Member

Related command

az vmss extension set

Description

Bug: Same type of extension will be parse together in a list causing extension name 'xxx' cannot be used for more than one extension.

Testing Guide

History Notes


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

🤖 PR Validation — ️✔️ All clear

Breaking Changes Tests
️✔️ None ️✔️ 130/130

@william051200
william051200 requested a review from a team as a code owner July 21, 2026 01:49
Copilot AI review requested due to automatic review settings July 21, 2026 01:49
@azure-client-tools-bot-prd

Copy link
Copy Markdown

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

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

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.

Pull request overview

Fixes a bug in the VMSS extension update flow where existing extensions weren’t being properly filtered out, leading to duplicate extensions and the service error extension name 'xxx' cannot be used for more than one extension. This is within the vm command module’s implementation of az vmss extension set.

Changes:

  • Update the extension de-duplication filter to compare against the correct extension field (type) when removing an existing extension before adding the new one.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 5329 to 5332
extension_profile['extensions'] = \
[x for x in extensions if
x.get('type_properties_type', '').lower() != extension_name.lower() or
x.get('type', '').lower() != extension_name.lower() or
x.get('publisher', '').lower() != publisher.lower()]
@yonzhan

yonzhan commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Compute

@yanzhudd

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).

@yanzhudd
yanzhudd merged commit db89d59 into Azure:dev Jul 21, 2026
61 checks passed
@william051200
william051200 deleted the fix-set-vmss-extension branch July 21, 2026 06:13
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 Compute az vm/vmss/image/disk/snapshot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants