Skip to content

Commit 2aa15c0

Browse files
Merge pull request #116 from microsoft/exp-changes-sub
fix: To reuse Log Analytics across subscriptions
2 parents 52bcd7c + 2f1971d commit 2aa15c0

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

docs/CustomizingAzdParameters.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,5 @@ azd env set AZURE_ENV_MODEL_CAPACITY 30
2525

2626
Set the Log Analytics Workspace Id if you need to reuse the existing workspace which is already existing
2727
```shell
28-
azd env set AZURE_ENV_LOG_ANALYTICS_WORKSPACE_ID '<Existing Log Analytics Workspace Id>'
28+
azd env set AZURE_ENV_LOG_ANALYTICS_WORKSPACE_ID '/subscriptions/<subscription-id>/resourceGroups/<resource-group>/providers/Microsoft.OperationalInsights/workspaces/<workspace-name>'
2929
```

infra/deploy_ai_foundry.bicep

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ param aiServicesId string
1515
param existingLogAnalyticsWorkspaceId string = ''
1616

1717
var useExisting = !empty(existingLogAnalyticsWorkspaceId)
18+
var existingLawSubscription = useExisting ? split(existingLogAnalyticsWorkspaceId, '/')[2] : ''
1819
var existingLawResourceGroup = useExisting ? split(existingLogAnalyticsWorkspaceId, '/')[4] : ''
1920
var existingLawName = useExisting ? split(existingLogAnalyticsWorkspaceId, '/')[8] : ''
2021

@@ -41,7 +42,7 @@ resource keyVault 'Microsoft.KeyVault/vaults@2022-07-01' existing = {
4142

4243
resource existingLogAnalyticsWorkspace 'Microsoft.OperationalInsights/workspaces@2020-08-01' existing = if (useExisting) {
4344
name: existingLawName
44-
scope: resourceGroup(existingLawResourceGroup)
45+
scope: resourceGroup(existingLawSubscription, existingLawResourceGroup)
4546
}
4647

4748
resource logAnalytics 'Microsoft.OperationalInsights/workspaces@2023-09-01' = if (!useExisting) {

0 commit comments

Comments
 (0)