-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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/v2) - Remove the deprecation #3372
Comments
I am ready to work on this. |
/assign |
I have tried finding the file where changes needed. unfortunately didnt get it. Let me know the file where changes needed. |
Hi @Sajiyah-Salat, This one seems a good issue ! 1 - Follow up the quick start (https://book.kubebuilder.io/quick-start.html) and check the warning when you run make install 2 - Do the change You will need to replace : patchesStrategicMerge 3 - Now, let's test
4 - Run make generate to regenerate all (samples) under testdata and docs within yout change prior send the PR Acceptance criteria
|
I have tried making directory test and it exits already. also I got three deprecation notice in make generate. still I have tried my best to follow the steps. |
What do you want to happen?
See that the default scaffolds:
Therefore we should replace patchesStrategicMerge by patches in the kustomize/v2 templates code. PS.: Update the code and use make generate to update the samples under testdata.
Extra Labels
No response
The text was updated successfully, but these errors were encountered: