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

Resource Deletion #90

Open
caleblloyd opened this issue Oct 26, 2022 · 1 comment
Open

Resource Deletion #90

caleblloyd opened this issue Oct 26, 2022 · 1 comment
Assignees

Comments

@caleblloyd
Copy link
Contributor

I am trying to understand the history of #56 with respect to this comment:

With finalizers gone, the Kubernetes Informers no longer send us an event when a resource is deleted

Why are finalizers gone? They are still documented as supported for CRDs:

https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definitions/#finalizers

Managed resource deletion in k8s is typically handled in finalizers, not a list/cleanup loop like was implemented in #56. Or at least it was 1-2 years ago, it has been a while since I worked on an Operator so have things changed? Or should we go back to using finalizers here?

@wallyqs
Copy link
Member

wallyqs commented Oct 31, 2022

The cleanup loop got added in order to avoid this deadlock that can happen when deleting CRDs, for which the workaround is to issue a kubectl patch command to remove the finalizers when deleting: kubernetes/kubernetes#60538 (comment)

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

2 participants