openedx
Demonstration Course not exporting in Open edX Olive Demo Server
An attempt on exporting the Demonstration Course course-v1:edX+DemoX+Demo_Course failed over the week-end on the previous Open edX Olive Demo Server. Something about a transcript file not being available.
A second attempt on exporting the Demonstration Course course-v1:edX+DemoX+Demo_Course failed today (Oct 24) with the export just hanging on us.
I imported the https://github.com/openedx/openedx-demo-course with the tarball available https://github.com/openedx/edx-demo-course/releases/download/v1.1/edx_demo_course_1_1.tar.gz. The course is available on the Open edX Olive Demo Server as course-v1:edX+DemoX+Demo_Course2
I did not have any problems exporting this course on the new server or in the old server.
We might be a defective Demonstration Course on the Open edX Olive Demo Server.
Please note that there seems to be differences in the contents of the course I imported and the course with is having export issues. There is a new "holding section" at the end of course-v1:edX+DemoX+Demo_Course2. It seems to replace the "prerequisites" section at the end of course-v1:edX+DemoX+Demo_Course.
We should determine which is the "real" Demonstration Course.
According to @mariajgrimaldi something similar happened during the nutmeg testing: https://github.com/openedx/build-test-release-wg/issues/158#issuecomment-1127615140
@sambapete and @mariajgrimaldi it seems you have passed the test, so we can mark it passed yes, in my opinion.
However you have found a new bug, i.e. the bad version of the demo course is being used in the Overhang Instances. For now, I have added this issue to the new tab in our test sheet named List of Bugs With No Test Cases as NETCB_5 so I can follow up with @regisb at an appropriate time.
@sambapete has given me a better understanding of this issue. We surely cannot have the "broken" demo course in Olive as it includes a failed test. We have another month to rectify this :)
@DeanJayMathew, @regisb, this is just about updating the version of the demo course in the demo instance, correct? Can we close it?
It seems to me that the only different between Pierre's import of the demo course and mine is that he imports v1.1 (from February 2022) of the course, while I import the version from open-release/olive.master. If there is an error in the Olive branch of the demo course, I think that it should be fixed upstream.
While I agree with Régis about the upstream problem with the open-release/olive.master, the README page of the repositories does include a unique link to the tarball for v1.1 . Is it the "current" version according to the authors? Should all other versions be void or considered experimental? It's a question for the authors of the repository. I suggest we close this issue and open a bug report or an issue for that repository.
We can leave this issue open for now, until we can open the issue properly upstream.
I can confirm that it's still not possible to export the demo course in olive.1. To reproduce:
- go to https://studio.demo.openedx.overhang.io/export/course-v1:edX+DemoX+Demo_Course
- login with the demo admin credentials
- click export
This should be fixed in the https://github.com/openedx/edx-demo-course repo by backporting some fixes from the master branch to the "open-release/olive.master" branch.
To resolve this issue it will be more convenient if we can import arbitrary versions of the demo course. So this other issue should probably be tackled first: https://github.com/overhangio/tutor/issues/730
This is working in the Palm demo: https://studio.palm.demo.overhang.io/export/course-v1:edX+DemoX+Demo_Course I'll be closing this then :)
