-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
kustomize edit add base
command adds dir as resource and not base
#1556
Comments
Would it be possible for kustomize to explicitly warn about these deprecation on stderr? |
Thanks for pointing that out! The issue is, I'm trying to apply my newly created
It seems that
Do you know where can I report that? |
Another thing that is quite confusing with the deprecation is that only using So now if you edit kustomizations with the kustomize cli and not by hand they won't be compatible with kubectl. eg.:
My kustomization in that case looks like this:
When can one expect a feature to also drip downstream into kubectl? |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
I am still getting this error.
|
/remove-lifecycle stale |
@tomjohnburton @reegnz #1647 (comment) It worked for me as well using working on windows docker for desktop
|
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
hey are there any news? |
@pommelinho Didn't use kustomize for a while now, but checking the go.mod for kubectl: https://github.com/kubernetes/kubectl/blob/18e781fa774127786b5a2092ed2dc0351dafdb87/go.mod#L46 Nope, it's still the same kustomize version that's used. :( |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@fejta-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/remove-lifecycle rotten |
/reopen |
@pierluigilenoci: You can't reopen an issue/PR unless you authored it or you are a collaborator. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/reopen |
@Shell32-Natsu: Reopened this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@Shell32-Natsu, Any news on this???? Still with a problem with kubectl... $ kubectl apply -k env/xxxx-westus-ppd-dev
error: rawResources failed to read Resources: Load from path ../../base failed: '../../base' must be
a file (got d='/Users/marcello.desales/dev/gitlab.xxx.com/xxx/yyy/k8s/helm/charts/agent/output/agent/templates/base') This is the version from $ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.8", GitCommit:"9f2892aab98fe339f3bd70e3c470144299398ace", GitTreeState:"clean", BuildDate:"2020-08-13T16:12:48Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.11", GitCommit:"3a3612132641768edd7f7e73d07772225817f630", GitTreeState:"clean", BuildDate:"2020-09-02T06:46:33Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"} |
Also experiencing this. Using |
We are in the last (hopefully) step to remove apimachinery from kustomize. #2506 is used to track this work. |
Seems like it is still an issue with kubectl 1.20.5 Using the multibase example from https://github.com/kubernetes-sigs/kustomize/tree/master/examples/multibases
It still works without issue using
|
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
this is fixed in the most recent version of kubectl. |
Version
Expected behavior
The command
kustomize edit add base ../base
should add a base to the kustomization file:#Actual behavior
Running the
kustomize edit add base ../base
command results in this:How to reproduce
The text was updated successfully, but these errors were encountered: