Bug Description
I can confirm that istiod helm chart (1.12.1 version) have by default includeInboundPorts: "*" in values.yaml.
But overriding the parameters doesn't take place since files/injection-template.yaml has this args:
"{{ annotation .ObjectMeta traffic.sidecar.istio.io/includeInboundPorts * }}"
That's why the sidecar is added with default and doesn't takes the override value
INBOUND_PORTS_INCLUDE=*
Version
$ kubectl version --short
Client Version: v1.19.3
Server Version: v1.19.16-gke.1500
$ helm list --namespace istio-system
NAME NAMESPACE REVISION UPDATED STATUS CHART APP VERSION
istio-base istio-system 5 2021-12-27 18:13:24.675766 +0200 IST deployed base-1.12.1 1.12.1
istiod istio-system 4 2021-12-27 18:15:38.536671 +0200 IST deployed istiod-1.12.1 1.12.1
Additional Information
No response
Bug Description
I can confirm that istiod helm chart (1.12.1 version) have by default includeInboundPorts: "*" in values.yaml.
But overriding the parameters doesn't take place since files/injection-template.yaml has this args:
"{{ annotation .ObjectMeta traffic.sidecar.istio.io/includeInboundPorts * }}"
That's why the sidecar is added with default and doesn't takes the override value
INBOUND_PORTS_INCLUDE=*
Version
Additional Information
No response