-
Notifications
You must be signed in to change notification settings - Fork 3
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
Start installing codejail requirements from base.txt instead of py38.txt #630
Comments
Nice, you caught this before I got around to bugging you about it :) Just a heads up, as I tried to explain here, we plan to keep requirements/edx/base.txt and requirements/edx-sandbox/base.txt synced in terms of constraints and language version. So, once Py3.8 support is dropped from edx-platform, it will be simultaneously be dropped for edx-sandbox. That might be exactly what you folks want, or it might not be, I'm not sure--just want to make sure you folks have all the information you need as you decide between |
Putting down a due date of "something like the Sumac release", so that we dont lose this. The thought is that this can be added to a "pipeline" related epic in H2 2024. |
Noting that it is entirely up to 2U when we want to get off an unsupported version of Python. It would be good for us to clean-up the unused file once we are done with it, but Axim does not plan on deleting this file. In other words, we don't have an external deadline for this work. |
Additional questions:
|
Note: Ubuntu 20.04 standard support runs out April 2025, and this upgrade work may need to be ticketed separately with SRE. I simply wanted to note that we need to take some action somewhere. |
[inform] We missed the due date. Do we want to schedule this work? |
FYI: @jristau1984 |
@robrap I believe that Arbi-BOM started the upgrade the sandbox environment to Python 3.11 as a pre-requisite for several other testing efforts where sandboxes were needed. This is the issue related to that effort: #850. It looks like there is a need to complete that work, which I will follow up with ARbi-Bom about in our next touchpoint. |
@timmc-edx: Can you confirm if this is complete? Thank you. |
No, our codejail is still running in Python 3.8.10. (This is distinct from 2U's sandboxes; "sandbox" in the meaning of this ticket actually refers to codejail.) EDIT: Here's a quick way to check the codejail environment if you can add a problem to a course:
|
We're considering whether it would be better to do this before or after containerization. |
Separate from 2U's edx-sandbox 3.8->3.11 upgrade, the cleanup that we're hoping to do from the openedx side is just to delete py38.txt, which is just a simple pointer to quince.txt. To avoid being broken by that cleanup, I recommend just updating your edx-sandbox deployment to point at |
@kdmccormick: Do you know why |
I do not-- I must have been confused. |
PR for pointing to more permanent requirements file: |
|
The structure of the edxapp-sandbox-requirements has changed and we need to switch in order to keep getting updates: openedx/edx-platform#34509
First step: Determine priority, may be able to lower it.
The text was updated successfully, but these errors were encountered: