Skip to content

Conversation

@aothms
Copy link
Collaborator

@aothms aothms commented Feb 1, 2025

First attempt combing two approaches:

  • I think as @rw-bsi suggested, use Popen and terminate when catching celery's injected soft timeout exception
  • but also a generic vacuum of any lingering process (not sure if necessary - it's what I started with)
  • we could also choose either one of the approaches I think

Tested with:

CELERY_TASK_SOFT_TIME_LIMIT = 4
CELERY_TASK_TIME_LIMIT = 6

@aothms aothms requested a review from rw-bsi February 1, 2025 17:21
@rw-bsi rw-bsi changed the title Terminate subprocesses in celery after timeout IVS-356 - Terminate subprocesses in celery after timeout Jul 15, 2025
@rw-bsi rw-bsi changed the title IVS-356 - Terminate subprocesses in celery after timeout IVS-356 - Terminate subprocesses in celery after timeout (PAUSED) Jul 15, 2025
@Ghesselink
Copy link
Contributor

Superceded by #241

@Ghesselink Ghesselink closed this Oct 15, 2025
Ghesselink added a commit that referenced this pull request Oct 15, 2025
…cesses-on-timeout

IVS-356 - Kill celery tasks properly after timeout (>#142)
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.

3 participants