Skip to content
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

Revert "build!: upgrade edxapp to Python 3.11" #29

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

UsamaSadiq
Copy link
Member

Reverts #28

@UsamaSadiq UsamaSadiq marked this pull request as ready for review June 3, 2024 08:58
@UsamaSadiq UsamaSadiq merged commit 3b26f5e into master Jun 3, 2024
4 checks passed
@UsamaSadiq UsamaSadiq deleted the revert-28-edxapp-python311-upgrade branch June 3, 2024 09:05
@ormsbee
Copy link

ormsbee commented Jun 5, 2024

😞 Can you share what it was that broke when you folks tried this latest upgrade? (I'm trying to figure out if there are any fixes that need to be backported into Redwood.

Thank you.

@ormsbee
Copy link

ormsbee commented Jun 5, 2024

Got this from @jristau1984 in Slack:

There was a deploy that happened right before 3.11 that caused an issue and the two were muddled together during the incident investigation.

We did see a spike in xblock related errors for the 20 min it was live,. The Arbi-bom team is looking into that before re-enabling, but a root cause/fix is not yet known.

@robrap
Copy link

robrap commented Jun 5, 2024

@UsamaSadiq and all: A reminder that all revert PRs should include as much information as possible about why we are reverting. This could include error details, etc. This will help both the community as well as our future selves, or others in 2U. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants