Skip to content

Conversation

@QuentinBtd
Copy link
Contributor

@QuentinBtd QuentinBtd commented Sep 9, 2025

What type of PR is this?
Helm Chart feature

Which issue does this PR fix?:

No issue

What does this PR do / Why do we need it?:
Add overrideRepository to image option to override only image repositories and continue to use tag value.

It is useful if you want to use, for example, public.ecr.aws/eks/amazon-k8s-cni repository with the tag provide by the default chart values.

Testing done on this change:

I've deployed VPC CNI using the chart with these values:

init:
  image:
    overrideRepository: public.ecr.aws/eks/amazon-k8s-cni-init
nodeAgent:
  image:
    overrideRepository: public.ecr.aws/eks/aws-network-policy-agent
image:
  overrideRepository: public.ecr.aws/eks/amazon-k8s-cni

Will this PR introduce any new dependencies?:

No

Will this break upgrades or downgrades? Has updating a running cluster been tested?:
No

Does this change require updates to the CNI daemonset config files to work?:

No

Does this PR introduce any user-facing change?:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@QuentinBtd QuentinBtd marked this pull request as ready for review September 12, 2025 14:37
@QuentinBtd QuentinBtd requested a review from a team as a code owner September 12, 2025 14:37
@QuentinBtd
Copy link
Contributor Author

👋🏻 @jaydeokar

@jaydeokar jaydeokar added this to the v1.20.3 milestone Sep 24, 2025
Copy link
Collaborator

@jaydeokar jaydeokar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jaydeokar jaydeokar merged commit ca3af2a into aws:master Sep 24, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants