Skip to content

[release/9.4] Fix Azure AI Foundry resource name#11038

Merged
sebastienros merged 5 commits into
release/9.4from
backport/pr-10934-to-release/9.4
Aug 22, 2025
Merged

[release/9.4] Fix Azure AI Foundry resource name#11038
sebastienros merged 5 commits into
release/9.4from
backport/pr-10934-to-release/9.4

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 21, 2025

Copy link
Copy Markdown
Contributor

Backport of #10934 to release/9.4

/cc @sebastienros

Customer Impact

Users can't resolve the model if the resource and model names don't match.

Testing

Unit testing and manual testing.
Deployed from the dashboard and using azd.

Risk

Low.

Regression?

No, new feature in 9.4

Comment thread src/Aspire.Hosting.Azure.AIFoundry/AzureAIFoundryDeploymentResource.cs Outdated
@sebastienros sebastienros added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Aug 21, 2025
@danmoseley danmoseley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 21, 2025

@eerhardt eerhardt 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.

LGTM. Thanks!

@eerhardt

Copy link
Copy Markdown
Member

Make sure to forward port this change back to main.

whytem pushed a commit to whytem/aspire that referenced this pull request Sep 16, 2025
This was referenced Sep 17, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants