search

aws / eks-charts

Amazon EKS Helm chart repository
Apache License 2.0
1.2k stars 966 forks source link

AWS AppMesh Controller - Add configuration to update ENVOY LOG FORMAT #665

Open pkalantri-ebsco opened 2 years ago

pkalantri-ebsco commented 2 years ago

Is your feature request related to a problem? I want to be able to remove the "%REQ(X-FORWARDED-FOR)%" from the envoy access logs from sidecar container as it does not meet my project's compliance requirements.

Standalone Envoy has the ability to do this: https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log/usage#default-format-string

Is your feature request related to a specific Helm chart, if yes mention name of the chart? appmesh-controller

Describe the solution you'd like I'd like a parameter called sidecar.logFormat that allows me to update the default envoy LOG format

Describe alternatives you've considered Since we have the AWS AppMesh Controller installed in our EKS cluster using the helm chart, there isn't much we can do since the controller is adding envoy to our virtual nodes as a sidecar.

This is pretty important for us since we are failing to meet our compliance requirements due to this.

chenggaw commented 2 years ago

We also need this feature @pkalantri-ebsco How to configure appmesh-controller to output envoy logs to /dev/stdout?