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

Goals of this project? #41

Closed philippfrank closed 2 years ago

philippfrank commented 2 years ago

To the PM/lead of this project: We really love LogDNA and the fact that you work on a TF provider is great. However, you do not support the most basic operations that make LogDNA great, for example:

In my naive thinking, I would expect that you cover existing resources (views and alerts being the most basic elements of your app) to an extent that one can work with it. Everything else is just frustrating.

cc @leeliu

alfeng6 commented 2 years ago

Hi @philippfrank ! PM here :)

Really appreciate your feedback for our TF provider. I love that you are enjoying our line templates feature to help with log consumption — just so I can better understand how you are envisioning our TF Provider supporting, do you imagine being able to define a persisted line template as a part of a View resource in TF?

Also we are investigating some of the other bugs you've reported and will get back to you asap. We should be supporting the ability to add Slack channel alerts to Views.

philippfrank commented 2 years ago

Thanks for the quick response:

[...] do you imagine being able to define a persisted line template as a part of a View resource in TF?

Yes image

We should be supporting the ability to add Slack channel alerts to Views

Could you point me to a piece of documentation about how I can attach an existing slack alert channel to a view (versus: creating an inline slack channel alert).

gjanco commented 2 years ago

Hi @philippfrank, thanks for your reply.

  1. 22 Is now in our backlog

  2. To clarify @alfeng6 response, with the current release we only support recreating the view specific alert. To attach a preset, we will be fixing it as stated here (https://github.com/logdna/terraform-provider-logdna/issues/40#issuecomment-1016837443)