-
Notifications
You must be signed in to change notification settings - Fork 366
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
MON-4118: chore: use alertmanager v2 in tests as v1 is not longer supported in Prometheus 3 #2544
base: master
Are you sure you want to change the base?
Conversation
…ported in Prometheus 3
…on as v1 is not longer suporter in Prom 3 chore: add a config validation check in case v1 in still using in one of the three AdditionalAlertmanagerConfig (Platform Prom, UWM Prom and Thanos ruler), this is temp and just meant to guide users during the first 4.X upgrade that ships Prom 3
@machine424: This pull request references MON-4118 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: machine424 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/label acknowledge-critical-fixes-only |
/retest-required |
FYI, from https://github.com/prometheus/alertmanager/releases/tag/v0.27.0 |
@machine424: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Thanks. |
/retest-required |
This will be mentioned in the release notes and maybe we'll upgrade a "known risk/ a warning" for ->4.19 upgrades.