-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: Drop the use of the Tubular repository
The Tubular repository has been deplrecated and the relevant scripts have been moved to the edx-platform codebase. * Drop the custom image build and rely on the openedx image. * Add support for Tutor 18 / Open edX Redwood (breaking change) Fixes: #32
- Loading branch information
Maari Tamm
committed
Jul 19, 2024
1 parent
63407e5
commit ea158b3
Showing
11 changed files
with
34 additions
and
50 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
- name: retirement-pipeline-configmap | ||
files: | ||
- plugins/retirement/build/retirement/pipeline_config/config.yml | ||
- plugins/retirement/apps/retirement/config.yml | ||
- plugins/retirement/apps/retirement/run_retirement_pipeline.sh |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,5 @@ | ||
############# Retirement worker service | ||
retirement-job: | ||
image: {{ RETIREMENT_DOCKER_IMAGE }} | ||
image: {{ DOCKER_IMAGE_OPENEDX }} | ||
volumes: | ||
- ../plugins/retirement/build/retirement/pipeline_config:/tubular/pipeline_config | ||
depends_on: | ||
- lms | ||
- ../plugins/retirement/apps/retirement:/openedx/edx-platform/scripts/user_retirement/plugin |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
13 changes: 13 additions & 0 deletions
13
tutorretirement/templates/retirement/apps/retirement/run_retirement_pipeline.sh
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
cool_off_days=$1 | ||
pip install --upgrade pip && pip install -r scripts/user_retirement/requirements/base.txt --exists-action w | ||
python scripts/user_retirement/get_learners_to_retire.py --config_file=scripts/user_retirement/plugin/config.yml --output_dir=scripts/user_retirement/plugin/learners_to_retire --cool_off_days="$cool_off_days" | ||
for filename in scripts/user_retirement/plugin/learners_to_retire/*;do | ||
if test ! -f "$filename";then | ||
echo "No users to retire" | ||
continue | ||
else | ||
IFS="=" read -r name username <"$filename" | ||
echo "Retiring $username" | ||
python scripts/user_retirement/retire_one_learner.py --config_file=scripts/user_retirement/plugin/config.yml --username="$username" | ||
fi | ||
done |
11 changes: 0 additions & 11 deletions
11
tutorretirement/templates/retirement/build/retirement/Dockerfile
This file was deleted.
Oops, something went wrong.
12 changes: 0 additions & 12 deletions
12
tutorretirement/templates/retirement/build/retirement/run_retirement_pipeline.sh
This file was deleted.
Oops, something went wrong.