Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Support externalTrafficPolicy in the Logstash Helm chart #1548

Closed
kostasb opened this issue Jan 21, 2022 · 0 comments · Fixed by #1570
Closed

Support externalTrafficPolicy in the Logstash Helm chart #1548

kostasb opened this issue Jan 21, 2022 · 0 comments · Fixed by #1570
Assignees
Labels
enhancement New feature or request logstash

Comments

@kostasb
Copy link

kostasb commented Jan 21, 2022

Describe the feature:

Setting the externalTrafficPolicy to Local does not appear to have any effect in the resulting service.

  service:
    type: LoadBalancer
    externalTrafficPolicy: Local
kubectl describe svc logstash-logstash

...(omitted)
External Traffic Policy:  Cluster
...

Describe a specific use case for the feature:
There is a number of reasons why someone would need to set the externalTrafficPolicy to Local, namely:

There are two available options: Cluster (default) and Local. Cluster obscures the client source IP and may cause a second hop to another node, but should have good overall load-spreading. Local preserves the client source IP and avoids a second hop for LoadBalancer and NodePort type Services, but risks potentially imbalanced traffic spreading.

from https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#preserving-the-client-source-ip

@jmlrt jmlrt added enhancement New feature or request logstash labels Feb 2, 2022
jmlrt added a commit to jmlrt/helm-charts that referenced this issue Feb 9, 2022
@jmlrt jmlrt self-assigned this Feb 9, 2022
jmlrt added a commit that referenced this issue Mar 2, 2022
* [logstash] add externalTrafficPolicy support

Fix #1548

* fixup! [logstash] add externalTrafficPolicy support
jmlrt added a commit to jmlrt/helm-charts that referenced this issue Mar 2, 2022
* [logstash] add externalTrafficPolicy support

Fix elastic#1548

* fixup! [logstash] add externalTrafficPolicy support
jmlrt added a commit that referenced this issue Mar 2, 2022
* [logstash] add externalTrafficPolicy support

Fix #1548

* fixup! [logstash] add externalTrafficPolicy support
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request logstash
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants