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

Update backend deps: Django 4, Postgres 15 #263

Merged
merged 5 commits into from
Nov 17, 2023
Merged

Update backend deps: Django 4, Postgres 15 #263

merged 5 commits into from
Nov 17, 2023

Conversation

rjruizes
Copy link
Contributor

This PR upgrades several Python dependencies used by the backend, importantly Django 3 to 4 and Postgres 9 to 15.

The mallard staging environment was previously undocumented, so I added a doc for that, along with instructions on updating production to the new Postgres version. This code is running in mallard now.

@rjruizes rjruizes requested a review from AstridKery August 25, 2023 19:35
@AstridKery
Copy link
Contributor

@rjruizes would you mind confirming that changing the first line in the Dockerfile from

FROM python:3.8-buster to FROM --platform=linux/amd64 python:3.8-buster does not break for your computer?

@rjruizes
Copy link
Contributor Author

Verified functional with change to platform in FROM tag and Postgres volume name change.

@AstridKery
Copy link
Contributor

@peridoteagle I did a pass and it seems to work locally on my machine. I also ran the backend tests just to make sure there weren't any weird things from updating the backend dependencies and didn't spot any errors. Would you mind testing on Mallard?

Copy link
Contributor

@AstridKery AstridKery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is ready to go

@AstridKery AstridKery merged commit 086caf4 into dev Nov 17, 2023
3 checks passed
@rjruizes rjruizes deleted the pg-upgrade branch October 1, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants