Skip to content

revert(worker): rely on visibility-timeout recovery (staging) - #285

Merged
suguanYang merged 1 commit into
stagingfrom
revert/wangbinqi/bounded-worker-shutdown-staging
Aug 14, 2026
Merged

suguanYang merged 1 commit into
stagingfrom
revert/wangbinqi/bounded-worker-shutdown-staging

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

Staging cherry-pick of the reviewed cleanup in #284. The patch IDs are identical.

  • Remove the ineffective bounded gevent cancellation introduced by fix(worker): bound gevent warm shutdown #282.
  • Retain the independent Redis visibility watchdog and 4,500-second visibility contract.
  • Retain billing/idempotency behavior and document the accepted interruption fallback in the worker bootstrap.

Validation

Merge after #284. Merging this PR will trigger the normal staging deployment.

The bounded gevent cancellation raced with an active PyMuPDF subprocess, removed its workspace, and still missed the ECS stop deadline. Keep the independent Redis visibility watchdog as the accepted interruption fallback.
@suguanYang
suguanYang merged commit e87a0fc into staging Aug 14, 2026
6 checks passed
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