Perf-alert Plugin transport settings addition
-
Hello all,
I was looking at the code for the
perf-alertplugin because I wasn't sure if it followed the same workflow as thebackup-reportsplugin where it sends emails and delivers to multiple transports. As it turns out, it only sends emails that are configured in the plugin, but I would love it to use the other transports available if they're set up liketransport-slack, etc. As a user who is interested in expanding my monitoring footprint in my infrastructure, I think theperf-alertplugin should follow the same reporting/alerting mechanisms as thebackup-reportplugin and notify totransport-slackalong with others. In my case I use Discord, but Mattermost, etc. users would all benefit based on the format thetransport-slackplugin already provides. Reading the code it seems like the Plugin system would benefit from having a singular "alerting" layer that you send messages to within each plugin and then the transport plugins just relay the messages to their appropriate APIs (this would likely require rethinking the plugin archictecture however, so not an actionable item), similar to a job queue.I would probably be able to file a PR to add this functionality, but I'm not confident enough to give it a go at the moment as I'd like to understand the implications of adding said code first. For now, I'll leave this as a feature request as I think most people would find this a welcome addition.
Hello! It looks like you're interested in this conversation, but you don't have an account yet.
Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.
With your input, this post could be even better 💗
Register Login