splunk-connect-for-syslog icon indicating copy to clipboard operation
splunk-connect-for-syslog copied to clipboard
verified publisher icon splunk

Helm Chart Service externalTrafficPolicy

Open roosterfish opened this issue 2 years ago • 3 comments

Hi,

When trying to use the Helm chart with a Service of type NodePort, it would be beneficial being able to set the externalTrafficPolicy setting through the values.yml file. Currently this is hardcoded to Local. Setting it to Cluster allows to make use of the traffic proxying feature between compute nodes of the cluster.

  • https://github.com/splunk/splunk-connect-for-syslog/blob/main/charts/splunk-connect-for-syslog/templates/service.yaml#L17
  • https://github.com/splunk/splunk-connect-for-syslog/blob/main/charts/splunk-connect-for-syslog/templates/service.yaml#L95

Is there any reason this is currently hardcoded?

Thanks!

roosterfish avatar Mar 01 '23 08:03 roosterfish

Hi,

Thanks for raising it , we will take this and review in our upcoming sprint.

The values.yaml is still evolving , when we see a chance we improve it.

Regards, Rahul

rjha-splunk avatar Mar 01 '23 08:03 rjha-splunk

Hi, From my point of view this is hardcoded to local as you would loose the original source IP of the logs when using Cluster.

ehlo550 avatar Mar 08 '23 09:03 ehlo550

Hi @ehlo550, you will indeed loose the IP information, if the traffic doesn't hit a pod on the same compute node you are connecting to. However having this configurable wouldn't require putting an additional LB in front (e.g. if MetalLB is not an option) since the cluster will take care about distributing the incoming traffic to active pods all over the cluster.

roosterfish avatar Mar 14 '23 09:03 roosterfish