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
Alertmanager: Update the prometheus/alertmanager to its latest version #3097
Conversation
@pracucci could you please take a look? 🙏 Would love if this could make it to next week's build. |
Sure @gotjosh. Looking into it. |
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.
This Alertmanager upgrade introduces two new settings which allow to read from file and send it content over the network:
- GlobalConfig > SMTPAuthPasswordFile
- EmailConfig > AuthPasswordFile
Similar configs are blocked in Mimir, because of security reasons. We should block these two new settings too. Please look at validateAlertmanagerConfig()
and its test TestValidateAlertmanagerConfig
.
- github.com/aws/aws-sdk-go v1.44.66 // indirect
+ github.com/aws/aws-sdk-go v1.44.94 // indirect I went through the AWS release notes and everything looks good to me. |
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.
LGTM!
What this PR does
Upgrades alertmanager to its latest version.
Which issue(s) this PR fixes or relates to
Fixes #2651, which was fixed via prometheus/alertmanager#2968
Fixes #2806
A revert of #2924, which was fixed via prometheus/alertmanager#3080
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]