Skip to content

feat: queued jobs table in database#81

Merged
marius-mather merged 21 commits into
mainfrom
job-queue-db
Jun 23, 2026
Merged

feat: queued jobs table in database#81
marius-mather merged 21 commits into
mainfrom
job-queue-db

Conversation

@marius-mather

@marius-mather marius-mather commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

SBP-422: add a queued_jobs table to the database, so the web service can write jobs to the queue, to be picked up by the scheduler.

Changes

  • QueuedJob database model
  • Workflow launch endpoint writes queued jobs to the database (interim step before we offload these to the job queue)
  • Refactor some existing workflow launch code to reduce duplication

How to Test

Run uv run pytest

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have added or updated documentation where necessary
  • I have run linting and unit tests locally
  • The code follows the project's style guidelines

@amandazhuyilan amandazhuyilan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good but reminder to please update the database diagram with generate_db_diagram.sh.

vtnphan
vtnphan previously approved these changes Jun 22, 2026

@vtnphan vtnphan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Look s good to me! Thanks for refactoring the workflow executors ^_^

@marius-mather marius-mather merged commit 98cde17 into main Jun 23, 2026
3 checks passed
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