[BugFix] Fix "monitoring" related issues in main.bicep (#991)

pamelafox · web-flow · commit f6e8729a7215 · 2023-11-24T20:32:55.000-08:00
* Change module location

* Make dashboard conditional
diff --git a/infra/main.bicep b/infra/main.bicep
@@ -110,7 +110,7 @@ resource storageResourceGroup 'Microsoft.Resources/resourceGroups@2021-04-01' ex
 }
 
 // Monitor application with Azure Monitor
-module monitoring './core/monitor/monitoring.bicep' = if (useApplicationInsights) {
+module monitoring 'core/monitor/monitoring.bicep' = if (useApplicationInsights) {
   name: 'monitoring'
   scope: resourceGroup
   params: {
@@ -122,7 +122,7 @@ module monitoring './core/monitor/monitoring.bicep' = if (useApplicationInsights
 }
 
 
-module applicationInsightsDashboard 'backend-dashboard.bicep' = {
+module applicationInsightsDashboard 'backend-dashboard.bicep' = if (useApplicationInsights) {
   name: 'application-insights-dashboard'
   scope: resourceGroup
   params: {

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ resource storageResourceGroup 'Microsoft.Resources/resourceGroups@2021-04-01' ex`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`// Monitor application with Azure Monitor`
`113`		`-module monitoring './core/monitor/monitoring.bicep' = if (useApplicationInsights) {`
	`113`	`+module monitoring 'core/monitor/monitoring.bicep' = if (useApplicationInsights) {`
`114`	`114`	`name: 'monitoring'`
`115`	`115`	`scope: resourceGroup`
`116`	`116`	`params: {`
`@@ -122,7 +122,7 @@ module monitoring './core/monitor/monitoring.bicep' = if (useApplicationInsights`
`122`	`122`	`}`
`123`	`123`
`124`	`124`
`125`		`-module applicationInsightsDashboard 'backend-dashboard.bicep' = {`
	`125`	`+module applicationInsightsDashboard 'backend-dashboard.bicep' = if (useApplicationInsights) {`
`126`	`126`	`name: 'application-insights-dashboard'`
`127`	`127`	`scope: resourceGroup`
`128`	`128`	`params: {`