Skip to content
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

Use server-side-apply for the operator #687

Open
czunker opened this issue Nov 16, 2022 · 0 comments
Open

Use server-side-apply for the operator #687

czunker opened this issue Nov 16, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@czunker
Copy link
Contributor

czunker commented Nov 16, 2022

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
We should give server side apply (SSA) a try for our operator.

From what I understood, this could reduce the code we have to maintain and make the reconcile more straight forward.

Describe alternatives you've considered
We can keep the operator as is. It works as it is.

Additional context
General docs about SSA:
https://kubernetes.io/docs/reference/using-api/server-side-apply/#using-server-side-apply-in-a-controller

Advantages to controllers:
https://kubernetes.io/blog/2022/10/20/advanced-server-side-apply/?utm_source=pocket_mylist#controller-authors

@czunker czunker added the enhancement New feature or request label Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant