Skip to content
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

Revert upstream alertmanager to v0.24.0 #2924

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

joeblubaugh
Copy link
Contributor

@joeblubaugh joeblubaugh commented Sep 9, 2022

Revert upstream alertmanager to v0.24.0 to prevent panics when unmarshalling email headers from configuration.

This is a workaround for: prometheus/alertmanager#3064

@CLAassistant
Copy link

CLAassistant commented Sep 9, 2022

CLA assistant check
All committers have signed the CLA.

This is a workaround for: prometheus/alertmanager#3064

To prevent panics  when unmarshalling email headers from configuration.
@joeblubaugh joeblubaugh force-pushed the revert-2841-upgrade-alertmanager branch from e482798 to 5be880c Compare September 9, 2022 03:32
@joeblubaugh
Copy link
Contributor Author

I've updated the change to be more targeted - it only resets the alertmanager version.

@joeblubaugh joeblubaugh changed the title Revert "Upgrade upstream Alertmanager" Revert upstream alertmanager to v0.24.0 Sep 9, 2022
@krajorama krajorama merged commit 68b6197 into main Sep 9, 2022
@krajorama krajorama deleted the revert-2841-upgrade-alertmanager branch September 9, 2022 04:57
grafanabot pushed a commit that referenced this pull request Sep 9, 2022
This is a workaround for: prometheus/alertmanager#3064

To prevent panics  when unmarshalling email headers from configuration.

(cherry picked from commit 68b6197)
krajorama pushed a commit that referenced this pull request Sep 9, 2022
This is a workaround for: prometheus/alertmanager#3064

To prevent panics  when unmarshalling email headers from configuration.

(cherry picked from commit 68b6197)

Co-authored-by: Joe Blubaugh <joe.blubaugh@grafana.com>
joeblubaugh added a commit that referenced this pull request Sep 9, 2022
This prevents a panic when unmarshalling certain strings in email
notifier configurations.

This is a workaround for: prometheus/alertmanager#3064
krajorama pushed a commit that referenced this pull request Sep 9, 2022
This prevents a panic when unmarshalling certain strings in email
notifier configurations.

This is a workaround for: prometheus/alertmanager#3064
gotjosh added a commit that referenced this pull request Sep 30, 2022
gotjosh added a commit that referenced this pull request Sep 30, 2022
#3097)

* Alertmanager: Update the prometheus/alertmanager to its latest version

Fixes #2651
Fixes #2806
A revert of #2924
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants