-
Notifications
You must be signed in to change notification settings - Fork 2
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
⬆️ django 4.2 #2976
⬆️ django 4.2 #2976
Conversation
3fe9c24
to
bb7348e
Compare
6e1a103
to
4705091
Compare
c94044d
to
d188f8b
Compare
There seems to be a bug with running spatialite 5 and django 4.2, that I'm seeing in my dev environment. Details here: |
fedc3bb
to
92bd117
Compare
I've reached out in a few places to get some guidance on this: |
It sounds like updating to Spatialite 5.1 would fix the issue: https://stackoverflow.com/a/78343541/173630 Now we just have to figure out how to set that up and build it on github actions and jenkins. Also, this upstream patch fixes the issue for spatialite 5.0.1: django/django#18083 |
1ee45a3
to
1c81137
Compare
ae44c05
to
8eaef88
Compare
@ndittren this should be good to deploy as long as we avoid the spatialite versions 5.0 / 5.0.1. Production is currently on ubuntu 20, which uses spatialite 4.3. In my dev environment I do have spatialite 5.0.1, and I'm working around this bug by manually applying this patch in django's code in my venv: django/django#18083 Moving forward, Ubuntu 24 is about to be released, and comes with Spatialite 5.1, and this bug is apparently fixed there. |
7c06030
to
83a3742
Compare
@nikolas Okay looks good to me. Footprints prod currently has an unhealthygroup alarm on cloudwatch. Should we wait for that to clear up? |
I think this can go through now that the alarm is green and footprints is fixed. |
No description provided.