Merge pull request #212 from SFTtech/milo/remove-docker-compose-links

chore: remove not recommended (and not needed) docker compose links
SFTtech · Mar 2, 2024 · 8906064 · 8906064
2 parents ccb3abb + 10f1c87
commit 8906064
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 19 deletions.
diff --git a/docker-compose.devel.yaml b/docker-compose.devel.yaml
@@ -12,8 +12,6 @@ services:
     depends_on:
       postgres:
         condition: service_healthy
-    links:
-      - postgres
 
   frontend-dev:
     build:
@@ -23,17 +21,12 @@ services:
     depends_on:
       api:
         condition: service_healthy
-    links:
-      - api
 
   nginx:
     image: docker.io/nginx:stable-alpine
     depends_on:
       api:
         condition: service_healthy
-    links:
-      - api
-      - frontend-dev
     volumes:
       - "./docker/nginx-dynamic:/etc/nginx/conf.d/default.conf"
     ports:
@@ -55,9 +48,6 @@ services:
     depends_on:
       api:
         condition: service_healthy
-    links:
-      - postgres
-      - "mailhog:mail"
 
   cron:
     build:
@@ -72,9 +62,6 @@ services:
       ABRECHNUNG_EMAIL__HOST: mail
       ABRECHNUNG_EMAIL__PORT: 1025
       ABRECHNUNG_EMAIL__MODE: smtp
-    links:
-      - postgres
-      - "mailhog:mail"
 
   postgres:
     image: docker.io/postgres:14-alpine

diff --git a/docker-compose.prod.yaml b/docker-compose.prod.yaml
@@ -10,21 +10,15 @@ services:
     depends_on:
       api:
         condition: service_healthy
-    links:
-      - api
 
   mailer:
     extends:
       file: docker-compose.base.yaml
       service: mailer
     image: quay.io/abrechnung/api:latest-release
-    links:
-      - api
 
   cron:
     extends:
       file: docker-compose.base.yaml
       service: cron
     image: quay.io/abrechnung/api:latest-release
-    links:
-      - api