Skip to content

Implement IngestionRun model#56

Merged
cgoudet merged 2 commits intomainfrom
47-ingestionrun-model
Apr 16, 2026
Merged

Implement IngestionRun model#56
cgoudet merged 2 commits intomainfrom
47-ingestionrun-model

Conversation

@AymanL
Copy link
Copy Markdown
Collaborator

@AymanL AymanL commented Apr 15, 2026 •
edited
Loading

Summary

Introduces the IngestionRun model to record every ingestion attempt with full stage and outcome tracking (closes #47). No pipeline rewiring — models, migration, and admin only.

Test plan

  • Run transitions to success with success_kind=metadata_only and stage=done
  • Run transitions to success with success_kind=full and stage=done
  • Failed run records error_message and error_stage
  • Duplicate non-empty DOI raises ValueError before any IngestionRun row is created

@AymanL AymanL changed the base branch from main to 46-document-source-file-link April 15, 2026 11:49
@AymanL AymanL requested a review from cgoudet April 15, 2026 16:05
Base automatically changed from 46-document-source-file-link to main April 16, 2026 06:44
cgoudet
cgoudet approved these changes Apr 16, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@cgoudet cgoudet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cgoudet cgoudet merged commit efaa602 into main Apr 16, 2026
1 check passed
@cgoudet cgoudet deleted the 47-ingestionrun-model branch April 16, 2026 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cgoudet cgoudet cgoudet approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

IngestionRun model

2 participants

@AymanL @cgoudet