-
Notifications
You must be signed in to change notification settings - Fork 445
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tutor mounts add /path-to-openedx-scorm-xblock
doesn't automatically install scorm-xblock
#997
Comments
I'm facing similar issue while installing cmi5 xblock for tutor quince. |
Just to make sure: did you really run Note: I'm unable to reproduce this issue on my computer, so I'm trying to understand what might be going on. |
No no, I only wrote that here as a placeholder command. I did replace the correct openedx-scorm-xblock path as the folder was correctly mounted to the |
In Open edX Dockerfile, dev and production requirements are prefixed by However, the weird/confusing part is that when I added |
@regisb and I debugged this issue and the root cause for this involves how the mounts are handled. During the build time, the mounts are copied and then installed in editable mode, thus generating egg-info (https://github.com/overhangio/tutor/blob/master/tutor/templates/build/openedx/Dockerfile#L277). This is also the reason that the installed xblocks were showing up when running The egg_info containing mounts are actually present in the image, which can be seen by doing:
The fix for this involves either using bind mount for the xblock dirs or re-installing the mounts (if egg_info is not present) during initialization (similar thing is already being done for platform https://github.com/overhangio/tutor/blob/master/tutor/templates/jobs/init/mounted-edx-platform.sh) |
That's a great explanation Dawoud. And it's very similar to that other issue that @kdmccormick faced a few months back: #813 Now I blame myself for not thinking about this earlier... |
Bug description
Using the latest mounting command
tutor mounts add /path-to-xblock
, I tried to install scorm-xblock in my tutor install. This was on a fresh install of tutor and only the default plugins were installed as well. However, tutor did not install scorm-xblock after I had mounted it using the above mentioned command and did not show up in the Advanced tab on a studio course even after I had addedscorm
to the advanced settings list.How to reproduce
On a fresh install of tutor quince 17.0, you can follow the following steps to reproduce the command.
tutor mounts add path-to-scorm-xblock
tutor images build openedx-dev
tutor dev stop && tutor dev start -d
tutor dev do importdemocourse
. Under Advanced Module List In the advanced settings of the course, addscorm
and save changes. Now, while adding a new unit, you should be able to see Scorm Module as an option in the advanced tab but that is not the case here.The output of
tutor mounts list
for me is:The output of
tutor config printvalue OPENEDX_EXTRA_PIP_REQUIREMENTS
for me is:[]
openedx-scorm-xblock does show for me at the
/mnt
path inside the LMS container.That is also the solution I am currently utilizing which is to follow the following steps:
tutor dev run lms bash
cd /mnt
pip install -e ./openedx-scorm-xblock
exit
tutor dev stop && tutor dev start -d
After these steps, openedx-scorm-xblock works as intended and hot-reloading works as well as it is installed in editable mode.
Surprisingly enough, while running
tutor images build openedx-dev
after adding the xblock as a mount, it seems that tutor does try to installopenedx-scorm-xblock
in editable mode as can be seen in the screenshot below.Environment
The text was updated successfully, but these errors were encountered: