Check the installed verson when extension has patch version by lubaihua33 · Pull Request #4432 · microsoft/lisa

lubaihua33 · 2026-04-23T07:06:44Z

Description

When installing a VM extension using the Python SDK or an ARM template, there is only a single parameter available for version control: typeHandlerVersion. This parameter only supports the Major.Minor format. However, for the pre-publish validation scenario, the version is in Major.Minor.Patch format.

Azure platform has the following behavior:
Customers are only able to specify the Major.Minor version during deployment of an extension. When AutoUpgradeMinorVersion is set "True", then the platform will deliver the latest Minor.Patch.Hotfix version of that Major version. If this value is set to "False" then the platform will deliver the latest Patch.Hotfix of the Major.Minor pair.

This PR adds a check for the installed version if the extension version passed in the case is Major.Minor.Patch format. From the response of GET VM extension, we can get the installed version from the instance_view.type_handler_version. Then check if the installed version same as the expected version. If yes, then the case is passed. If not, the case is failed.

Related Issue

Type of Change

Checklist

Description is filled in above
No credentials, secrets, or internal details are included
Peer review requested (if not, add required peer reviewers after raising PR)
Tests executed and results posted below

Test Validation

Test for the following runbooks:

Case verify_vm_extension_install_uninstall should be PASSED.
variable:

name: extension_publisher
value: "Microsoft.Azure.Monitor"
is_case_visible: true
name: extension_type
value: "AzureMonitorLinuxAgent"
is_case_visible: true
name: extension_version
value: "1.7.1"
is_case_visible: true

Case verify_vm_extension_install_uninstall should be PASSED.
variable:

name: extension_publisher
value: "Microsoft.Azure.Monitor"
is_case_visible: true
name: extension_type
value: "AzureMonitorLinuxAgent"
is_case_visible: true
name: extension_version
value: "1.7"
is_case_visible: true

Case verify_vm_extension_install_uninstall should be FAILED.
variable:

name: extension_publisher
value: "Microsoft.Azure.Monitor"
is_case_visible: true
name: extension_type
value: "AzureMonitorLinuxAgent"
is_case_visible: true
name: extension_version
value: "1.7.0"
is_case_visible: true

Key Test Cases:

verify_vm_extension_install_uninstall
Impacted LISA Features:

VM Extension
Tested Azure Marketplace Images:

-Canonical 0001-com-ubuntu-server-focal 20_04-lts-gen2 latest

Test Results

Image	VM Size	Result
		PASSED / FAILED / SKIPPED

github-actions · 2026-04-23T07:11:48Z

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

	Count
✅ Passed	0
❌ Failed	0
⏭️ Skipped	1
Total	1

Test case details

Test Case	Status	Time (s)	Message
verify_vm_extension_install_uninstall (lisa_0_0)	⏭️ SKIPPED	8.377	skipped: Required runbook variable(s) are missing or empty: extension_publisher='', extension_type='', extension_version

Copilot

Pull request overview

This PR improves Azure VM extension version validation for pre-publish scenarios where an extension version may be provided as Major.Minor.Patch, while Azure deployment APIs only accept Major.Minor for typeHandlerVersion. It adds helper logic to normalize requested versions for installation and (optionally) validate the installed patch version via VM extension instance view.

Changes:

Added version parsing/normalization and installed-version lookup helpers to AzureExtension.
Updated verify_vm_extension_install_uninstall to install using normalized Major.Minor while validating installed Major.Minor.Patch when requested.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
lisa/sut_orchestrator/azure/features.py	Add helpers to normalize typeHandlerVersion input and retrieve installed extension version from instance view.
lisa/microsoft/testsuites/vm_extensions/generic_vm_extension.py	Use normalization for install and assert installed patch version when a patch version is requested.

github-actions · 2026-04-23T07:33:06Z

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

	Count
✅ Passed	0
❌ Failed	0
⏭️ Skipped	1
Total	1

Test case details

Test Case	Status	Time (s)	Message
verify_vm_extension_install_uninstall (lisa_0_0)	⏭️ SKIPPED	6.304	skipped: Required runbook variable(s) are missing or empty: extension_publisher='', extension_type='', extension_version

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

Copilot · 2026-04-30T05:50:46Z

+    def get_installed_type_handler_version(self, name: str) -> str:
+        extension_obj = self.get(name=name)
+


get_installed_type_handler_version() ultimately makes an Azure SDK call via self.get(). The Azure common module documents using global_credential_access_lock around SDK calls to avoid auth-file access conflicts in parallel runs. Consider wrapping the self.get(...) call (or the underlying SDK call) with global_credential_access_lock to align with that pattern and reduce intermittent failures under parallel execution.

Copilot · 2026-04-30T05:50:46Z

+                f"'{version}', actual '{installed_version}'. Please double confirm if "
+                f"the Azure platform supports {installed_version}"


