Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Goals of this project? #41

Closed
philippfrank opened this issue Jan 6, 2022 · 3 comments
Closed

Goals of this project? #41

philippfrank opened this issue Jan 6, 2022 · 3 comments

Comments

@philippfrank
Copy link

philippfrank commented Jan 6, 2022

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
Copy link

alfeng6 commented Jan 6, 2022

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
Copy link
Author

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
Copy link
Contributor

gjanco commented Jan 19, 2022

Hi @philippfrank, thanks for your reply.

  1. Support for line template #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 (Bug: provider will update resources in place when they have slack channels attached #40 (comment))

@gjanco gjanco closed this as completed Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants