search

logdna / terraform-provider-logdna

Terraform provider to simplify creation of views, preset alerts and other resources in LogDNA!
https://registry.terraform.io/providers/logdna/logdna/latest
MIT License
11 stars 12 forks source link

Slack alert channel support #14

Closed jsoh closed 2 years ago

jsoh commented 3 years ago

Reading through the source, I only see support for email, pagerduty, and webhoot. Are there plans to support Slack alert channels?

https://github.com/logdna/terraform-provider-logdna/blob/main/logdna/resource_alert.go

dm36 commented 3 years ago

@jsoh hi jimmy- thank you for your interest!! That is correct- we currently support email, pagerduty and webhook via the provider and we do plan to expand support to Slack alert channels

philippfrank commented 3 years ago

@dm36 Is this something that still could be expected?

mdeltito commented 3 years ago

@philippfrank yes, support for Slack as an alerting channel is still planned in our backlog

mdeltito commented 3 years ago

@jsoh @philippfrank one thing worth noting is that if you configure a webhook channel using the Slack webhook URL hooks.slack.com, you can accomplish essentially the same integration today. We do still plan to evaluate more explicit support for Slack (among other integrations) in the provider as well, but this might be a good solution in the meantime.

philippfrank commented 2 years ago

@mdeltito It is unnecessary complex though. The provider does already allow to define a (reusable) slack alert channel. Why isn't there a way to attach that channel to a view?

mdeltito commented 2 years ago

@philippfrank the slack_channel type was added after my comment above, we do now support that as a first-class channel type in the provider. it sounds like your other bug report is addressing a different concern, we'll take a look 👍