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

api/v2: fix alerts with multiple receivers #1948

Merged
merged 1 commit into from
Jul 3, 2019

Conversation

simonpasquier
Copy link
Member

Closes #1942

@simonpasquier simonpasquier requested review from stuartnelson3 and mxinden and removed request for stuartnelson3 July 2, 2019 13:02
@simonpasquier
Copy link
Member Author

@mxinden @stuartnelson3 friendly ping. I think it would be worth having it in v0.18.0.

@simonpasquier simonpasquier merged commit 98a604e into prometheus:master Jul 3, 2019
@simonpasquier simonpasquier deleted the fix-1942 branch July 3, 2019 14:34
simonpasquier added a commit to simonpasquier/alertmanager that referenced this pull request Jul 3, 2019
Signed-off-by: Simon Pasquier <[email protected]>
simonpasquier added a commit that referenced this pull request Jul 3, 2019
* Cut v0.18.0-rc.0

Signed-off-by: Simon Pasquier <[email protected]>

* fix typo

Signed-off-by: Simon Pasquier <[email protected]>

* Fix more small typos

Signed-off-by: Simon Pasquier <[email protected]>

* include #1948

Signed-off-by: Simon Pasquier <[email protected]>

* Fix release date

Signed-off-by: Simon Pasquier <[email protected]>
DuskEagle pushed a commit to DuskEagle/alertmanager that referenced this pull request Aug 1, 2019
DuskEagle pushed a commit to DuskEagle/alertmanager that referenced this pull request Aug 1, 2019
* Cut v0.18.0-rc.0

Signed-off-by: Simon Pasquier <[email protected]>

* fix typo

Signed-off-by: Simon Pasquier <[email protected]>

* Fix more small typos

Signed-off-by: Simon Pasquier <[email protected]>

* include prometheus#1948

Signed-off-by: Simon Pasquier <[email protected]>

* Fix release date

Signed-off-by: Simon Pasquier <[email protected]>
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.

v0.17.0 - Receivers in the v2 alerts endpoint are wrong when using continue: true in routes
2 participants