Skip to content

Commit d7d7b52

Browse files
committed
Add warning message for Service Health alerts configuration and update condition for built-in policy assignment
1 parent adc72d9 commit d7d7b52

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

eslzArm/eslz-portal.json

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1005,6 +1005,16 @@
10051005
},
10061006
"visible": true
10071007
},
1008+
{
1009+
"name": "multipleServiceHealthWarning",
1010+
"type": "Microsoft.Common.InfoBox",
1011+
"visible": true,
1012+
"options": {
1013+
"text": "There are 2 ways to enable Service Health alerts via Azure Policy in ALZ. One is via the built-in policy (recommended) and the other is via Azure Monitor Baseline Alerts (AMBA). Both are valid and achieve the same outcome of enabling Service Health on all Azure Subscriptions in-scope of the policy assignment at the intermediate root Management Group, e.g. Contoso. You should pick only one way to enable Service Health alerts and not both. We recommend using the built-in policy as it is simpler to use and does not require any additional configuration. To deploy via AMBA you must select 'No' to the built-in policy, above, and 'Yes' in the AMBA section below.",
1014+
"uri": "https://learn.microsoft.com/azure/service-health/service-health-alert-deploy-policy",
1015+
"style": "Warning"
1016+
}
1017+
},
10081018
{
10091019
"name": "esAmbaTitle",
10101020
"type": "Microsoft.Common.Section",
@@ -1052,7 +1062,7 @@
10521062
}
10531063
]
10541064
},
1055-
"visible": true
1065+
"visible": "[equals(steps('monitor').enableServiceHealthBuiltIn, 'No')]"
10561066
},
10571067
{
10581068
"name": "enableMonitorBaselines",

eslzArm/eslzArm.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2888,7 +2888,7 @@
28882888
},
28892889
{
28902890
// Assigning Service Health built-in policy to intermediate root management group if condition is true
2891-
"condition": "[and(or(not(empty(parameters('singlePlatformSubscriptionId'))), not(empty(parameters('managementSubscriptionId')))), equals(parameters('enableServiceHealthBuiltIn'), 'Yes'))]",
2891+
"condition": "[and(or(not(empty(parameters('singlePlatformSubscriptionId'))), not(empty(parameters('managementSubscriptionId')))), equals(parameters('enableServiceHealthBuiltIn'), 'Yes'), equals(parameters('enableServiceHealth'), 'No'))]",
28922892
"type": "Microsoft.Resources/deployments",
28932893
"apiVersion": "2020-10-01",
28942894
"name": "[variables('deploymentNames').monitorServiceHealthBuiltInDeploymentName]",

0 commit comments

Comments
 (0)