Skip to content
This repository was archived by the owner on Aug 1, 2024. It is now read-only.

Commit 664c44f

Browse files
authored
fix: in coursegraph provision, run lms command instead of exec'ing (#823)
This ensures that the provisioning script works whether or not the lms was already started. If an lms container doesn't exist, then `docker-compose run` ensures that one is created to run the management command. Using `docker-compose exec`, the script would fail with "no container lms_1" if lms wasn't already running.
1 parent 39571ca commit 664c44f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

provision-coursegraph.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,6 @@ sleep 10 # Give Neo4j some time to boot up.
1818

1919
echo -e "${GREEN} Updating LMS courses in Coursegraph...${NC}"
2020

21-
docker-compose exec lms bash -c 'source /edx/app/edxapp/edxapp_env && cd /edx/app/edxapp/edx-platform/ && ./manage.py lms dump_to_neo4j --host coursegraph.devstack.edx --user neo4j --password edx'
21+
docker-compose run lms bash -c 'source /edx/app/edxapp/edxapp_env && cd /edx/app/edxapp/edx-platform/ && ./manage.py lms dump_to_neo4j --host coursegraph.devstack.edx --user neo4j --password edx'
2222

2323
echo -e "${GREEN} Coursegraph is now up-to-date with LMS!${NC}"

0 commit comments

Comments
 (0)