Fix AMBA Service Health deployment conditions #1876
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview/Summary
Resolves deployment issues related to Service Health alerts. Previously, Service Health was not deployed when selected unless Azure Monitor Baseline Alerts were also selected.
This PR fixes/adds/changes/removes
enableMonitorBaselines
and/orenableServiceHealth
is set to YesenableAMBAConnectivity
(and other initiative assignments) to No whenenableMonitorBaselines
is No andenableServiceHealth
is Yes.ALZWebhookServiceUri
to null ifambaAgServiceHook
is empty; otherwise, it uses the value ofambaAgServiceHook
. Single platform was missed in Fixes empty parameters for amba service hook and email contact #1818ALZMonitorActionGroupEmail
to null ifambaAgEmailContact
is empty; otherwise, it uses the value ofambaAgEmailContact
. Single platform was missed in Fixes empty parameters for amba service hook and email contact #1818Breaking Changes
None
Testing Evidence
Test 1: Dedicated platform
Test 2: Dedicated platform
Test 3: Single platform
Test 4: Single platform
Testing URLs
Azure Public
As part of this Pull Request I have
main
branch/docs/wiki/whats-new.md
)