Skip to content

Fixes #15: Fix missing notifications for Jobs cancelled during cutover - #93

Merged
jnovinger merged 1 commit into
mainfrom
15-an-interrupted-cutover-can-omit-a-cancelled-runs
Oct 30, 2026
Merged

jnovinger merged 1 commit into
mainfrom
15-an-interrupted-cutover-can-omit-a-cancelled-runs

Conversation

@pheus

@pheus pheus commented Oct 30, 2026

Copy link
Copy Markdown
Collaborator

Fixes: #15

Cutover now saves a cancelled Job's failed state and owner notification in a single transaction, so an interrupted pass can safely resume without losing the notification. If a retried Job already has a notification, it is refreshed and marked unread rather than duplicated.

Cutover now fails a waiting Job and writes its owner's notice in
one transaction, so an interrupted pass leaves neither and the
resumed pass sends it. A retried Job's earlier notice is re-issued.

Fixes #15
@pheus
pheus marked this pull request as ready for review October 30, 2026 15:46
@pheus
pheus requested a review from jnovinger October 30, 2026 15:47
@jnovinger
jnovinger merged commit 6cca157 into main Oct 30, 2026
20 checks passed
@jnovinger
jnovinger deleted the 15-an-interrupted-cutover-can-omit-a-cancelled-runs branch October 30, 2026 19: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.

An interrupted cutover can omit a cancelled run's notification

2 participants