You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm not sure if holding sql files inside the chart is a good idea as it will become out of date quickly.
If you can propose (or even better - implement) some solution to make it not become out of date
I'm not sure if holding sql files inside the chart is a good idea as it will become out of date quickly.
This issue never really crossed my mind.
I've been had the SQL files as a ConfigMap in a local Helm chart for several months without encountering any issues. It's also worth noting that the last change to the Invidious SQL schema was made over two years ago (see iv-org/invidious@cdc007c).
On the other hand, I'm not too sure how to handle migrations for the database if they ever become necessary in the future. Perhaps a solution could be implementing a custom migration script that is triggered by a Helm hook?
Invidious does indeed have a built-in migration system, which to this date has not been used because a proper init and upgrade process for docker needs to be documented.
We have a bunch of PRs making DB changes that are waiting to be merged.
Originally posted by @JuniorJPDJ in #1 (comment)
CC: @perennialtech
The text was updated successfully, but these errors were encountered: