Skip to content

Conversation

@qinxx108
Copy link
Contributor

this uses the new error with reason to determine based on status code what the reason is for the notifiers:

  • Pagerduty
  • Opsgenie
  • Pushover
  • Victorops
  • Webhook
  • Wechat

Signed-off-by: Yijie Qin <qinyijie@amazon.com>
@qinxx108 qinxx108 force-pushed the 4xx-error-for-notification-failure-receivers branch from 3bb4e4a to b9b68d2 Compare March 28, 2023 22:26
Signed-off-by: Yijie Qin <qinyijie@amazon.com>
@simonpasquier simonpasquier merged commit 9a8d1f9 into prometheus:main Apr 7, 2023
@simonpasquier
Copy link
Member

Thanks!

hoperays pushed a commit to hoperays/alertmanager that referenced this pull request Apr 23, 2023
* add reason code to notifiers

Signed-off-by: Yijie Qin <qinyijie@amazon.com>
fgouteroux pushed a commit to fgouteroux/alertmanager that referenced this pull request Apr 25, 2023
* add reason code to notifiers

Signed-off-by: Yijie Qin <qinyijie@amazon.com>
Signed-off-by: François Gouteroux <francois.gouteroux@gmail.com>
fgouteroux pushed a commit to fgouteroux/alertmanager that referenced this pull request Apr 25, 2023
* add reason code to notifiers

Signed-off-by: Yijie Qin <qinyijie@amazon.com>
Signed-off-by: François Gouteroux <francois.gouteroux@gmail.com>
radek-ryckowski pushed a commit to goldmansachs/alertmanager that referenced this pull request Nov 6, 2023
* add reason code to notifiers

Signed-off-by: Yijie Qin <qinyijie@amazon.com>
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 this pull request may close these issues.

2 participants