Skip to content
This repository has been archived by the owner on Oct 5, 2023. It is now read-only.

fix rg location, name and tags #215

Merged
merged 1 commit into from
Jul 25, 2023
Merged

fix rg location, name and tags #215

merged 1 commit into from
Jul 25, 2023

Conversation

arjenhuitema
Copy link
Collaborator

Files changed:

src\resources\Microsoft.Authorization\policySetDefinitions\ALZ-MonitorLandingZone.json

Changes:

  1. Removed VMInsightsAlertResourceGroupName parameter
  2. Removed VMInsightsAlertResourceGroupLocation parameter
  3. Removed VMInsightsAlertResourceGroupTags parameter
  4. For all VM alerts affected: the value of alertResourceGroupName, alertResourceGroupTags, alertResourceGroupLocation changed to the value of existing parameters ALZMonitorResourceGroupName, ALZMonitorResourceGroupTags, ALZMonitorResourceGroupLocation
  5. Added alertResourceGroupName, alertResourceGroupTags and alertResourceGroupLocation parameters to Deploy_VM_HeartBeat_Alert.

@arjenhuitema arjenhuitema linked an issue Jul 25, 2023 that may be closed by this pull request
Log search rules created in AlzMonitoring-rg, other rules in rg-alz-monitor #211
Closed
paulgrimley
paulgrimley approved these changes Jul 25, 2023
View reviewed changes
Copy link
Contributor

@paulgrimley paulgrimley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@paulgrimley paulgrimley reopened this Jul 25, 2023
@paulgrimley paulgrimley merged commit 1b73b84 into Azure:main Jul 25, 2023
1 check passed
@arjenhuitema arjenhuitema deleted the issue211 branch August 1, 2023 07:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@paulgrimley paulgrimley paulgrimley approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Log search rules created in AlzMonitoring-rg, other rules in rg-alz-monitor
2 participants
@arjenhuitema @paulgrimley