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.

Sign up for GitHub

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

Jump to bottom

Alertmanager mTLS config #4241

Open
TamasNeumer opened this issue Sep 2, 2021 · 5 comments · May be fixed by #6489
Open

Alertmanager mTLS config #4241

TamasNeumer opened this issue Sep 2, 2021 · 5 comments · May be fixed by #6489
Labels
help wanted kind/feature

Comments

@TamasNeumer
Copy link
Contributor

What is missing?

AlertManager has recently introduced configuration options for mTLS:

Why do we need it?

To be able to configure AlertManager gossip to use mTLS.
Would be nice in general to be able to add flags to the AlertManager container itself.

Anything else we need to know?:
I think in the alertmanager statefulset generator we don't have any option for this.
@simonpasquier simonpasquier pinned this issue Sep 6, 2021
@TamasNeumer
Copy link
Contributor Author

@simonpasquier @brancz

I'm happy to help and implement the feature, however some guidance would be helpful.

  • As far as I see an additional field needs to be added to the AlertmanagerSpec API which should point to a ConfigMap. The ConfigMap is mounted as a volume.
  • Then it's the user's responsibility to deploy the ConfigMap with the desired content
  • We then add an additional (Second?) reloader which targets the the ConfigMap?

Thanks!

@simonpasquier
Copy link
Contributor

It would be more friendly to expose the different settings in the Alertmanager CRD rather than forcing the user to craft the config manually. You may want to have a look at what exists for web TLS config (https://github.com/prometheus-operator/prometheus-operator/blob/master/Documentation/api.md#webtlsconfig).
cc @fpetkovski who worked on the implementation for web TLS.

@raptorsun raptorsun mentioned this issue Feb 23, 2022
Closed
5 tasks
@mviswanathsai
Copy link
Contributor

Is this still relevant?

@simonpasquier
Copy link
Contributor

yes

@mviswanathsai
Copy link
Contributor

Taking this up!

@mviswanathsai mviswanathsai linked a pull request Apr 9, 2024 that will close this issue
Draft
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted kind/feature
Projects
None yet
Milestone
No milestone
3 participants
@simonpasquier @TamasNeumer @mviswanathsai