You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on May 2, 2024. It is now read-only.
moving the package into $(tutor config printroot)/env/build/openedx/requirements
listing the package in $(tutor config printroot)/env/build/openedx/requirements/private.txt
rebuilding the base edx-platform image (tutor images build openedx). Because the local packages are installed so high up in the Dockerfile, this rebuild breaks most of the cache and thus can take 20+ minutes.
The comprehensive image rebuild makes sense when the user intends to use these requirements on a production site: it is important that the local packages be built into the base image, and it's important that production static assets are built afterward, etc.
Developers, on the other hand, don't need the packages built into the base image. They would be happy if the package was just built in to the development variant of the image, which takes much less time (~1 minute) to rebuild.
Background
The current package dev notes involve:
$(tutor config printroot)/env/build/openedx/requirements
$(tutor config printroot)/env/build/openedx/requirements/private.txt
tutor images build openedx
). Because the local packages are installed so high up in the Dockerfile, this rebuild breaks most of the cache and thus can take 20+ minutes.The comprehensive image rebuild makes sense when the user intends to use these requirements on a production site: it is important that the local packages be built into the base image, and it's important that production static assets are built afterward, etc.
Developers, on the other hand, don't need the packages built into the base image. They would be happy if the package was just built in to the development variant of the image, which takes much less time (~1 minute) to rebuild.
Tasks
Notes
Prototype implementation here: kdmccormick/tutor#26
The text was updated successfully, but these errors were encountered: