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

uwsgi.ini should not be copied to bind-mounted edx-platform #846

Open
regisb opened this issue May 30, 2023 · 2 comments
Open

uwsgi.ini should not be copied to bind-mounted edx-platform #846

regisb opened this issue May 30, 2023 · 2 comments
Assignees
Labels
enhancement Enhancements will be processed by decreasing priority

Comments

@regisb
Copy link
Contributor

regisb commented May 30, 2023

Is your feature request related to a problem? Please describe.

When we bind-mount edx-platform in the lms/cms container, uwsgi.ini is copied to the host directory. As a consequence, git status reports some modified contents.

@bradenmacdonald helpfully reported this issue here and suggested to fix it by git-ignoring uwsgi.ini in edx-platform. But I think that the issue should be resolved in Tutor.

Describe the solution you'd like

Tutor tries to avoid putting extra files in edx-platform. We should follow this rule of thumb and move uwsgi.ini outside of edx-platform. The problem is that /openedx has become quite a mess in the lms/cms containers:

$ tutor local run --no-deps lms ls /openedx
bin     data          locale  nodeenv       requirements  themes
config  edx-platform  media   node_modules  staticfiles   venv

We should take the chance to reorganize this directory.

@DawoudSheraz
Copy link
Contributor

For now, let's move uwsgi.ini one step out i.e. to openedx directory. The re-org of openedx directory itself should be done separately as I suspect it will involve some discussion. I will create an issue for that.

@DawoudSheraz
Copy link
Contributor

WIP PR #1036

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancements will be processed by decreasing priority
Projects
Development

No branches or pull requests

2 participants