From b1e91e36262d12c94f225f182f44f3dc9d37066d Mon Sep 17 00:00:00 2001 From: Ghassan Maslamani Date: Sun, 12 Sep 2021 15:53:23 +0300 Subject: [PATCH] BugFix: fixes slow installation close #488 Change the source of installation Ipyton, install it from PyPy instead of source, which shall speed up the process of building the docker image for openedx-dev --- CHANGELOG.md | 2 ++ tutor/templates/build/openedx-dev/Dockerfile | 5 +---- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 5ee9b2b..62ca36a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,8 @@ Note: Breaking changes between versions are indicated by "💥". - [Improvement] In the "openedx" Docker images, convert git patches to cherry-picks for a cleaner source tree. - 💥[Feature] Make it possible to override local job configuration. This deprecates the older model for running jobs which dates back from a long time ago. +- [Bugfix] Install Ipython directly from pypy instead of installing it from source (the reason it was installed from source is no longer relevant). The effect of this shall speed up the process of openedx-dev Docker image. + ## v12.0.4 (2021-08-12) - [Security] Apply security patch [28442](https://github.com/edx/edx-platform/pull/28442). diff --git a/tutor/templates/build/openedx-dev/Dockerfile b/tutor/templates/build/openedx-dev/Dockerfile index 8d9afbf..fdf9253 100644 --- a/tutor/templates/build/openedx-dev/Dockerfile +++ b/tutor/templates/build/openedx-dev/Dockerfile @@ -8,10 +8,7 @@ RUN apt update && \ # Install dev python requirements RUN pip install -r requirements/edx/development.txt -# We install ipython from source to avoid too many deprecation warnings -# https://github.com/ipython/ipython/issues/12206 -# We might be able to avoid this once they make a release later than 7.19.0. -RUN pip install ipdb==0.13.4 git+https://github.com/ipython/ipython.git@d0649a54a8936a8019d54549779dc92bcbde4e68#egg=ipython +RUN pip install ipdb==0.13.4 ipython==7.27.0 {{ patch("openedx-dev-dockerfile-post-python-requirements") }}