-
-
Notifications
You must be signed in to change notification settings - Fork 349
Add notifications to Infrastructure Monitoring #2315
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
Comments
I'd like to work on this issue as my first contribution. Please assign it to me if it's still available. |
Hello. Sure. Since it's not a small feat, can you please let us know how you are going to implement it on a high level, pointing out to necessary blocks/codebases? Then we can move ahead. |
Hey @gorkem-bwl — thanks for the ping! Here’s how I’d tackle adding notifications:
Event should include things like service name, status, timestamp, etc. Provider adapters Slack (pkg/notifications/slack) Discord (pkg/notifications/discord) Telegram (pkg/notifications/telegram) Webhook (pkg/notifications/webhook) Config changes notifications:
slack:
enabled: true
webhook_url: https://hooks.slack.com/…
discord:
enabled: false
telegram:
enabled: true
bot_token: ABC123
chat_id: 98765
webhook:
enabled: true
url: https://example.com/webhook CLI flags bash Unit tests for each notifier (mock HTTP). End-to-end test simulating a status change. Docs Let me know if you'd like me to proceed. |
@PaulyBearCoding I think it may be better for you to check other Github projects. Checkmate is a complex project and needs an understanding/reading of underlying codebase before really starting with anything. |
Thanks for the feedback, @gorkem-bwl! 🙏 I definitely want to get up to speed on the Checkmate codebase before jumping in. Could you point me toward:
Once I’ve got a feel for those areas, I can open small PRs—whether it’s improving docs, adding tests, or scaffolding the notifications package—and build from there. I just want to learn more and contribute. |
Hello. I can confidently say that Checkmate is a large project, and is not for beginners, unfortunately. |
Just like uptime monitoring, Slack/Discord/Telegram/Webhook notifications should be a part of Infra monitoring.
The text was updated successfully, but these errors were encountered: