Skip to content

Commit c1862a0

Browse files
[release/9.4] Fix Azure AI Foundry resource name (#11038)
1 parent 9942803 commit c1862a0

File tree

4 files changed

+30
-9
lines changed

4 files changed

+30
-9
lines changed

playground/AzureAIFoundryEndToEnd/AzureAIFoundryEndToEnd.AppHost/foundry.module.bicep

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ resource foundry 'Microsoft.CognitiveServices/accounts@2024-10-01' = {
2222
}
2323

2424
resource chat 'Microsoft.CognitiveServices/accounts/deployments@2024-10-01' = {
25-
name: 'qwen2.5-0.5b'
25+
name: 'chat'
2626
properties: {
2727
model: {
2828
format: 'Microsoft'

src/Aspire.Hosting.Azure.AIFoundry/AzureAIFoundryDeploymentResource.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public class AzureAIFoundryDeploymentResource : Resource, IResourceWithParent<Az
2727
public AzureAIFoundryDeploymentResource(string name, string modelName, string modelVersion, string format, AzureAIFoundryResource parent)
2828
: base(name)
2929
{
30-
DeploymentName = modelName;
30+
DeploymentName = name;
3131
ModelName = modelName;
3232
ModelVersion = modelVersion;
3333
Format = format;

tests/Aspire.Hosting.Azure.Tests/AzureAIFoundryExtensionsTests.cs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ public void AddDeployment_ShouldAddDeploymentToResource()
3030
var resource = Assert.Single(builder.Resources.OfType<AzureAIFoundryResource>());
3131
var deployment = Assert.Single(resource.Deployments);
3232
Assert.Equal("deployment1", deployment.Name);
33+
Assert.Equal("deployment1", deployment.DeploymentName);
3334
Assert.Equal("gpt-4", deployment.ModelName);
3435
Assert.Equal("1.0", deployment.ModelVersion);
3536
Assert.Equal("OpenAI", deployment.Format);
@@ -112,17 +113,17 @@ public void RunAsFoundryLocal_DeploymentIsMarkedLocal()
112113
}
113114

114115
[Fact]
115-
public void RunAsFoundryLocal_DeploymentConnectionString_HasModelProperty()
116+
public async Task RunAsFoundryLocal_DeploymentConnectionString_HasModelProperty()
116117
{
117118
using var builder = TestDistributedApplicationBuilder.Create();
118119
var foundry = builder.AddAzureAIFoundry("myAIFoundry");
119120
var deployment = foundry.AddDeployment("deployment1", "gpt-4", "1.0", "OpenAI");
120121
foundry.RunAsFoundryLocal();
121122
var resource = Assert.Single(builder.Resources.OfType<AzureAIFoundryResource>());
122123
Assert.Single(resource.Deployments);
123-
var connectionString = deployment.Resource.ConnectionStringExpression.ValueExpression;
124-
Assert.Contains("Model=gpt-4", connectionString);
125-
Assert.Contains("DeploymentId=gpt-4", connectionString);
124+
var connectionString = await deployment.Resource.ConnectionStringExpression.GetValueAsync(default);
125+
Assert.Contains("Model=deployment1", connectionString);
126+
Assert.Contains("DeploymentId=deployment1", connectionString);
126127
Assert.Contains("Endpoint=", connectionString);
127128
Assert.Contains("Key=", connectionString);
128129
}
@@ -135,6 +136,7 @@ public async Task AddAzureAIFoundry_GeneratesValidBicep()
135136
var foundry = builder.AddAzureAIFoundry("foundry");
136137
var deployment1 = foundry.AddDeployment("deployment1", "gpt-4", "1.0", "OpenAI");
137138
var deployment2 = foundry.AddDeployment("deployment2", "Phi-4", "1.0", "Microsoft");
139+
var deployment3 = foundry.AddDeployment("my-model", "Phi-4", "1.0", "Microsoft");
138140

139141
using var app = builder.Build();
140142
var model = app.Services.GetRequiredService<DistributedApplicationModel>();

tests/Aspire.Hosting.Azure.Tests/Snapshots/AzureAIFoundryExtensionsTests.AddAzureAIFoundry_GeneratesValidBicep#00.verified.bicep

Lines changed: 22 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ resource foundry 'Microsoft.CognitiveServices/accounts@2024-10-01' = {
2222
}
2323

2424
resource deployment1 'Microsoft.CognitiveServices/accounts/deployments@2024-10-01' = {
25-
name: 'gpt-4'
25+
name: 'deployment1'
2626
properties: {
2727
model: {
2828
format: 'OpenAI'
@@ -38,7 +38,7 @@ resource deployment1 'Microsoft.CognitiveServices/accounts/deployments@2024-10-0
3838
}
3939

4040
resource deployment2 'Microsoft.CognitiveServices/accounts/deployments@2024-10-01' = {
41-
name: 'Phi-4'
41+
name: 'deployment2'
4242
properties: {
4343
model: {
4444
format: 'Microsoft'
@@ -56,8 +56,27 @@ resource deployment2 'Microsoft.CognitiveServices/accounts/deployments@2024-10-0
5656
]
5757
}
5858

59+
resource my_model 'Microsoft.CognitiveServices/accounts/deployments@2024-10-01' = {
60+
name: 'my-model'
61+
properties: {
62+
model: {
63+
format: 'Microsoft'
64+
name: 'Phi-4'
65+
version: '1.0'
66+
}
67+
}
68+
sku: {
69+
name: 'GlobalStandard'
70+
capacity: 1
71+
}
72+
parent: foundry
73+
dependsOn: [
74+
deployment2
75+
]
76+
}
77+
5978
output aiFoundryApiEndpoint string = foundry.properties.endpoints['AI Foundry API']
6079

6180
output endpoint string = foundry.properties.endpoint
6281

63-
output name string = foundry.name
82+
output name string = foundry.name

0 commit comments

Comments
 (0)