The assertion message reads a bit unclear/grammatically off ("double confirm") and doesn’t give a concrete next step beyond "platform supports". Consider rephrasing to something like: verify which patch version Azure delivers for the requested Major.Minor and check the extension’s published versions in this region; include the expected/actual values once.

Suggested change

f"'{version}', actual '{installed_version}'. Please double confirm if "

f"the Azure platform supports {installed_version}"

f"'{version}', actual '{installed_version}'. Verify which patch "

f"version Azure delivers for the requested major.minor version and "

f"check whether this extension version is published in the current "

f"region."

github-actions · 2026-04-30T05:50:52Z

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

	Count
✅ Passed	0
❌ Failed	0
⏭️ Skipped	1
Total	1

Test case details

Test Case	Status	Time (s)	Message
verify_vm_extension_install_uninstall (lisa_0_0)	⏭️ SKIPPED	6.214	skipped: Required runbook variable(s) are missing or empty: extension_publisher='', extension_type='', extension_version

github-actions · 2026-04-30T06:00:42Z

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

	Count
✅ Passed	0
❌ Failed	0
⏭️ Skipped	1
Total	1

Test case details

Test Case	Status	Time (s)	Message
verify_vm_extension_install_uninstall (lisa_0_0)	⏭️ SKIPPED	7.734	skipped: Required runbook variable(s) are missing or empty: extension_publisher='', extension_type='', extension_version

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

github-actions · 2026-04-30T06:11:00Z

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

	Count
✅ Passed	0
❌ Failed	0
⏭️ Skipped	1
Total	1

Test case details

Test Case	Status	Time (s)	Message
verify_vm_extension_install_uninstall (lisa_0_0)	⏭️ SKIPPED	8.195	skipped: Required runbook variable(s) are missing or empty: extension_publisher='', extension_type='', extension_version

github-actions · 2026-04-30T06:26:07Z

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

	Count
✅ Passed	0
❌ Failed	0
⏭️ Skipped	1
Total	1

Test case details

Test Case	Status	Time (s)	Message
verify_vm_extension_install_uninstall (lisa_0_0)	⏭️ SKIPPED	8.604	skipped: Required runbook variable(s) are missing or empty: extension_publisher='', extension_type='', extension_version

Copilot AI review requested due to automatic review settings April 23, 2026 07:06

lubaihua33 requested review from LiliDeng and johnsongeorge-w as code owners April 23, 2026 07:06

Copilot started reviewing on behalf of lubaihua33 April 23, 2026 07:07 View session

Copilot AI reviewed Apr 23, 2026

View reviewed changes

lubaihua33 force-pushed the baihua/fix0423 branch from c4dfcaf to 74bff72 Compare April 23, 2026 07:29

Copilot AI review requested due to automatic review settings April 30, 2026 05:46

lubaihua33 force-pushed the baihua/fix0423 branch from 74bff72 to 97ada11 Compare April 30, 2026 05:46

Copilot started reviewing on behalf of lubaihua33 April 30, 2026 05:47 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

lubaihua33 force-pushed the baihua/fix0423 branch from 97ada11 to 691c431 Compare April 30, 2026 05:56

Copilot AI review requested due to automatic review settings April 30, 2026 06:03

lubaihua33 force-pushed the baihua/fix0423 branch from 691c431 to 6624f91 Compare April 30, 2026 06:03

Copilot started reviewing on behalf of lubaihua33 April 30, 2026 06:04 View session

lubaihua33 force-pushed the baihua/fix0423 branch from 6624f91 to be44310 Compare April 30, 2026 06:06

Copilot AI reviewed Apr 30, 2026

View reviewed changes

Check the installed verson when extension has patch version

936d7bd

lubaihua33 force-pushed the baihua/fix0423 branch from be44310 to 936d7bd Compare April 30, 2026 06:21

		def get_installed_type_handler_version(self, name: str) -> str:
		extension_obj = self.get(name=name)

		f"'{version}', actual '{installed_version}'. Please double confirm if "
		f"the Azure platform supports {installed_version}"

-                f"'{version}', actual '{installed_version}'. Please double confirm if "
-                f"the Azure platform supports {installed_version}"
+                f"'{version}', actual '{installed_version}'. Verify which patch "
+                f"version Azure delivers for the requested major.minor version and "
+                f"check whether this extension version is published in the current "
+                f"region."

Conversation

lubaihua33 commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Related Issue

Type of Change

Checklist

Test Validation

Test Results

Uh oh!

github-actions Bot commented Apr 23, 2026

✅ AI Test Selection — PASSED

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Apr 23, 2026

✅ AI Test Selection — PASSED

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI Apr 30, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 30, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Apr 30, 2026

✅ AI Test Selection — PASSED

Uh oh!

github-actions Bot commented Apr 30, 2026

✅ AI Test Selection — PASSED

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

github-actions Bot commented Apr 30, 2026

✅ AI Test Selection — PASSED

Uh oh!

github-actions Bot commented Apr 30, 2026

✅ AI Test Selection — PASSED

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

lubaihua33 commented Apr 23, 2026 •

edited

Loading