azurerm_monitor_activity_log_alert - support for service_health

[HappyTobi]

Hi all,

just a small pr to add more fine grained settings for "ServiceHealth" alerts.

I changed:

• Add new Rest API for Log Alerts
• Add Documentation for fine grained settings
• Add new ServiceHealth Settings
• Add Tests

New TF style to setup ServiceHealth

resource "azurerm_monitor_activity_log_alert" "main" {
  name                = "example-activitylogalert"
  resource_group_name = azurerm_resource_group.main.name
  scopes              = [azurerm_resource_group.main.id]
  description         = "This alert will monitor a specific storage account updates."

  criteria {
    category = "ServiceHealth"    
    servicehealth {
      events   = ["Incident", "Maintenance"]
      regions  = ["Global", "West Europe"]
      services = ["App Service (Linux)", "Network Infrastructure"]
    }
  }

  action {
    action_group_id = azurerm_monitor_action_group.main.id
  }
}

Added requested feature from issues:
#2996, #7392

Regards,
Tobi

[ArcturusZhang]

Hey @HappyTobi thanks for this PR!
I just left some minor comments, could you please take a look? Thanks

[HappyTobi]

Hi @ArcturusZhang, @tombuildsstuff

I updated the code and the documentation.

Think it's fine now.

Regards,

[jdheyburn]

This is supposed to be service_health.

[HappyTobi]

I will update the docs

[skyflash711]

Are these changes deployed ? if it is yes then how can I use this fine granular piece of code ? Because right now it is giving error "Blocks of type "servicehealth" are not expected here."

[HappyTobi]

@skyflash711 no it's an PR er have to wait till it's reviewed and merged.

After that, a comment from the bot will be added with the version where feature is part of.

[ClaudiaBaur]

Is there anything open why this PR cannot be merged? Just asking as we are waiting to consume it... Thanks!

[mbfrahry]

Hey @HappyTobi! Thanks for opening up this PR. It's on the right track but there are a few things I've highlighted below that will need resolved before we can merge

[mbfrahry]

LGTM! Thanks for this @HappyTobi

[ghost]

This has been released in version 2.56.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.56.0"
}
# ... other configuration ...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!