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

tutor config save interpolates values and replaces them in config #1192

Closed
misilot opened this issue Jan 28, 2025 · 1 comment
Closed

tutor config save interpolates values and replaces them in config #1192

misilot opened this issue Jan 28, 2025 · 1 comment

Comments

@misilot
Copy link

misilot commented Jan 28, 2025

Bug description

Tutor overwrites values in config.yml, where I am trying to update the Registry for our openedx and mfe images, so they can be built on one system and promoted to production.

How to reproduce

edit the local config.yml in the TUTOR_ROOT directory.
Add:

DOCKER_IMAGE_OPENEDX: docker.io/orgname/openedx:{{ TUTOR_VERSION }}"
MFE_DOCKER_IMAGE: docker.io/orgname/openedx-mfe:19.0.0-indgio

I'd actually like to use this, but MFE_VERSION isn't available to tutor config save

MFE_DOCKER_IMAGE: docker.io/orgname/openedx-mfe:{{ MFE_VERSION }}

run tutor config save
Get the following output:

DOCKER_IMAGE_OPENEDX: docker.io/orgname/openedx:19.0.1
MFE_DOCKER_IMAGE: docker.io/orgname/openedx-mfe:19.0.0-indigo

I'd like to have it pull like it does the correct values TUTOR_VERSION and MFE_VERSION so the values mimic and get updated whenever the plugins and application are updated.

Environment

Ubuntu 24.04
tutor, version 19.0.1
Additional context

@DawoudSheraz DawoudSheraz moved this from Pending Triage to Backlog in Tutor project management Jan 29, 2025
@regisb
Copy link
Contributor

regisb commented Jan 30, 2025

Tutor's config.yml includes only rendered values. Editing this file manually is never a good idea. If you need to modify the config values, you should create a plugin. In your particular case, you should make use of the following filter: https://docs.tutor.edly.io/reference/api/hooks/catalog.html#tutor.hooks.Filters.CONFIG_OVERRIDES

I'm closing this since this isn't an actual issue. For support on how to use Tutor, please use the Open edX forum: https://discuss.openedx.org/

@regisb regisb closed this as completed Jan 30, 2025
@github-project-automation github-project-automation bot moved this from Backlog to Done in Tutor project management Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

2 participants