-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
namePrefix in base breaks patchesJson6902 in overlay #972
Comments
Also, is there some documentation about the matching criteria in |
See also #316 |
pohly
added a commit
to pohly/kustomize
that referenced
this issue
Apr 18, 2019
The expectation is that the base entity can be referenced by its name with prefix, because the overlay shouldn't have to know how the base is generated. But currently the entity is only found when using the name without prefix. Related-To: kubernetes-sigs#972
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For a reproducible test, see #971
I'm using kustomize master, currently 177297c
I'm trying to patch a DaemonSet, but kustomize keeps complaining that it
couldn't find target apps_v1beta2_DaemonSet|~X|~P|pmem-csi-node|~S
.The target is defined as:
and there such a DaemonSet in the output when I comment out that part above:
The problem is that the base has
namePrefix: pmem-csi-
, sopmem-csi-node
is the result of a rename.It works when using
name: node
.This looks like a layering violation to me. The overlay shouldn't have to know how the base produces its output. For example, it might use
namePrefix
now and later get changed to .yaml files where that prefix gets inserted by hand. Then the output is the same, but the overlay no longer works.It also is non-obvious to the user. It cost me a few hours to figure out the reason for the
couldn't find target
error, because I was matching mytarget
specification against the output of the base.The text was updated successfully, but these errors were encountered: