Skip to content

Resolves deadlock issue with the readiness check.#127

Merged
winkingturtle-vmw merged 1 commit intomainfrom
readiness-check-deadlock
Jan 15, 2026
Merged

Resolves deadlock issue with the readiness check.#127
winkingturtle-vmw merged 1 commit intomainfrom
readiness-check-deadlock

Conversation

@geofffranks
Copy link
Copy Markdown
Contributor

A deadlock occurs when the storenode.run() function exits due to startup health check execution timing out, while the readiness check has already succeeded, placing a state transition into the readiness channel that will never be read. As a result, the readiness check was never processing signals to exit, resulting in stale containers.

This resolves this, and also resolves a hypothetical deadlock if the readiness channel is unable to be read from when transitioning from ready to not-ready.

Summary

no

Backward Compatibility

Breaking Change? no

A deadlock occurs when the storenode.run() function exits
due to startup health check execution timing out, while the
readiness check has already succeeded, placing a state transition
into the readiness channel that will never be read. As a result,
the readiness check was never processing signals to exit, resulting in
stale containers.

This resolves this, and also resolves a hypothetical deadlock if the
readiness channel is unable to be read from when transitioning from
ready to not-ready.
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Application Runtime Platform Working Group Jan 15, 2026
@winkingturtle-vmw winkingturtle-vmw merged commit 8eaf4cb into main Jan 15, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Application Runtime Platform Working Group Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants