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

Add alerting to this plugin #14

Open
rajeshggwp opened this issue Apr 2, 2023 · 6 comments · May be fixed by #23
Open

Add alerting to this plugin #14

rajeshggwp opened this issue Apr 2, 2023 · 6 comments · May be fixed by #23

Comments

@rajeshggwp
Copy link

The plugin currently does not support alerting and hence creating this issue to enhance the plugin to support alerting. I found this grafana doc which explains how to add alerting to custom backend plugin - https://grafana.com/tutorials/build-a-data-source-backend-plugin/#enable-grafana-alerting

@rajeshggwp
Copy link
Author

I tried building the plugin with the change mentioned in the above doc but I ran into issues while adding the alerting rule in grafana.
@meln5674 I would like to contribute to this issue and let me know your thoughts on this issue

@meln5674
Copy link
Owner

meln5674 commented Apr 2, 2023

I've not actually used grafana alerting before, so if that's something you'd know more about, I'd be more than happy to accept a PR. If you do, please make sure to add something to the integration tests to exercise it.

@rajeshggwp
Copy link
Author

got it! I'll give it a shot.

@stefkauff
Copy link

Hi, any update on this?
Would definitely be a nice feature to have.

@rajeshggwp
Copy link
Author

I got occupied with other work, I'll make the changes and test it. Based on Grafana's documentation, this change should be straight-forward. https://grafana.com/developers/plugin-tools/tutorials/build-a-data-source-backend-plugin#enable-grafana-alerting

@rajeshggwp rajeshggwp linked a pull request Oct 13, 2023 that will close this issue
@alexandre-lelain
Copy link

Hello,

Looking forward to seeing this feature enabled for the plugin !

Thank you :)

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

Successfully merging a pull request may close this issue.

4 participants