laudibert
Repos
10
Followers
2
Following
4

Events

[provider] Support monitor `notification_preset_name`

Implementing issue #1729

I do not have access to a testing/sandbox Datadog environnement unfortunately so I could not run the acceptance tests.

Created at 3 days ago
laudibert create branch monitor-notification-preset-name
Created at 3 days ago
Feature Request: Add option to toggle additionnal content in monitors notifications

Hello

Affected Resource(s)

  • datadog_monitor

Terraform Configuration Files

resource "datadog_monitor" "this" {
  notification_preset_name = "hide_all"
}

Expected Behavior

It would be nice to have the new parameter notification_preset_name as documented in the Datadog documentation that allows to hide content (query and mentions) on the monitor notification messages.

The API documentation seems limited as you can only find reference to this field in the examples for some reason.

Using the export feature on Datadog, possible values seem to be : hide_query, hide_handles and hide_all.

Thanks for all the hardwork.

Created at 1 week ago
issue comment
Memory leak?

3.11.0 deployed here and memory leak is still there.

It seems directly related to HTTP requests, a health checkup route that just returns true is being called every second. Other than that, no activity happens on that application.

Stack

  • Node.js version: 16.14.2
  • Agent version: 7.40.0
  • Relevant library versions:
    • @nestjs/common@9.2.0
    • express@4.18.2
Created at 2 weeks ago