fix: remove venv volume mount conflicting with entrypoint.sh #49
Reference in New Issue
Block a user
No description provided.
Delete Branch "fix/remove-venv-mount"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Removes the
/mnt/HoardingCow_docker_data/Hermes/venv:/opt/hermes/.venvvolume mount from the hermes service.Problem: This volume mount overrides the container's built-in
.venvwith whatever is on the host. On a fresh container start or after a clean build, an empty/missing host directory causes/opt/hermes/docker/entrypoint.shline 62 (source .venv/bin/activatewithset -e) to fail with:The old inline entrypoint (
bash /opt/data/hermes-tools/install.sh && . /opt/hermes/.venv/bin/activate && uv pip install ...) masked this because it sourced the venv before reaching entrypoint.sh. The cleaned-up entrypoint (fromfix/hermes-build) removed that guard, exposing the latent issue.Fix: The Docker image already builds a complete venv in
/opt/hermes/.venv— persisting it on the host was unnecessary and fragile.