Skip to content

fix: Remove usages of devstack_docker. #162

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

Merged
merged 2 commits into from
Jul 28, 2025

Conversation

dianakhuang
Copy link
Member

@dianakhuang dianakhuang commented Jul 23, 2025

The devstack_docker settings file was removed from the edx-platform repository, and we had a bunch of references to it, including in the provisioning scripts.

The correct way to handle this is to use the devstack.py file, which is mounted in the host already.

This also adds in some devstack-specific cms.yml and lms.yml configuration files that then get mounted into the lms and cms containers. These are ported from the devstack-experimental.yml files that were previously in edx-platform and were removed. openedx/edx-platform#36908


I've completed each of the following or determined they are not applicable:

  • Made a plan to communicate any major developer interface changes (or N/A)

@dianakhuang dianakhuang force-pushed the diana/remove-devstack-docker branch from c0c86d2 to b13df2b Compare July 28, 2025 19:55
The devstack_docker settings file was removed from the edx-platform
repository, and we had a bunch of references to it, including in the
provisioning scripts.

The correct way to handle this is to use the devstack.py file, which is
mounted in the host already.
These were removed from the edx-platform repository.
Moving them here so devstack can still use them.
@dianakhuang dianakhuang force-pushed the diana/remove-devstack-docker branch from b13df2b to b7ca8e1 Compare July 28, 2025 20:16
@dianakhuang dianakhuang merged commit be4586c into master Jul 28, 2025
19 checks passed
@dianakhuang dianakhuang deleted the diana/remove-devstack-docker branch July 28, 2025 21:29
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