immich-app / immich-charts

Helm chart implementation of Immich
https://immich.app
GNU Affero General Public License v3.0
107 stars 45 forks source link

Update postgresql dependency version to 12.6.9 to support arm64 #33

Closed choikangjae closed 8 months ago

choikangjae commented 1 year ago

The current version of postgresql 11.8.1 does not support arm64. Updated to the latest version 12.6.9 to support arm64.

PixelJonas commented 1 year ago

IIRC upgrading from version 11 to 12 of that chart is a breaking change, right? I remember having to dump/restore all my databases.

@choikangjae did you perform that upgrade? Otherwise, we should bump this version when we bump the chart version to another major release and indicate a breaking change in our README.

https://artifacthub.io/packages/helm/bitnami/postgresql#to-12-0-0 https://www.postgresql.org/docs/15/upgrading.html

bo0tzz commented 1 year ago

This chart update also updates the postgresql major version from 14 to 15. A major postgres upgrade needs manual intervention, and on top of that Immich is currently still using postgres 14. I think we should hold off on this upgrade until the main Immich project goes through with it as well, so the required steps can be communicated through the usual channels.

choikangjae commented 1 year ago

Thank you for the replies. It was the fresh installation for me so haven't aware of the breaking change. I think adding one line of NOTICE to ARM64 Users on README would work for now.

PixelJonas commented 11 months ago

@bo0tzz FYI the bitnami postgresql chart is now running v16 postgresql and still does not support direct updating. 😢

bo0tzz commented 8 months ago

Superseded by #53