-
Notifications
You must be signed in to change notification settings - Fork 549
Alert manager based utilization enhancement #4788
Conversation
f31c93b
to
c7de70c
Compare
1373b8f
to
2193336
Compare
tags: | ||
- 'stopped-by-alert-manager' |
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.
We discussed about move tags
under tag-jobs
. Can we achieve this?
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.
We discussed about move
tags
undertag-jobs
. Can we achieve this?
It is legal to mix dict / string in the same list in yaml, but it becomes difficult to check if tag-jobs
is one of the available actions in the template. So I suggest to keep the current schema.
2cf1fad
to
1acb47c
Compare
1f7ffeb
to
8910f3e
Compare
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 this enhance.
Duplicated by #4940 |
This PR is for the first part of issue #4789
job-exporter
prometheus
job-handler
: deal with webhook request & redirect toRestServer
alert-handler
, send alert email to user when possible, change email template toejs
Realization Details: