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

Add receiver, fix get_alerts filtering #40

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

partcyborg
Copy link
Contributor

@partcyborg partcyborg commented Sep 30, 2024

Add receiver as an acceptable query parameter for get_alerts().

Receiver is a valid parameter as shown in the openapi spec

Also, fix the filter generation for the v2 api. In the V2 API, each <key>="<val>" parameter is passed as an individual filter get parameter, not as a comma separated string

Add `receiver` as an acceptable query parameter for `get_alerts()`.

Receiver is a valid parameter as shown in the [openapi
spec](https://github.com/prometheus/alertmanager/blob/e1492602209b86e0ca6d7671c7353b62a31b897b/api/v2/openapi.yaml#L171)
@partcyborg partcyborg changed the title Add receiver parameter to get_alerts() Add receiver, fix get_alerts filtering Sep 30, 2024
@jpavlav jpavlav merged commit 4add45e into ABORGT:master Sep 30, 2024
1 check failed
@partcyborg
Copy link
Contributor Author

Thanks @jpavlav for the prompt response! Are there any plans to release a new package with recent changes to pypy?

@jpavlav
Copy link
Collaborator

jpavlav commented Oct 2, 2024

Oh sure. I'll see if I can get to it today. Thanks for the contributions!

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