Skip to content

Commit

Permalink
- version removed from docker-compose yaml
Browse files Browse the repository at this point in the history
- `/healthcheck` route shortened to `/health`
  • Loading branch information
bulletinmybeard committed Mar 30, 2024
1 parent 9660663 commit 3b98176
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion audit_logger/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ def search_audit_log_entries(
raise HTTPException(status_code=500, detail="Failed to query audit logs")


@app.get("/healthcheck", response_class=JSONResponse)
@app.get("/health", response_class=JSONResponse)
async def health_check() -> Dict[str, str]:
"""
Health check endpoint used by Docker to check if the Elasticsearch instance/host is ready.
Expand Down
1 change: 0 additions & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
version: '3'
services:
audit-logger:
container_name: audit-logger
Expand Down
2 changes: 1 addition & 1 deletion docker/audit-logger/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ HEALTHCHECK \
--timeout=10s \
--start-period=1m \
--retries=3 \
CMD if [ "$ENABLE_HEALTHCHECK" = "true" ]; then curl --fail http://127.0.0.1:8000/healthcheck || exit 1; else exit 0; fi
CMD if [ "$ENABLE_HEALTHCHECK" = "true" ]; then curl --fail http://127.0.0.1:8000/health || exit 1; else exit 0; fi

COPY pyproject.toml poetry.lock ./
RUN \
Expand Down

0 comments on commit 3b98176

Please sign in to comment.