Skip to content

Commit

Permalink
docker configured for removing jaeger
Browse files Browse the repository at this point in the history
  • Loading branch information
ozgurkara committed Jan 30, 2024
1 parent b6b5ee0 commit bd3ef40
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 10 deletions.
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,5 @@ COPY main.py src
WORKDIR src
RUN pip install -r requirements.txt

EXPOSE 8081
CMD ["uvicorn", "main:app", "--host", "0.0.0.0", "--port", "8081"]
EXPOSE 8080
CMD ["uvicorn", "main:app", "--host", "0.0.0.0", "--port", "8080"]
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ This architecture;
There are ready implementations;
* Redis cache
* Swagger (http://0.0.0.0:8080)
* Opentracing via Jaeger (http://0.0.0.0:16686/)


# How to add the new use case?
Expand Down
7 changes: 0 additions & 7 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,20 +13,13 @@ services:
- .env
depends_on:
- redis
- jaeger
command: uvicorn main:app --host 0.0.0.0 --port 8080 --reload
environment:
REDIS_HOST: redis
JAEGER_HOST: jaeger
redis:
restart: always
image: redis:5.0.7
ports:
- "6379:6379"
volumes:
- .data/db:/data
jaeger:
image: jaegertracing/all-in-one:latest
ports:
- "5775:5775/udp"
- "16686:16686"

0 comments on commit bd3ef40

Please sign in to comment.