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

[BUG] docker-compose-postorius.yaml is unmaintained #605

Open
duck-rh opened this issue Feb 3, 2023 · 4 comments
Open

[BUG] docker-compose-postorius.yaml is unmaintained #605

duck-rh opened this issue Feb 3, 2023 · 4 comments
Labels

Comments

@duck-rh
Copy link

duck-rh commented Feb 3, 2023

Tag/version of Container Images
0.4.5

Quack,

First I would like to point out that the doc does not make it clear that docker-compose-postorius.yaml is postorius-only while the other one is postorius+hyperkitty and in fact I chose to use this one to be sure to also get postorius installed, fail.

Anyway, I noticed the example PostgreSQL links use postgres:// which is deprecated, but docker-compose.yaml was updated to the postgresql:// URL.

Also recently sqlalchemy was bumped and the PostgreSQL container too since this verison requires at least PG 11, but docker-compose-postorius.yaml as not updated.

I think that's all but I might have missed other differences.

I'll get back to trying to make this work. Thanks for maintaining these images.
Regards.
\_o<

@duck-rh duck-rh added the bug label Feb 3, 2023
@maxking
Copy link
Owner

maxking commented Mar 10, 2023

First I would like to point out that the doc does not make it clear that docker-compose-postorius.yaml is postorius-only while the other one is postorius+hyperkitty and in fact I chose to use this one to be sure to also get postorius installed, fail.

Yeah, we are missing specific docs related to the postorious related image.

Also, there is a lack of testing in general for the postorius image and that’s something i’ve been meaning to get to. I’ll keep this issue open to get it to it. It doesn’t get used a lot either (atleast, based on the questions I get) and for how long it remains broken before someone reports it.

I’ll try to extend the basic testing we do for other 2 container images with Postorius.

@beardedfool
Copy link

Slight embarrased, as a noob, with how long I spend trying to get that work.

Could you maybe drop a note in compose-postorius.yaml with a reference to this ticket please, for the next guy?

Cheers for all you've done here

@jakicoll
Copy link
Contributor

Just a quick note: #650 fixes this, since the String "postgres" in the docker-compose-postorious.yaml will be replaced by $DATABASE_URL_SCHEMA, which is set to "postgresql" via the .env file.

Copy link

This issue has not been updated for more than 1year

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Nov 19, 2024
@maxking maxking reopened this Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants