Skip to content

Commit

Permalink
Fix Warnings in Docker Compose (#3120)
Browse files Browse the repository at this point in the history
* update makefiles and .yml files for docker

* Add environment variables in makefiles.

* Remove  unused variable

* Move variables to Makefile.

* Remove spacing.
  • Loading branch information
ParagEkbote authored Jan 16, 2025
1 parent 5711204 commit e50cb89
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
2 changes: 2 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ export PORT_SSE_API := 8185
export PORT_WORKER := 8186
export PORT_WEBHOOK := 8187
export PORT_REVERSE_PROXY := 8100
export API_HF_JWT_PUBLIC_KEY_URL := https://hub-ci.huggingface.co/api/keys/jwt
export API_HF_JWT_ADDITIONAL_PUBLIC_KEYS :=

# environment variables per target
start: export COMPOSE_PROJECT_NAME := datasets-server
Expand Down
1 change: 1 addition & 0 deletions e2e/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ export ADMIN_UVICORN_NUM_WORKERS := 2
export ADMIN_UVICORN_PORT := 9081
export API_HF_JWT_ALGORITHM := EdDSA
export API_HF_JWT_PUBLIC_KEY_URL := https://hub-ci.huggingface.co/api/keys/jwt
export API_HF_JWT_ADDITIONAL_PUBLIC_KEYS :=
# ^ it's not tested in the e2e tests, but it's good to ensure we're able to fetch it at least.
export ADMIN_HF_TIMEOUT_SECONDS := 10
export API_HF_TIMEOUT_SECONDS := 10
Expand Down

0 comments on commit e50cb89

Please sign in to comment.