Skip to content

Mark job ABORTED on interrupt during metadata/find_datasets - #96

Merged
koenvo merged 1 commit into
mainfrom
fix/aborted-during-discovery
Aug 12, 2026
Merged

Mark job ABORTED on interrupt during metadata/find_datasets#96
koenvo merged 1 commit into
mainfrom
fix/aborted-during-discovery

Conversation

@koenvo

@koenvo koenvo commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

The KeyboardInterrupt/SystemExit handlers only wrapped the task-execution phase, so a Ctrl-C or Cloud Run SIGTERM during metadata or find_datasets propagated uncaught and left the up-front RUNNING summary as a zombie.

This PR fixes that

@koenvo
koenvo force-pushed the fix/aborted-during-discovery branch from c8a5e0e to 2c2f231 Compare August 12, 2026 11:15
The KeyboardInterrupt/SystemExit handlers only wrapped the task-execution phase,
so a Ctrl-C or Cloud Run SIGTERM during metadata or find_datasets propagated
uncaught and left the up-front RUNNING summary as a zombie. execute() now wraps
the whole _execute_locked run: any interrupt flips the still-RUNNING summary to
ABORTED and persists it directly (not via yield, which a killed consumer may
never drain). If an inner task-phase handler already marked the summary, its
richer partial-results version is left untouched (state != RUNNING guard).

Claude-Session: https://claude.ai/code/session_01B5EfLJqoafjW1FhvkxGSmg
@koenvo
koenvo force-pushed the fix/aborted-during-discovery branch from 2c2f231 to 7e646ee Compare August 12, 2026 11:19
@koenvo
koenvo merged commit fca042d into main Aug 12, 2026
13 checks passed
@koenvo
koenvo deleted the fix/aborted-during-discovery branch August 12, 2026 11:26
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