Open BryanFauble opened 1 week ago
Hi @BryanFauble, Thank you for opening this issue!
Would you be willing to contribute a PR for this?
Thanks @grandwizard28 ,
This is something that gives a trivial benefit so I don't want to spend time on it for now.
Something of a larger benefit that I started to look at was starting to consider how k8s secrets might be used to configure things like the clickhouse password. However, there was some more work to consider how stuff like the traces url could also be fed in as it includes the username/password as well:
The secretFrom stanza was something that the underlying clickhouse implementation is doing: https://altinity.com/blog/clickhouse-confidential-using-kubernetes-secrets-with-the-altinity-operator
Let's discuss this on a separate issue: https://github.com/SigNoz/charts/issues/525.
I'd like to keep this open for supporting multiple alert managers :)
In this helm chart it looks like 1 url is ever used/considered: https://github.com/SigNoz/charts/blob/bb53857ff7deff14b779e18475bec32b790327ea/charts/signoz/templates/_helpers.tpl#L298
Ask: When running multiple replicas of alert mamager (https://github.com/SigNoz/alertmanager) allow for any alerts to be sent to 1 or more configured replicas.
Taken from the Readme of the signoz fork of alertmanager it supports the overall ask of this issue, it shows how it expects prometheus to point at multiple alert manager instances:
Here are more links to code where changes are likely needed: