-
Notifications
You must be signed in to change notification settings - Fork 13
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
Upgrade to Redwood #53
Conversation
@MoisesGSalas Hi, the PR branch name is incorrect, it is currently |
When we attempted this branch in sandbox (https://github.com/overhangio/openedx-release-demo/actions/runs/9077645999/job/24943000965), it failed with the following error:
We have tutor-indigo enabled on the sandbox, not sure if that is impacting this. |
@DawoudSheraz, I don't think it's related to tutor-indigo. If I were to take a wild guess it seems like base openedx image isn't tagged before the codejail image is built. I didn't test with I will test to see if I can replicate the error, in the meantime I guess you can add a step to the action that builds the openedx image first and then runs |
Actually, Open edX image is building first in the sequence. I can add the explicit step but I am not sure if that would work. You can see the build logs on https://github.com/overhangio/openedx-release-demo/actions/runs/9077645999/job/24943000965. |
@DawoudSheraz, I pushed a small workaround to avoid the error when the base image isn't tagged. I will look for a longer term solution. I pushed the commit to both this mistiped branch redwoood to the new branch redwood that is in #54 |
Description
WIP
Initial redwood support