-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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 Alertmanager config options to Alertmanager crd #2766
Comments
The reload should happen automatically. The reason we haven't added the Alertmanager configuration itself to the CRD is because we see similar collaboration potential with Alertmanager route configurations and receivers as there is with ServiceMonitors. As in each namespace should be able to specify these for themselves. |
Thanks for the reply :) |
Yes exactly. It'll be some time, but we already have some of this work planned. |
Great! The route is kind of recursive, it's a bit complex. |
I would like to see this happen and would be willing to help out... would we first want to see schema, before we start working on code? |
Yes defining a schema in advance would be good, feel free to write up initial outline so we have something to start with. Thanks! |
Hello! I've started a design document: Feel free to comment there... |
@simonpasquier That's great! Will take a look at this! |
Closed by #3451 |
What is missing?
Currently Alertmanager crd spec doesn't have config options like prometheus crd
Just wondering if it's reasonable to add alert manager config to alertmanager crd spec?
And then regenerate the alertmanager secrets and then call alertmanager's reload interface?
Why do we need it?
Currently we've to edit the alertmanager secret manually and then reload
The text was updated successfully, but these errors were encountered: