bcda: adopt only jobs whose files are still served; survive a cancelled-while-adopting job #1296

Closed
opened 2026-10-25 20:36:16 +00:00 by kert · 1 comment
Owner

Upstream review docs/superpowers/specs/2026-10-25-bcda-upstream-review.md §D. completed_jobs() keeps every Task with FHIR status completed, which includes Archived/Expired jobs whose files are gone (410); since bcda-app #1495 a Completed job can be DELETEd by any sandbox user and turns Cancelled (404) mid-download.

  • filter completed_jobs() to end within 24 h (server file retention)
  • Store.adopt: finish as failed with the server diagnostics, keeping the files already downloaded, instead of raising mid-way
  • Exports page: hide/flag jobs older than 24 h
Upstream review docs/superpowers/specs/2026-10-25-bcda-upstream-review.md §D. `completed_jobs()` keeps every Task with FHIR status `completed`, which includes Archived/Expired jobs whose files are gone (410); since bcda-app #1495 a Completed job can be DELETEd by any sandbox user and turns Cancelled (404) mid-download. - [ ] filter `completed_jobs()` to `end` within 24 h (server file retention) - [ ] `Store.adopt`: finish as `failed` with the server diagnostics, keeping the files already downloaded, instead of raising mid-way - [ ] Exports page: hide/flag jobs older than 24 h
kert added this to the P52: BCDA v3 migration — client, crosswalk-driven flatteners, vendored upstream fixtures, proving notebook milestone 2026-10-25 20:36:16 +00:00
kert added the cli label 2026-10-25 20:36:16 +00:00
Author
Owner

Done in 2cc1300: Client.completed_jobs(max_age_hours=24) drops Archived/Expired jobs; downloads attempt every file and record failures per file (the files ledger keeps what landed; resume finishes the rest).

Done in 2cc1300: `Client.completed_jobs(max_age_hours=24)` drops Archived/Expired jobs; downloads attempt every file and record failures per file (the files ledger keeps what landed; `resume` finishes the rest).
kert closed this issue 2026-10-25 21:11:48 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: homelab/stack#1296