Skip to content

Remove and migrate more celery tasks#4742

Merged
jmsmkn merged 8 commits into
mainfrom
remove_old_celery_tasks
May 28, 2026
Merged

Remove and migrate more celery tasks#4742
jmsmkn merged 8 commits into
mainfrom
remove_old_celery_tasks

Conversation

@jmsmkn
Copy link
Copy Markdown
Member

@jmsmkn jmsmkn commented May 28, 2026

These have now been migrated to lambda_tasks and the periodic tasks set up.

See #4408

@jmsmkn jmsmkn changed the title Remove old celery tasks Remove and migrate more celery tasks May 28, 2026
@jmsmkn jmsmkn requested a review from amickan May 28, 2026 13:43
Copy link
Copy Markdown
Contributor

@amickan amickan left a comment

Choose a reason for hiding this comment

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

LGTM!

I have a general question though: how do you decide which tasks to migrate to lambda tasks when / in what order? And how does the period task set-up work these days (if we add new periodic tasks, should these use the lambda decorator and how do we define when they should run)?

@jmsmkn
Copy link
Copy Markdown
Member Author

jmsmkn commented May 28, 2026

how do you decide which tasks to migrate to lambda tasks when / in what order?

I'm slowly going through and seeing what adds new features to test - this time it's calling new functions and mixed signatures. Trying to do things that will migrate entire apps at a time. Also avoiding anything people might be working on right now (invoices, challenges, evaluations, algorithms, reader studies).

And how does the period task set-up work these days

It's done using event bridge scheduler. See the latest commit in the cloud infrastructure repo.

@jmsmkn jmsmkn merged commit 1c4506d into main May 28, 2026
22 of 24 checks passed
@jmsmkn jmsmkn deleted the remove_old_celery_tasks branch May 28, 2026 15:07
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.

2 participants