-
Notifications
You must be signed in to change notification settings - Fork 147
Conversation
…manager, nodeexporter
Fork source code. Add monitoring serivces: prometheus, grafana, alertmanager
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking the time for preparing this contribution.
Sorry, if this now sounds harsh, but most of the changes in here are totally unrelated to updating telebot to v2. Some files simply don't belong in the repository at all, like .docx
files.
I stopped reviewing the files at some point, so only in the first half there are comments. Please address these before I take another look.
"github.com/oklog/run" | ||
"github.com/prometheus/alertmanager/notify" | ||
"github.com/prometheus/alertmanager/template" | ||
"github.com/prometheus/client_golang/prometheus" | ||
"github.com/prometheus/client_golang/prometheus/promhttp" | ||
"gopkg.in/alecthomas/kingpin.v2" | ||
"github.com/vu-long/alertmanager-bot/pkg/alertmanager" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to clone this project into $GOPATH/src/github.com/metalmatze/alertmanager-bot
and not update these import paths.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add me as a contributor, it let me push my branch to your repos.
@@ -0,0 +1,144 @@ | |||
version: "3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we want to merge this example at all, please open another Pull Request for doing so.
@metalmatze OK let me do other PR for telebot v2. |
No description provided.