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

Pluggable strategies for ResourceManager #509

Closed
scothis opened this issue Apr 4, 2024 · 1 comment
Closed

Pluggable strategies for ResourceManager #509

scothis opened this issue Apr 4, 2024 · 1 comment

Comments

@scothis
Copy link
Member

scothis commented Apr 4, 2024

The ResourceManger manages the lifecycle of a resource by calling create/update/delete as needed to keep the actual and desired state in sync. Currently the update part is hard coded as a client.Update() call. There are other ways to manage "updating" a resource including various flavors of patching and most interestingly server-side apply. Using server-side apply with duck types should make update operations safe, negating the need for #506.

A user should be able to override the "update" strategy for the resource manager in a ChildReconciler/ChildSetReconciler/AggregateReconciler/anywhere else the ResourceManager is used today.

@scothis
Copy link
Member Author

scothis commented Feb 8, 2025

#548 moved ResourceManager to be a strategy, users can define a custom manager. As of now, there is a single implementation encapsulating the previous behavior.

@scothis scothis closed this as completed Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant