Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[release/13.0] Fix subscription ID not being disabled on Azure provisioning dialog - #12913

#12913
Merged
joperezr merged 2 commits into
release/13.0microsoft/aspire:release/13.0from
backport/pr-12888-to-release/13.0microsoft/aspire:backport/pr-12888-to-release/13.0Copy head branch name to clipboard
Nov 21, 2025
Merged

[release/13.0] Fix subscription ID not being disabled on Azure provisioning dialog#12913
joperezr merged 2 commits into
release/13.0microsoft/aspire:release/13.0from
backport/pr-12888-to-release/13.0microsoft/aspire:backport/pr-12888-to-release/13.0Copy head branch name to clipboard

Conversation

@github-actions

@github-actions github-actions Bot commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

Backport of #12888 to release/13.0

/cc @JamesNK

Customer Impact

There are some bugs on the Azure provisioning dialog:

  • If the customer has Azure:SubscriptionId in their config then the Azure provisioning dialog is broken. Subscription locations are never loaded and can't be selected. Major impact.
  • If the customer doesn't have Azure:SubscriptionId in their config then the "Subscription Id" input is enabled before values are loaded into it by a tenant being selected. Minor impact.

Testing

Manual testing

Risk

Low

Regression?

Yes, this regressed in 13 when tenant selection was added to the dialog.

@github-actions

Copy link
Copy Markdown
Contributor Author

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 12913

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 12913"

@JamesNK JamesNK added the Servicing-consider Issue for next servicing release review label Nov 11, 2025
@JamesNK
JamesNK requested a review from davidfowl November 11, 2025 23:58
@JamesNK JamesNK added the area-integrations Issues pertaining to Aspire Integrations packages label Nov 11, 2025

@joperezr joperezr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regression and scoped fix. Approved. Should we add a test here too?

@joperezr joperezr added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 20, 2025
@joperezr
joperezr merged commit 42ccc16 into release/13.0 Nov 21, 2025
296 checks passed
@joperezr
joperezr deleted the backport/pr-12888-to-release/13.0 branch November 21, 2025 16:59
@joperezr

Copy link
Copy Markdown
Member

Discussed yesterday. Tests will be added in main.

This was referenced Dec 19, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-integrations Issues pertaining to Aspire Integrations packages Re-opened Github-Action PR Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.