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

DM-38376: Improve Kopf configuration and error handling #713

Merged
merged 1 commit into from
Mar 17, 2023
Merged

Conversation

rra
Copy link
Member

@rra rra commented Mar 17, 2023

Set a timeout on Kubernetes watches in Kopf rather than using an unlimited watch. nolar/kopf#585 says that watches without an expiration sometimes go silent and stop receiving events, which is consistent with some problems we've seen.

Mark Kubernetes object parsing failures as permanent so that Kopf won't retry with the same version of the object. Mark Kubernetes API failures as temporary so that Kopf will use its normal delay.

Set a timeout on Kubernetes watches in Kopf rather than using an
unlimited watch. nolar/kopf#585 says that
watches without an expiration sometimes go silent and stop receiving
events, which is consistent with some problems we've seen.

Mark Kubernetes object parsing failures as permanent so that Kopf
won't retry with the same version of the object. Mark Kubernetes API
failures as temporary so that Kopf will use its normal delay.
@rra rra enabled auto-merge March 17, 2023 15:50
@rra rra merged commit cb28d1b into main Mar 17, 2023
@rra rra deleted the tickets/DM-38376 branch March 17, 2023 15:50
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

Successfully merging this pull request may close these issues.

1 participant