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

Adjust postgres image refs in examples to a well supported version #1764

Open
MegadoomerX opened this issue Jun 7, 2022 · 6 comments · May be fixed by #2310
Open

Adjust postgres image refs in examples to a well supported version #1764

MegadoomerX opened this issue Jun 7, 2022 · 6 comments · May be fixed by #2310
Assignees
Labels
3. to review enhancement examples Compose/Dockerfile/etc good first issue integration: database Integration with any Nextcloud supported database

Comments

@MegadoomerX
Copy link

I find it curious that the Docker Compose examples contain a fixed version tag for MariaDB (mariadb:10.5) while a rolling release tag was chosen for the PostgreSQL container (postgres:alpine).

This is especially strange to me, as MariaDB is developed to be mostly cross-compatible in versions while PostgreSQL needs a manual database dump and rebuild on every major update. Also, the current MariaDB version 10.5 is out of date (currently 10.8.3 is the latest release).

Is there any reasoning for this that I'm unaware of?

@martadinata666
Copy link

there used to be some issue with 10.6 #1492 , so it tagged for fixed version 10.5, but you can use latest or 10.8 nowadays.

@joshtrichards
Copy link
Member

Hi @MegadoomerX - No. They're just examples. It takes a lot of work keeping them all up-to-date. It's also expected that those using them educate themselves on Docker basics and some thought into their environment's specific requirements. There's only so much we can do. :-) That said, this is a community project so you're welcome to submit PRs to try to clean some of them up a bit.

@joshtrichards joshtrichards added docs examples Compose/Dockerfile/etc labels Aug 20, 2023
@joshtrichards joshtrichards added integration: database Integration with any Nextcloud supported database enhancement and removed docs labels Oct 23, 2023
@joshtrichards joshtrichards changed the title Docker Compose examples Database Container versions Adjust postgres image refs in examples to a well supported version Jun 20, 2024
@joshtrichards
Copy link
Member

joshtrichards commented Jun 20, 2024

We should probably just lock all the postgres examples to 15 at the moment, since that's commonly supported across all maintained Nextcloud Server versions at the moment (and to be consistent with how we handle MariaDB).

@J0WI
Copy link
Contributor

J0WI commented Oct 11, 2024

MariaDB was changed to a LTS tag since they introduces STS versions. PostgreSQL does not have any STS versions and the latest version of Nextcloud usually supports all maintained releases.

@joshtrichards
Copy link
Member

PostgreSQL does not have any STS versions and the latest version of Nextcloud usually supports all maintained releases.

The latest postgres (17) isn't officially supported in any currently supported version of Nextcloud Server.

The other reason to lock things, perhaps more important and which is unique to postgres, is that the there is no "simple upgrade path" for their image between major versions. See docker-library/postgres#37

For #2310 I picked v15 because it's the highest in-common across Nc v28-v30.

@J0WI
Copy link
Contributor

J0WI commented Oct 24, 2024

The latest postgres (17) isn't officially supported in any currently supported version of Nextcloud Server.

TBH, I think it's just a question of updating the documentation for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement examples Compose/Dockerfile/etc good first issue integration: database Integration with any Nextcloud supported database
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants