-
Notifications
You must be signed in to change notification settings - Fork 8
Open
Description
Problem 1: AUTHORIZED_USERS vs config_manager
The hummingbot/condor:latest Docker image on Docker Hub requires the AUTHORIZED_USERS environment variable, but the current repository code uses the config_manager approach with ADMIN_USER_ID.
Error
When running via Docker Compose without AUTHORIZED_USERS:
ValueError: invalid literal for int() with base 10: ''
File "/app/utils/config.py", line 8
int(user_id) for user_id in os.environ.get("AUTHORIZED_USERS", "").split(",")
Expected Behavior
The Docker image should match the current repo code which uses:
ADMIN_USER_IDenvironment variableTELEGRAM_ALLOWED_IDSenvironment variableconfig_managerfor user approval via Telegram
Problem 2: Watchtower Docker API Version
The Watchtower container in docker-compose.yml uses Docker API version 1.25, which is rejected by newer Docker daemons (v24+).
Error
Error response from daemon: client version 1.25 is too old. Minimum supported API version is 1.44, please upgrade your client to a newer version
Suggested Fix
Add to the Watchtower service in docker-compose.yml:
environment:
DOCKER_API_VERSION: "1.44"---
Workarounds
- Add
AUTHORIZED_USERS=<user_id>to.env(works but shouldn't be needed) - Add
DOCKER_API_VERSION: "1.44"to Watchtower environment - Build image locally with
docker build -t hummingbot/condor:local .
Request
Please:
- Rebuild and push the Docker image to Docker Hub with the latest code (using
ADMIN_USER_IDandTELEGRAM_ALLOWED_IDSinstead ofAUTHORIZED_USERS) - Update
docker-compose.ymlto includeDOCKER_API_VERSION: "1.44"for Watchtower compatibility
Environment
- Docker: 29.1.3 (API 1.52, minimum 1.44)
- condor repo: latest main branch
- Docker image: hummingbot/condor:latest
Thanks!
Metadata
Metadata
Assignees
Labels
No labels