Skip to content

fix: Add missing Svc Health Alert policy param overrides for assignment#1145

Merged
oZakari merged 2 commits into
mainfrom
fix-policy-assignment-overrides
Mar 17, 2026
Merged

fix: Add missing Svc Health Alert policy param overrides for assignment#1145
oZakari merged 2 commits into
mainfrom
fix-policy-assignment-overrides

Conversation

@oZakari
Copy link
Copy Markdown
Contributor

@oZakari oZakari commented Mar 17, 2026

Overview/Summary

This pull request enhances the configuration and deployment of the "Deploy-SvcHealth-BuiltIn" policy assignment by introducing new parameters for service health alert resource group and action group resources. These changes provide greater flexibility and customization for service health alerting in Azure environments.

Related Issues/Work Items

Fixes Azure/Azure-Landing-Zones#4086

This PR fixes/adds/changes/removes

Service Health Alert Configuration Enhancements:

  • Added new parameters parServiceHealthAlertResourceGroupName and parServiceHealthAlertActionGroupResources to the alzDefaultPolicyAssignments.bicep module, allowing users to specify the resource group and action group settings for service health alert rules.
  • Updated the parameters file (alzDefaultPolicyAssignments.parameters.all.json) to include default values for the new service health alert parameters, ensuring consistent deployment configuration.

Policy Assignment Improvements:

  • Modified the "Deploy-SvcHealth-BuiltIn" policy assignment module to pass the new parameters (resourceGroupName, actionGroupResources, and resourceGroupLocation) as overrides, enabling the policy to use the specified resource group and action group configuration.
  • Updated the policy assignment template (policy_assignment_es_deploy_svchealth_builtin.tmpl.json) to define the new parameters and provide default values, supporting the enhanced configuration options.

Breaking Changes

None

Testing Evidence

Replace this with any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

@oZakari oZakari added the Type: Bug 🪲 Something isn't working label Mar 17, 2026
@oZakari oZakari merged commit 7a96913 into main Mar 17, 2026
14 checks passed
@oZakari oZakari deleted the fix-policy-assignment-overrides branch March 17, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug 🪲 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Parameter in Deploy-SvcHealth-BuiltIn Assignment needs to have `${default_location} placeholders overriden

2 participants