-
Notifications
You must be signed in to change notification settings - Fork 47
Open
Description
The bar tools (src/teradata_mcp_server/tools/bar/bar_tools.py) define their own log directory.
log_dir = os.path.join(os.path.dirname(__file__), '../../../logs')
This points to the package runtime and causes the Docker (and likely CLI) deployment to fail as this path would be read only (binary).
Eg Docker:
xxx:~/mcp/teradata-mcp-server$ docker-compose up
[...]
teradata-mcp-server-1 | os.makedirs(log_dir, exist_ok=True)
teradata-mcp-server-1 | ~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^
teradata-mcp-server-1 | File "<frozen os>", line 228, in makedirs
teradata-mcp-server-1 | PermissionError: [Errno 13] Permission denied: '/usr/local/lib/python3.13/site-packages/teradata_mcp_server/tools/bar/../../../logs'
teradata-mcp-server-1 exited with code 1
xxx:~/mcp/teradata-mcp-server$
Use the standard logging and log directory mechanism.
Metadata
Metadata
Assignees
Labels
No labels