Skip to content

Refactor set-image to use kustomize filters properly#1209

Merged
efiacor merged 3 commits intokptdev:masterfrom
nokia:set-image-refact
Jan 12, 2026
Merged

Refactor set-image to use kustomize filters properly#1209
efiacor merged 3 commits intokptdev:masterfrom
nokia:set-image-refact

Conversation

@mozesl-nokia
Copy link
Copy Markdown
Contributor

Closes kptdev/kpt#4344

Rewrote the logic so that even with the "default" image paths we use the kustomize filter that the additionalImageFields method uses. Thus, the vendored in third_party files are now removed.

This fixes the issue where tags are appended without a :. Also, initContainers now works in the default case.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 9, 2026

Deploy Preview for krm-function-catalog canceled.

Name Link
🔨 Latest commit aa86b94
🔍 Latest deploy log https://app.netlify.com/projects/krm-function-catalog/deploys/6964b9a92b1004000865b0cc

Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
Copy link
Copy Markdown
Collaborator

@liamfallon liamfallon left a comment

Choose a reason for hiding this comment

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

Nice cleanup of the function. It's also great to use the kustomize code directly and get rid of the cloned code.

Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
@efiacor efiacor merged commit 635727d into kptdev:master Jan 12, 2026
11 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.

The set-image function sometimes omits the : between the image name and tag if newTag is not specified

3 participants