Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

DRAFT: updating daprd & dapr containers to dapr 1.12.3 #143

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 18 additions & 8 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ services:
- redis

basket-api-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
network_mode: "service:basket-api"
depends_on:
- basket-api
Expand All @@ -51,7 +51,7 @@ services:
- sqldata

catalog-api-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
network_mode: "service:catalog-api"
depends_on:
- catalog-api
Expand All @@ -65,7 +65,7 @@ services:
- sqldata

identity-api-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
network_mode: "service:identity-api"
depends_on:
- identity-api
Expand All @@ -79,7 +79,7 @@ services:
- sqldata

ordering-api-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
network_mode: "service:ordering-api"
depends_on:
- ordering-api
Expand All @@ -91,7 +91,7 @@ services:
dockerfile: src/Services/Payment/Payment.API/Dockerfile

payment-api-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
network_mode: "service:payment-api"
depends_on:
- payment-api
Expand All @@ -109,7 +109,7 @@ services:
- sqldata

webshoppingagg-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
network_mode: "service:webshoppingagg"
depends_on:
- webshoppingagg
Expand All @@ -121,7 +121,14 @@ services:
dockerfile: src/ApiGateways/Envoy/Dockerfile

webshoppingapigw-dapr:
image: "daprio/daprd:1.9.4"
image: "daprio/daprd:1.12.3"
paulyuk marked this conversation as resolved.
Show resolved Hide resolved
command: [
"./daprd",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"./daprd",
"/daprd",

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll try to sneak in this nit :)

"--app-id", "webshoppingapigw",
"--app-port", "3000",
"--placement-host-address", "dapr-placement:50006", # Dapr's placement service can be reach via the docker DNS entry
"--resources-path", "./dapr/components"
]
network_mode: "service:webshoppingapigw"
depends_on:
- webshoppingapigw
Expand All @@ -133,4 +140,7 @@ services:
dockerfile: src/Web/WebStatus/Dockerfile

dapr-placement:
image: "daprio/dapr:1.9.4"
image: "daprio/dapr:1.12.3"
command: ["./placement", "--port", "50006"]
paulyuk marked this conversation as resolved.
Show resolved Hide resolved
ports:
- "50006:50006"
paulyuk marked this conversation as resolved.
Show resolved Hide resolved