Azure / azure-monitor-baseline-alerts

Azure Monitor Baseline Alerts
MIT License
163 stars 239 forks source link

[Bug]: Wrong variables for Enable Common Alert Schema in Policy Definition `Deploy_ServiceHealth_ActionGroups` and `Deploy_Suppression_AlertProcessing_Rule` #345

Closed haflidif closed 1 month ago

haflidif commented 1 month ago

Check for previous/existing GitHub issues

Description

Hey.

For this Configuration in Notification Assets on Policy Definition: Deploy_ServiceHealth_ActionGroups and Deploy_Suppression_AlertProcessing_Rule

image

There is a wrong parameter/variable name on three deployment types within services/AlertsManagement/actionRules/Deploy-AlertProcessingRule-Deploy.json

Will open up a PR shortly to these two PolicyDefintions to fix this.

Brunoga-MS commented 1 month ago

Hello @haflidif , thanks for your feedback. We are aware of this bug and have already captured in our backlog. We are working to get it solved soon and will keep you posted on the progress.

Thanks, Bruno.

haflidif commented 1 month ago

Hello @haflidif , thanks for your feedback. We are aware of this bug and have already captured in our backlog. We are working to get it solved soon and will keep you posted on the progress.

Thanks, Bruno.

Thanks @Brunoga-MS, sounds good, otherwise I have a open PR that fixes this, but you maybe want to have it within some kind of release maybe ?

Brunoga-MS commented 1 month ago

Having it as part of the next release is exactly the idea and the path that normally follow. However, we will release the fix most probably before the release so you can just deploy the main version over the current release version to benefit from the fix.