Skip to content

Fix race in Reconcile where ApplyStatus is called on a deleted object#109

Merged
justinmir merged 1 commit intomainfrom
fix-race-apply-status
Feb 11, 2026
Merged

Fix race in Reconcile where ApplyStatus is called on a deleted object#109
justinmir merged 1 commit intomainfrom
fix-race-apply-status

Conversation

@justinmir
Copy link
Copy Markdown
Contributor

💸 TL;DR

Wrap the k8s NotFound error in ApplyStatus instead of discarding it, and ignore NotFound errors from ApplyStatus in the FSM reconciler since the object no longer exists.

✅ Checks

  • CI tests (if present) are passing
  • Adheres to code style for repo
  • Contributor License Agreement (CLA) completed if not a Reddit employee

Wrap the k8s NotFound error in ApplyStatus instead of discarding it,
and ignore NotFound errors from ApplyStatus in the FSM reconciler
since the object no longer exists.
@justinmir justinmir requested a review from a team as a code owner February 11, 2026 14:53
@justinmir justinmir merged commit 8173c7f into main Feb 11, 2026
1 check passed
@justinmir justinmir deleted the fix-race-apply-status branch February 11, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants