update tests to use python 3.13 #9028

bavneetsingh16 · 2025-07-30T22:09:59Z

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

Related command

General Guidelines

Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
My extension version conforms to the Extension version schema

For new extensions:

My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

azure-client-tools-bot-prd · 2025-07-30T22:10:05Z

Validation for Breaking Change Starting...

Thanks for your contribution!

azure-client-tools-bot-prd · 2025-07-30T22:10:08Z

Hi @bavneetsingh16,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

yonzhan · 2025-07-30T22:10:20Z

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

github-actions · 2025-07-30T22:10:21Z

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

Copilot

Pull Request Overview

This PR introduces PowerShell-based integration tests for Azure CLI K8s extensions and configurations. The tests utilize the Pester framework and are designed to validate Azure Arc K8s extension and configuration functionality.

Key changes:

Adds comprehensive test infrastructure with helper functions for extension and configuration testing
Provides test suites for various Azure extensions (Azure Monitor, Azure ML, Azure Defender, Azure Policy)
Creates test configurations for Flux Git/Blob storage scenarios
Establishes CI/CD pipeline templates for automated testing

Reviewed Changes

Copilot reviewed 29 out of 33 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
testing/test/helper/*.ps1	Helper functions for extension data retrieval and common test operations
testing/test/extensions/*/.ps1	Test suites for Azure extensions using Pester framework
testing/test/configurations/*.ps1	Test suites for Flux configuration scenarios with various storage backends
testing/pipeline/*.yml	Azure DevOps pipeline configuration for automated testing
testing/*.ps1	Main test orchestration and environment setup scripts
testing/docs/*.md	Documentation for test authoring and setup procedures

Comments suppressed due to low confidence (4)

testing/pipeline/k8s-custom-pipelines.yml:22

Python 3.13 may not be available or supported in Azure DevOps hosted agents. Consider using a stable Python version like 3.11 or 3.12.

  jobs:

testing/pipeline/templates/run-test.yml:22

Python 3.13 may not be available or supported in Azure DevOps hosted agents. Consider using a stable Python version like 3.11 or 3.12.

    displayName: 'Use Python 3.13'

testing/pipeline/k8s-custom-pipelines.yml:44

Python 3.13 may not be available or supported in Azure DevOps hosted agents. Consider using a stable Python version like 3.11 or 3.12.

      displayName: 'Use Python 3.13'

testing/pipeline/k8s-custom-pipelines.yml:81

Python 3.13 may not be available or supported in Azure DevOps hosted agents. Consider using a stable Python version like 3.11 or 3.12.

      displayName: 'Use Python 3.13'

Copilot · 2025-07-30T22:11:13Z

testing/test/configurations/Helper.ps1

+    )
+
+    $output = kubectl get gitconfigs -A -o json | ConvertFrom-Json
+    return $output.items | Where-Object { $_.metadata.name -eq $configurationName }


The variable $configurationName is not defined in this function. It should be $configName to match the parameter name.

Suggested change

return $output.items | Where-Object { $_.metadata.name -eq $configurationName }

return $output.items | Where-Object { $_.metadata.name -eq $configName }

Copilot · 2025-07-30T22:11:13Z

testing/test/configurations/Helper.ps1

+    )
+
+    $output = kubectl get fc -A -o json | ConvertFrom-Json
+    return $output.items | Where-Object { $_.metadata.name -eq $configurationName }


The variable $configurationName is not defined in this function. It should be $configName to match the parameter name.

Suggested change

return $output.items | Where-Object { $_.metadata.name -eq $configurationName }

return $output.items | Where-Object { $_.metadata.name -eq $configName }

Copilot · 2025-07-30T22:11:14Z

testing/test/configurations/Flux.PrivateKey.Tests.ps1

+        {
+            $readyConfigs = 0
+            foreach($configData in $CONFIG_ARR) {
+                $output = az k8s-configuration flux show -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters -n $CONFIGdATA.Item1


There is a typo in the variable name $CONFIGdATA.Item1. It should be $configData.Item1 with lowercase 'd' to match the variable defined in the loop.

Suggested change

$output = az k8s-configuration flux show -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters -n $CONFIGdATA.Item1

$output = az k8s-configuration flux show -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters -n $configData.Item1

Copilot · 2025-07-30T22:11:14Z

testing/test/extensions/public/AzureMonitor.Tests.ps1

+
+    It "Performs another list after the delete" {
+        $output = az $Env:K8sExtensionName list -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters
+        $extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }


The filter condition should use $extensionType instead of $extensionName to correctly filter by extension type.

Suggested change

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionType }

Copilot · 2025-07-30T22:11:15Z

testing/test/extensions/public/AzureDefender.Tests.ps1

+
+    It "Performs another list after the delete" {
+        $output = az $Env:K8sExtensionName list -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters
+        $extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }


The filter condition should use $extensionType instead of $extensionName to correctly filter by extension type.

Suggested change

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionType }

Copilot · 2025-07-30T22:11:15Z

testing/test/extensions/private-preview/AzurePolicy.Tests.ps1

+
+    It "Performs another list after the delete" {
+        $output = az $Env:K8sExtensionName list -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters
+        $extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }


The filter condition should use $extensionType instead of $extensionName to correctly filter by extension type.

Suggested change

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionType }

Copilot · 2025-07-30T22:11:15Z

testing/test/extensions/public/AzureMLKubernetes.Tests.ps1

+
+    It "Performs another list after the delete" {
+        $output = az k8s-extension list --cluster-name $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters
+        $extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }


The filter condition should use $extensionType instead of $extensionName to correctly filter by extension type.

Suggested change

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionName }

$extensionExists = $output | ConvertFrom-Json | Where-Object { $_.extensionType -eq $extensionType }

github-actions · 2025-07-30T22:11:18Z

For more info about extension versioning, please refer to Extension version schema

Bavneet Singh and others added 5 commits May 28, 2025 12:13

add pester tests for k8s-configuration

4c06e4b

fix CI test cases

08cb376

Merge branch 'Azure:main' into main

dc5beeb

update tests to use python 3.13

189b8a1

update kind version

d12b034

Copilot AI review requested due to automatic review settings July 30, 2025 22:10

azure-client-tools-bot-prd bot added this to the September 2025 (2025-09-02) milestone Jul 30, 2025

bavneetsingh16 closed this Jul 30, 2025

Copilot AI reviewed Jul 30, 2025

View reviewed changes

bavneetsingh16 deleted the python313_test branch July 31, 2025 21:24

	return $output.items \| Where-Object { $_.metadata.name -eq $configurationName }
	return $output.items \| Where-Object { $_.metadata.name -eq $configName }

	$output = az k8s-configuration flux show -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters -n $CONFIGdATA.Item1
	$output = az k8s-configuration flux show -c $ENVCONFIG.arcClusterName -g $ENVCONFIG.resourceGroup --cluster-type connectedClusters -n $configData.Item1

	$extensionExists = $output \| ConvertFrom-Json \| Where-Object { $_.extensionType -eq $extensionName }
	$extensionExists = $output \| ConvertFrom-Json \| Where-Object { $_.extensionType -eq $extensionType }

update tests to use python 3.13 #9028

update tests to use python 3.13 #9028

Uh oh!

Conversation

bavneetsingh16 commented Jul 30, 2025

Related command

General Guidelines

About Extension Publish

Uh oh!

azure-client-tools-bot-prd bot commented Jul 30, 2025

Uh oh!

azure-client-tools-bot-prd bot commented Jul 30, 2025

Uh oh!

yonzhan commented Jul 30, 2025

Uh oh!

github-actions bot commented Jul 30, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jul 30, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants