Skip to content

feat: add templates with tutor14 support#18

Merged
mariajgrimaldi merged 2 commits into
mainfrom
MJG/tutor-14
Oct 18, 2022
Merged

feat: add templates with tutor14 support#18
mariajgrimaldi merged 2 commits into
mainfrom
MJG/tutor-14

Conversation

@mariajgrimaldi
Copy link
Copy Markdown
Contributor

@mariajgrimaldi mariajgrimaldi commented Oct 11, 2022

Description

This PR adds support for tutor 14+ k8s templates and needed configurations. Roughly, this is what changed:

What's changed

  1. Containers entrypoint for cms-worker and lms-worker from:
["celery", "worker", "--app=cms.celery", "--loglevel=info", "--hostname=edx.cms.core.default.%%h", "--maxtasksperchild", "100", "--exclude-queues=edx.lms.core.default"]
["celery", "worker", "--app=lms.celery", "--loglevel=info", "--hostname=edx.lms.core.default.%%h", "--maxtasksperchild=100", "--exclude-queues=edx.cms.core.default"]

To:

["celery", "--app=cms.celery", "worker", "--loglevel=info", "--hostname=edx.cms.core.default.%%h", "--max-tasks-per-child", "100", "--exclude-queues=edx.lms.core.default"]
["celery", "--app=lms.celery", "worker", "--loglevel=info", "--hostname=edx.lms.core.default.%%h", "--max-tasks-per-child=100", "--exclude-queues=edx.cms.core.default"]
  1. apps/openedx/config/partials/auth.yml doesn't have CONTENTSTORE: null

  2. mongob parameters changed replicaSet to None

    mongodb_parameters = {
         "password": None,
         {% endif %}
         "db": "{{ MONGODB_DATABASE }}",
    +    "replicaSet": None,
     }
  3. It was added to openedx/config.settings/partials/common_lms.py:

    +BULK_EMAIL_SEND_USING_EDX_ACE = True
    +
    +# Make it possible to hide courses by default from the studio
    +SEARCH_SKIP_SHOW_IN_CATALOG_FILTERING = False

How to test

  1. Use this branch MJG/tutor-14
  2. Use the manifests-a-j-goddard branch MJG/nutmeg-installation (I know it's nuez)
  3. Generate manifests using the reference. Notice we're using template_root: "tutor14"

@mariajgrimaldi mariajgrimaldi changed the base branch from main to MJG/config-manifests October 12, 2022 17:48
@mariajgrimaldi mariajgrimaldi marked this pull request as ready for review October 12, 2022 17:59
@mariajgrimaldi mariajgrimaldi requested a review from a team October 14, 2022 14:20
@mariajgrimaldi mariajgrimaldi changed the base branch from MJG/config-manifests to main October 18, 2022 16:27
@mariajgrimaldi mariajgrimaldi merged commit cc392bb into main Oct 18, 2022
@MoisesGSalas MoisesGSalas deleted the MJG/tutor-14 branch November 16, 2023 15:19
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