Conversation
|
Does |
Yes, any time the volumes are removed the users will need to run alembic upgrade again |
Ah, I misunderstood. I thought you meant "for this PR to work then a fresh backend must be built", not "if a fresh backend is built, alembic has to be run" |
|
For clarity, I ran |
shilorigins
left a comment
There was a problem hiding this comment.
This seems to work for me, so let's get it merged in.
|
Realizing that I shouldn't have manually changed the values for I may also want to spend some time learning how to properly use alembic. |
Description
Fix alembic versioning. #44 accidentally introduced 2 alembic heads.
Just fixing the versioning wasn't enough though as #56 was still preventing the backend from starting up properly. Temporarily remove the auto-updating for alembic.
Users will need to run
docker exec squirrel-api alembic upgrade headon a fresh backend build again.Motivation
Some Alembic issue introduced in #44 & #56.