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

Alloy Clustering mode is currently incompatible with running inside an Istio mesh when deployed via the Helm Chart #800

Closed
josh-thisisbud opened this issue May 8, 2024 · 3 comments · Fixed by #820
Labels
bug Something isn't working

Comments

@josh-thisisbud
Copy link
Contributor

josh-thisisbud commented May 8, 2024

What's wrong?

Currently this Alloy chart is falling into this trap when run with Istio as a sidecar - https://istio.io/latest/docs/ops/common-problems/network-issues/#503-error-while-accessing-headless-services

The fix is relatively simple, either;

  • Allow the end user to specify the port name used instead of hardcoding it to http. This will allow istio users to fallback to the TCP support which would work for this traffic.
  • Alternatively, send a host header containing the hostname which was looked up on the request to join a cluster

Steps to reproduce

Install Alloy via the helm chart on a Kubernetes cluster running Istio in strict mode and with allow_any traffic profile enabled. Have Alloy run with an istio sidecar. Attempt to turn on clustering mode. See that no nodes are able to communicate with each other.

System information

No response

Software version

Alloy v1.0.0

Configuration

No response

Logs

No response

@josh-thisisbud
Copy link
Contributor Author

@rfratto As we use the published helm chart, how would I go about getting a new version cut with this fix in it? Is it done on a specific schedule? Could it already be done and I'm just not seeing it on ArtifactHub?

@rfratto
Copy link
Member

rfratto commented May 22, 2024

@josh-thisisbud We don't have a normal release cadence for the Helm chart. We typically put out a release when enough changes accumulate. But since you're waiting for this change and I don't know of any alternatives to use a non-release version, I'll open a PR to release a new version now.

@josh-thisisbud
Copy link
Contributor Author

josh-thisisbud commented May 22, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants