Add Azure portal links to deploy summaries#16142
Conversation
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 16142Or
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 16142" |
Surface Azure Portal links for resource groups, deployments, Container Apps, and App Service resources in Azure deploy summaries. Keep portal URL generation in shared helpers and update Azure deployer tests to use GUID-based ARM resource IDs so portal link parsing can continue using ResourceIdentifier. Co-authored-by: Copilot <[email protected]>
d67561e to
e8f9014
Compare
|
Re-running the failed jobs in the CI workflow for this pull request because 1 job was identified as retry-safe transient failures in the CI run attempt.
|
|
🎬 CLI E2E Test Recordings — 69 recordings uploaded (commit View recordings
📹 Recordings uploaded automatically from CI run #24439151813 |
Documentation Check ✅No documentation PR is required for this change. Reason: This PR adds Azure Portal links to the deploy summary output (a UX improvement to existing CLI output). It introduces no new APIs, configuration options, CLI commands, or behavioral changes that users need to learn about. The existing [Azure Container Apps deployment docs]((aspire.dev/redacted) already cover portal navigation post-deployment. The improvement is self-evident from the CLI output itself.
|
Description
This follow-up fixes #16119 by making the Azure deployment summary easier to navigate after a publish.
AzurePortalUrlsand resource-specific link composition inContainerAppUrlsandAppSvcUrls.Azure.Core.ResourceIdentifier.Validation:
dotnet test tests/Aspire.Hosting.Azure.Tests/Aspire.Hosting.Azure.Tests.csproj -tl:off -- --filter-method "*.DeployAsync_WithContainerAppExternalEndpoint_IncludesPortalLinksInSummary" --filter-method "*.DeployAsync_WithAppServiceExternalEndpoint_IncludesPortalLinksInSummary" --filter-not-trait "quarantined=true" --filter-not-trait "outerloop=true"Fixes #16119
Checklist
<remarks />and<code />elements on your triple slash comments?aspire.devissue: