-
Notifications
You must be signed in to change notification settings - Fork 472
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
The client is missing Apply() functions, making use alongside other client-go packages difficult #1639
Comments
/assign |
Just FYI this is going to have a real world negative impact on all users: kubernetes/kubernetes#101384 |
Could you maintain the server-side apply client in a separate package - so it's opt-in? |
Hello, I want to work on this issue. But I cant see the whole picture. A more detailed description and some resource links would be helpful. Thank you. |
I would like to work on this @shaneutt @therealmitchconnors |
Sounds good @ashutosh887, but since I've noticed that you've taken some other issues on in the last couple days: please note that this one could take some significant time, let us know if you get stuck or need any help along the way! 🖖 /assign @ashutosh887 |
I'll try to work on it @shaneutt |
@ashutosh887 do you still have time for this? Anything we can do to help support you? |
@shaneutt what needs to happen to move this forward? It was disappointing to see it removed from the GA tracker. SSA is the standard that K8s seems to be trying to get everyone to move to, but lack of support in critical libraries makes doing the right thing very difficult... |
FYI - there's a PR out here #2537 /unassign @ashutosh887 |
@howardjohn are the concerns in your comment still relevant? |
Yes, but its not clear the impact. K8s types are far larger than gw-api, so it may not matter much in practice here. |
Hey @shaneutt I can take on this issue but I need some more details like steps to get started |
hey @Bharadwajshivam28 there's already a PR out of for this - #2537 Forgot to link it back to the issue. Secondly, keep an eye on who's assigned to the issue cause if there's already someone assigned you don't want two people creating duplicate work. |
Since K8s 1.22, client-go has supported server-side-apply through functions like Apply(), ApplyStatus(), etc. Because the gateway api client library lacks these functions, it must be called in non-idiomatic ways, increasing friction for those who intend to automatically control gateways.
The text was updated successfully, but these errors were encountered: