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

End of support for Compose V1 #1431

Closed
Madscientiste opened this issue Aug 4, 2023 · 6 comments · Fixed by #1490
Closed

End of support for Compose V1 #1431

Madscientiste opened this issue Aug 4, 2023 · 6 comments · Fixed by #1490
Labels
enhancement New feature or request

Comments

@Madscientiste
Copy link

Using the install-prod.sh script on a clean installed server with latest docker fails because it checks for docker-compose command which doesn't not exist for my version (Docker version 24.0.2, build cb74dfc).

Describe the solution you'd like
replace all docker-compose by docker compose.

Should i open a PR for that ? install-demo.sh would also need to be updated

@Madscientiste Madscientiste added the enhancement New feature or request label Aug 4, 2023
@knadh
Copy link
Owner

knadh commented Aug 27, 2023

@mr-karan it's okay to switch away from docker-compose, right?

@mr-karan
Copy link
Collaborator

Yes, docker-compose is deprecated since June 2023.

@Madscientiste PR is appreciated.

@MaximilianKohler
Copy link
Contributor

MaximilianKohler commented Aug 27, 2023

@mr-karan does this mean that all instances of docker-compose in the docs https://listmonk.app/docs/ need to be changed to docker compose? Including apt install postgresql docker docker-compose?

Looking at the docker docs:

I think apt install postgresql docker docker-compose gets changed to apt install postgresql docker docker-compose-plugin?

And no changes to docker-compose.yml name or content right?

BTW, I'm no expert and guessed at the commands here https://listmonk.app/docs/upgrade/#downgrade. Could you verify it's correct?

@mr-karan
Copy link
Collaborator

Only the docker-compose CLI needs to be changed to docker compose.

I think apt install postgresql docker docker-compose gets changed to apt install postgresql docker docker-compose-plugin?

docker compose is bundled along with docker package. Specific packages that need to be installed are listed here: https://docs.docker.com/engine/install/ubuntu/#install-docker-engine

BTW, we don't have specific install instructions for Docker, the same way we don't have for PostgreSQL either. It's upto the users how they install and configure these environments before running Listmonk.

And no changes to docker-compose.yml name or content right?

Nope.

@MaximilianKohler
Copy link
Contributor

docker compose is bundled along with docker package. Specific packages that need to be installed are listed here: https://docs.docker.com/engine/install/ubuntu/#install-docker-engine

Are you sure about that? According to that link you have to run sudo apt-get install docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin. So they are explicitly installing docker-compose-plugin as well.

BTW, we don't have specific install instructions for Docker, the same way we don't have for PostgreSQL either. It's upto the users how they install and configure these environments before running Listmonk.

Yes, I see that now. I patched together a guide for newbies (now linked in the docs) with a lot of difficulty, and by asking people. I also added a bunch of docker commands to the docs for various basic usage, which I patched together from the github issues. I've been trying to improve the docs so that other people don't have to struggle with everything like I did. I think I will add the docker & postgresql install commands to the docs as well.

Everyone should be able to use listmonk without being a linux expert. Even @knadh said he's not familiar with docker commands, so it shouldn't be expected that everyone who wants to use listmonk will/should be. There needs to be basic install commands that anyone can use. Experts can always customize the commands themselves.

It sounds like you're not interested in updating the docs? I'll try to do it and hopefully you can look over the pull request to verify it.

I'm going to change sudo apt install postgresql docker docker-compose to sudo apt install postgresql docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin. Hopefully that's correct...

@mr-karan
Copy link
Collaborator

I've sent PR for replacing docker-compose with docker compose in the existing docs and installation scripts.

My point was that since we never had installation docs for docker (arguably, it's the user's preference to run with docker - so the expectation is that docker will be set up on host as well).

I've seen very few projects which actually show installation steps of the dependencies in their own docs. This is what I personally believe but @knadh can take the final call here. Anyway, that should be opened as a new issue as it's unrelated to the docker-compose / docker compose changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
4 participants