Questions , is there a reason why this is hardcode...
# support
p
Questions , is there a reason why this is hardcoded? https://github.com/SigNoz/signoz/blob/28bf2fe3f7c0b013685921c9d4bfdb254ac42e81/pkg/query-service/constants/constants.go#L15 ? It's causes probems when I used the helm chart to install cause then my alertmanager service will be on
<http://signoz-alertmanager:9093/api/>
? 🙂
a
oh..that needs to extracted as a config/env. Earlier the service name of alertmanager in helm chart was supposed to be
alertmanager
so the the const would work. But I think helm chart has evolved and this needs fixing. Thanks for pointing it out. Would you please create a github issue so that we can track this publically? We shall fix this soon.
@User quick fix would be to change service name of alertmanager to be available at
<http://alertmanager:9093/api/>
p
made a naiv approach with a PR: https://github.com/SigNoz/signoz/pull/698
💯 1
a
merged..will be released shortly
🙌 1