Skip to content

Commit eb06e0d

Browse files
fix: update dependency condition for log analytics workspace in Bicep module
1 parent d723ef3 commit eb06e0d

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

infra/main.bicep

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,7 @@ module virtualNetwork 'modules/virtualNetwork.bicep' = if (enablePrivateNetworki
380380
resourceSuffix: solutionSuffix
381381
deployBastionAndJumpbox: deployBastionAndJumpbox
382382
}
383-
dependsOn: enableMonitoring ? [logAnalyticsWorkspace] : []
383+
dependsOn: (enableMonitoring && !useExistingLogAnalytics) ? [logAnalyticsWorkspace] : []
384384
}
385385

386386
// ========== Private DNS Zones ========== //

scripts/post_deploy.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,9 @@ def get_values_from_az_deployment(resource_group: str) -> Optional[Dict[str, str
161161

162162
outputs = json.loads(result.stdout)
163163

164-
# Build a case-insensitive lookup map for output keys
165-
# ARM/Bicep auto-camelCases output names (e.g. APP_SERVICE_NAME -> apP_SERVICE_NAME)
164+
# Build a case-insensitive lookup map for output keys.
165+
# ARM/Bicep preserve output names as authored in the template; this map lets
166+
# us perform lookups without worrying about casing in our code.
166167
outputs_lower = {k.lower(): v for k, v in outputs.items()}
167168

168169
# Extract values from deployment outputs

0 commit comments

Comments
 (0)