-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Add Patch in our clients #9562
Comments
kubernetes/kubernetes#27293 adds to clientset |
@Kargakis dunno if this will be picked up with @smarterclayton rebase, but we just need to rerun the update-clientset script. |
(I will add |
I don't see a strong need, places that need it can lead the transition to clientsets. Are you going to update for the rest of our objects and pull over the one-offs? |
@deads2k yeah, more I'm digging into what everything will need update, more I'm inclined to simple transition to clientsets. Is the plan to switch |
We want to try, but we'll probably have to fix up the generator on our way through. Right now, "unusual" resources have to have hand-coded extension points wired in and if you're really unusual you have to write the entire thing yourself. We've been enforcing the "use a namespacer" rule during reviews, so we've started subdividing already |
@Kargakis we will start transition to cliensets for selected controllers after 3.3 |
@mfojtik fixed? |
Currently added upstream in various pulls, this issue is for the work that needs to be done for our namespacers.
@deads2k @smarterclayton @mfojtik
The text was updated successfully, but these errors were encountered: