[CI] Add Azure CLI as Helix test dependency for basic tests #13420
+27
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Copilot
AI
changed the title
[WIP] Add az CLI as a dependency for AzDO Helix runs
Add Azure CLI as Helix test dependency for basic tests
Dec 9, 2025
radical
reviewed
Dec 9, 2025
tests/helix/send-to-helix-inner.proj
Outdated
|
|
||
| <!-- Azure CLI URL for Helix runs - Windows x64 zip version --> | ||
| <_AzureCliUrl Condition="'$(OS)' == 'Windows_NT'">https://aka.ms/installazurecliwindowszipx64</_AzureCliUrl> | ||
| <!-- Note: Azure CLI for non-Windows is not currently configured as the aka.ms link is Windows-specific --> |
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 13420Or
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 13420" |
Member
|
/azp run aspire-tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Member
|
/azp run aspire-tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Member
|
@copilot Add command to the helix targets to:
|
Contributor
Author
Member
|
/azp run aspire-tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
``` System.InvalidOperationException: Step 'provision-env' failed: Exception of type 'Aspire.Hosting.Azure.AzureCliNotOnPathException' was thrown. | ---> Aspire.Hosting.Azure.AzureCliNotOnPathException: Exception of type 'Aspire.Hosting.Azure.AzureCliNotOnPathException' was thrown. | at Aspire.Hosting.Azure.Provisioning.Internal.BicepCliCompiler.CompileBicepToArmAsync(String bicepFilePath, CancellationToken cancellationToken) in /_/src/Aspire.Hosting.Azure/Provisioning/Internal/BicepCompiler.cs:line 41 | at Aspire.Hosting.Azure.Provisioning.BicepProvisioner.GetOrCreateResourceAsync(AzureBicepResource resource, ProvisioningContext context, CancellationToken cancellationToken) in /_/src/Aspire.Hosting.Azure/Provisioning/Provisioners/BicepProvisioner.cs:line 159 | at Aspire.Hosting.Azure.AzureBicepResource.ProvisionAzureBicepResourceAsync(PipelineStepContext context, AzureBicepResource resource) in /_/src/Aspire.Hosting.Azure/AzureBicepResource.cs:line 338 | at Aspire.Hosting.Azure.AzureBicepResource.ProvisionAzureBicepResourceAsync(PipelineStepContext context, AzureBicepResource resource) in /_/src/Aspire.Hosting.Azure/AzureBicepResource.cs:line 361 | at Aspire.Hosting.Azure.AzureBicepResource.ProvisionAzureBicepResourceAsync(PipelineStepContext context, AzureBicepResource resource) in /_/src/Aspire.Hosting.Azure/AzureBicepResource.cs:line 361 | at Aspire.Hosting.Azure.AzureBicepResource.<>c__DisplayClass0_0.<<-ctor>b__2>d.MoveNext() in /_/src/Aspire.Hosting.Azure/AzureBicepResource.cs:line 53 | --- End of stack trace from previous location --- | at Aspire.Hosting.Pipelines.DistributedApplicationPipeline.ExecuteStepAsync(PipelineStep step, PipelineStepContext stepContext) in /_/src/Aspire.Hosting/Pipelines/DistributedApplicationPipeline.cs:line 760 | --- End of inner exception stack trace --- | at Aspire.Hosting.Pipelines.DistributedApplicationPipeline.ExecuteStepAsync(PipelineStep step, PipelineStepContext stepContext) in /_/src/Aspire.Hosting/Pipelines/DistributedApplicationPipeline.cs:line 765 | at Aspire.Hosting.Pipelines.DistributedApplicationPipeline.<>c__DisplayClass18_0.<<ExecuteStepsAsTaskDag>g__ExecuteStepWithDependencies|0>d.MoveNext() in /_/src/Aspire.Hosting/Pipelines/DistributedApplicationPipeline.cs:line 576 ```
radical
force-pushed
the
copilot/add-az-cli-dependency
branch
from
3c75681 to
81efe83
Compare
radical
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds Azure CLI as a dependency for Helix basic test runs, following the established pattern for Azure Functions CLI. The implementation downloads the Azure CLI zip from an aka.ms redirect URL, extracts it to az-cli, and adds its bin directory to the PATH for test execution. Currently configured for Windows only with placeholders for future non-Windows support.
Key Changes
- Introduced
NeedsAzCliproperty to conditionally enable Azure CLI installation - Added download and extraction target that handles the aka.ms redirect URL lacking a file extension
- Configured PATH updates and verification commands for Helix environments
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tests/helix/send-to-helix-inner.proj | Adds _AzureCliWindowsUrl property, creates DownloadAzureCliForHelix target to download/extract Azure CLI, configures HelixCorrelationPayload and HelixPreCommand items for PATH setup and verification |
| tests/helix/send-to-helix-basictests.targets | Enables Azure CLI for basic tests by setting NeedsAzCli=true |
Co-authored-by: Copilot <[email protected]>
radical
reviewed
Dec 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Adds Azure CLI to Helix test runs following the existing pattern for Azure Functions CLI. The CLI is downloaded from the aka.ms redirect, extracted to
az-cli, and added to PATH for test execution.Changes
tests/helix/send-to-helix-inner.proj:_AzureCliUrlproperty pointing tohttps://aka.ms/installazurecliwindowszipx64NeedsAzCliconditional ItemGroup with:HelixCorrelationPayloadfor downloading/extracting toaz-clidestinationHelixPreCommandentries prependingaz-cli/binto PATH (Windows:%HELIX_CORRELATION_PAYLOAD%\az-cli\bin, non-Windows:$HELIX_CORRELATION_PAYLOAD/az-cli/bin)diron Windows,ls -laon Linux) and test tool accessibility (az --version)tests/helix/send-to-helix-basictests.targets:NeedsAzCli=trueto enable for basic test runsNotes
NeedsAzureFunctionsCliimplementationFixes these failures on azdo:
Checklist
Original prompt
This pull request was created as a result of the following prompt from Copilot chat.
💡 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.