digitalocean / terraform-provider-digitalocean

Terraform DigitalOcean provider
https://registry.terraform.io/providers/digitalocean/digitalocean/latest/docs
Mozilla Public License 2.0
503 stars 276 forks source link

Support App Platform alerts via slack #802

Open hstaudacher opened 2 years ago

hstaudacher commented 2 years ago

Is your feature request related to a problem? Please describe.

When configuring an alert within the digitalocean_app spec only an email alert will be configured. It would be nice if one can choose between email/slack as within the do UI.

andrewsomething commented 2 years ago

Thanks for opening an issue to track this. As mentioned in the PR adding initial alert support, destinations can not be configured in the app spec itself. You need to call a separate endpoint to add Slack hooks or other emails:

https://docs.digitalocean.com/reference/api/api-reference/#operation/assign_alert_destinations

This might end up needing to be a separate app alert destinations resource.

hstaudacher commented 2 years ago

I wasn't aware of #797. Thx for pointing me to it. The problem I see right now is that the endpoint you are mentioning requires an alert id which is not exposed when creating the app in terraform. So, I guess would have to list all alerts for all apps and update the destinations. Is this what you would a suggest? If not can you point me to the right direction? Thx in advance.

andrewsomething commented 2 years ago

Yes. doctl also has commands for using both of those if the CLI makes it easier for you.

ricardo01010 commented 2 years ago

Hello, are you planning to include this in the module? That is, the integration of slack is native.

voxivoid commented 2 years ago

it would be great to support this since it is a very common use case for the alerts

hstaudacher commented 1 year ago

Any progress on this?

WouterSioen commented 11 months ago

This is a feature we're missing as well, so it would be very nice to see this.

mwyld commented 7 months ago

This would be a great feature :)