kolorful

Results 13 comments of kolorful

We've seen a similar issue with ubuntu ``` 2022-06-22 20:39:49.2879 info Traffic Agent Init v1.12.6) 2022-06-22 20:39:49.2920 error failed to clear chain TEL_INBOUND_TCP: running [/sbin/iptables -t nat -N TEL_INBOUND_TCP --wait]:...

Thanks, that did fix the init container issue. The sidecar now crashes due to ``` error: mkdir /tel_app_exports/app/var/run: read-only file system ``` Looks like we have to set `readOnlyRootFilesystem` to...

Looks like due to a hostpath mount of the pod under `/var/xyz`, `/tel_app_mounts/app/var` becomes owned by root. ``` ls -la /tel_app_exports/app lrwxrwxrwx 1 1001 1001 24 Jun 23 16:00 var...

I tried adding these on Ingress object but didn't change the header at all either. (Based on the "fix" in https://github.com/kubernetes/ingress-nginx/issues/3481#issuecomment-569683634) ``` annotations: nginx.ingress.kubernetes.io/configuration-snippet: | more_set_input_headers "X-Forwarded-Port: 443" ``` and...

Seems like this has somewhat been discussed in terms of the we can't override this header. https://github.com/kubernetes/ingress-nginx/issues/3481#issuecomment-443311017 @aledbf do you think it would make sense to add an config to...

@david-yu yes, I'll close mine since https://github.com/hashicorp/consul-k8s/pull/3693 includes my change and added more features to it. Thank @jukie for incorporating the changes to your PR.

This can be closed since https://github.com/hashicorp/consul-k8s/pull/3693 has the change.

Is there anything else needed for this MR to be merged? https://github.com/helm/helm/blob/main/CONTRIBUTING.md#size-labels seems to indicate PRs need two maintainers' approval for merging?