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

Alertmanager deployment in vm/victoria-metrics-alert doesn't support IPv6 only k8s cluster #856

Closed
fayak opened this issue Feb 12, 2024 · 2 comments · Fixed by #1440
Closed
Assignees
Labels
question Further information is requested

Comments

@fayak
Copy link

fayak commented Feb 12, 2024

Hello,

I tried deploying vm/victoria-metrics-alert with alertmanager enabled on my IPv6 only k8s cluster, but some hardcoded values in the chart prevent me from doing so.

In victoria-metrics-alert/templates/alertmanager-deployment.yaml

- --web.listen-address=0.0.0.0:9093
- --cluster.advertise-address=$(POD_IP):6783

should be configurable somehow to cope with IPv6 only, as the IP need some renaming/brackets.

I'm not sure what the best implementation for such configuration should be.

Best regards,

zekker6 added a commit that referenced this issue Feb 23, 2024
Allow overriding Alertmanager listen addr to support IPv6-only clusters. See: #856
zekker6 added a commit that referenced this issue Feb 23, 2024
Allow overriding Alertmanager listen addr to support IPv6-only clusters. See: #856
@zekker6
Copy link
Contributor

zekker6 commented Feb 23, 2024

Hello @fayak, #883 added support of specifying listen address of Alertmanager via alertmanager.listenAddress. This was included in release v0.9.1 of victoria-metrics-alert chart. Could you please confirm if that helps in your case?

@zekker6 zekker6 self-assigned this Feb 23, 2024
@fayak
Copy link
Author

fayak commented Feb 23, 2024

Hello @zekker6 !
Thank you for the commit, I need to test in my setup because I think I have to rewrite - --cluster.advertise-address=$(POD_IP):6783 to - --cluster.advertise-address=[$(POD_IP)]:6783 as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants