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

changes to docker-compose.yml to work with 1.17.1 #166

Closed
anewton1998 opened this issue Jun 1, 2019 · 3 comments
Closed

changes to docker-compose.yml to work with 1.17.1 #166

anewton1998 opened this issue Jun 1, 2019 · 3 comments

Comments

@anewton1998
Copy link

Installing on Ubuntu, it comes with docker-compose 1.17.1.

I had to change my docker-compose.yml file to work with this version. It is below:

version: '3.3'

services:
  db:
    image: postgres
    restart: always
    environment:
      POSTGRES_USER: rrr
      POSTGRES_PASSWORD: rrr
      POSTGRES_DB: rrr
    healthcheck:
      test: ["CMD-SHELL", "pg_isready -U rrr"]
      interval: 5s
      timeout: 5s
      retries: 20
  lemmy:
    image: dessalines/lemmy:latest
    ports:
      - "8536:8536"
    environment:
      LEMMY_FRONT_END_DIR: /app/dist
      DATABASE_URL: postgres://rrr:rrr@db:5432/rrr
      JWT_SECRET: emdashg
      HOSTNAME: rrr
    restart: always
    depends_on: 
      - db 
@dessalines
Copy link
Member

I can't use v3 cause it doesn't work with the health check: peter-evans/docker-compose-healthcheck#3 (comment)

What was the error with the other stuff? BTW I'm on docker-compose version 1.24.0

@anewton1998
Copy link
Author

With the file as is, it claimed it didn't understand the service lemmy. When I changed the version to 3.3, it then said depends_on had to be an array.

As Ubuntu uses a down-level version of docker-compose, perhaps the thing to do is close this issue so others may find it (if they have the problem). I would assume manual install of a later version of docker-compose would work just fine. I know I've done that on other Ubuntu systems in the past.

@dessalines
Copy link
Member

Ya dang, it looks like v2.4 requires at least docker 17.12.0. I'll add docker version requirements to the readme.

dessalines added a commit that referenced this issue Sep 13, 2019
iav pushed a commit to iav/lemmy that referenced this issue Jun 5, 2020
eiknat pushed a commit to eiknat/lemmy that referenced this issue Aug 29, 2020
Fix/emoji suggestions

Closes LemmyNet#152, LemmyNet#166, LemmyNet#168, and LemmyNet#173

See merge request chapo-sandbox/lemmy!311
dessalines pushed a commit that referenced this issue Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants