init.image.pullPolicy and nodeAgent.image.pullPolicy not being respect

[zyue110026]

We have noticed two instances of incorrect helming in your Kubernetes configuration files. Hardcoded or default configuration values in the template are considered anti-pattern and configurable values in the values.yaml file are not being respect, resulting in user-provided configurations never being applied. Additionally, we provide anecdotal evidence from trivy-operator#729 regarding the incorrect helming defect.

What happened:

init.image.pullPolicy and nodeAgent.image.pullPolicy not take affect when changing them to other values. This is due to missing implementation for this value in daemonset.yaml file:

amazon-vpc-cni-k8s/charts/aws-vpc-cni/templates/daemonset.yaml

Line 41 in 2884e90

initContainers:

amazon-vpc-cni-k8s/charts/aws-vpc-cni/templates/daemonset.yaml

Line 128 in 2884e90

image: {{ include "aws-vpc-cni.nodeAgentImage" . }}

Attach logs

What you expected to happen:

change image pullpolicy for initcontainer and nodeAgent image can take affect, or delete these two values in value.yaml file

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

• Kubernetes version (use kubectl version):
• CNI Version: main version
• OS (e.g: cat /etc/os-release):
• Kernel (e.g. uname -a):

[orsenthil]

This is fixed in #3010 and will be available in next release.