Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: don't run Celery workers in dev mode #1041

Closed
wants to merge 1 commit into from

Commits on Jun 10, 2024

  1. feat!: don't run Celery workers in dev mode

    Tutor's importing * from devstack.py[1] for the development settings,
    and that means that we aren't using Celery workers at all in dev mode
    (see [2]).  This makes it so they don't start in dev mode, thus saving
    everyboding a significant chunk of RAM.
    
    [1] https://github.com/overhangio/tutor/blob/master/tutor/templates/apps/openedx/settings/lms/development.py#L3
    [2] https://github.com/openedx/edx-platform/blob/master/lms/envs/devstack.py#L35
    
    To do so, we rely on Docker compose profiles[3].
    
    [3] https://docs.docker.com/compose/profiles/
    
    BREAKING CHANGE: the `COMPOSE_PROJECT_STARTED` hook signature had to be changed to
    accomodate profile selection.
    arbrandes committed Jun 10, 2024
    Configuration menu
    Copy the full SHA
    e3919ee View commit details
    Browse the repository at this point in the history