Skip to content

Ecs and celery#65

Open
gracewhitney wants to merge 14 commits into
mainfrom
ecs_and_celery
Open

Ecs and celery#65
gracewhitney wants to merge 14 commits into
mainfrom
ecs_and_celery

Conversation

@gracewhitney
Copy link
Copy Markdown
Contributor

@gracewhitney gracewhitney commented Apr 24, 2026

Adds ecs deployment support, with or without celery-backed worker service. I tested building the template and spot-checked various things, but it would be good to test building and running an ecs deploy from sample-django-app to see if it actually works

@gracewhitney gracewhitney requested a review from zags April 24, 2026 21:06
@gracewhitney gracewhitney marked this pull request as ready for review April 24, 2026 21:06
Comment thread {{cookiecutter.project_slug}}/common/models.py
Comment thread {{cookiecutter.project_slug}}/config/settings.py
sender.add_periodic_task(schedule, task.s(), name=task_name)


# TODO: Delete me
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

⛏️ use the full "TODO: delete me; this is just a reference example"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Deleting entirely since update_task_monitor is now a permanent reference example

Comment thread {{cookiecutter.project_slug}}/terraform/envs/production/main.tf Outdated
Comment thread {{cookiecutter.project_slug}}/Dockerfile
Comment thread {{cookiecutter.project_slug}}/requirements.in
@gracewhitney
Copy link
Copy Markdown
Contributor Author

Retested, I believe this is now ready to go

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