Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Commit

Permalink
feat: Remove now-unneeded call to seed permissions for demo course fo…
Browse files Browse the repository at this point in the history
…rums (#6974)

This is now redundant with the earlier `import` call, which at this point
goes ahead and does the seeding.

One of these is probably used by sandboxes, and the other is probably not,
but both are referenced by `ansible-provision.sh`. Might as well remove
from both so that at least they're consistent with devstack, and so that
this is not propagated forward to whatever eventually replaces the sandbox
setup scripts.

See openedx-unsupported/devstack#1129 for details.
  • Loading branch information
timmc-edx committed Aug 3, 2023
1 parent 55e1d77 commit 098818c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 9 deletions.
6 changes: 0 additions & 6 deletions playbooks/roles/demo/tasks/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,3 @@
with_items:
- "{{ demo_test_and_staff_users }}"
when: demo_checkout.changed

- name: seed the forums for the demo course
shell: ". {{ demo_edxapp_env }} && {{ demo_edxapp_venv_bin }}/python ./manage.py lms --settings={{ demo_edxapp_settings }} seed_permissions_roles {{ demo_course_id }}"
args:
chdir: "{{ demo_edxapp_code_dir }}"
when: demo_checkout.changed
3 changes: 0 additions & 3 deletions util/jenkins/demo-course-provisioner.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,5 @@ for user in honor audit verified staff ; do
# Enroll users in the demo course
docker run --network=host --rm -u='www-data' -e NO_PREREQ_INSTALL="1" -e SKIP_WS_MIGRATIONS="1" -e LMS_CFG=/edx/etc/lms.yml -e DJANGO_SETTINGS_MODULE=lms.envs.docker-production -e SERVICE_VARIANT=lms -e EDX_PLATFORM_SETTINGS=docker-production -v /edx/etc/lms.yml:/edx/etc/lms.yml -v /edx/var/edx-themes:/edx/var/edx-themes -v /edx/var/edxapp:/edx/var/edxapp -v /var/run/mysqld/mysqld.sock:/var/run/mysqld/mysqld.sock ${app_repo}:latest python3 manage.py lms enroll_user_in_course -e \$email -c course-v1:edX+DemoX+Demo_Course
done
# Seed forums for the demo course
docker run --network=host --rm -u='www-data' -e NO_PREREQ_INSTALL="1" -e SKIP_WS_MIGRATIONS="1" -e LMS_CFG=/edx/etc/lms.yml -e DJANGO_SETTINGS_MODULE=lms.envs.docker-production -e SERVICE_VARIANT=lms -e EDX_PLATFORM_SETTINGS=docker-production -v /edx/etc/lms.yml:/edx/etc/lms.yml -v /edx/var/edx-themes:/edx/var/edx-themes -v /edx/var/edxapp:/edx/var/edxapp -v /var/run/mysqld/mysqld.sock:/var/run/mysqld/mysqld.sock ${app_repo}:latest python3 manage.py lms seed_permissions_roles course-v1:edX+DemoX+Demo_Course
EOF
}

0 comments on commit 098818c

Please sign in to comment.