Skip to content

Conversation

@vemonet
Copy link
Collaborator

@vemonet vemonet commented Nov 27, 2025

Add docker volume in docker-compose.yml for the search API data, for now mainly to make the conversations logs persistent (will need to see later how we want to store and access these logs on the long run)

…ta, for now mainly to make the conversations logs persistent
@vemonet vemonet requested review from Michal-Kolomanski and tobiasschweizer and removed request for tobiasschweizer November 27, 2025 15:53
- search-api-data:/app/data
env_file: [ keys.env ]
environment:
- RUST_LOG=info
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this still up to date (since this got refactored from Rust to Python)? I tried to set this to "debug" but I do not see any effect

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, it is not for the stdout/stderr logs (which goes to usual docker output)

It is for a JSONL file that contains the details of each request, each line is a JSON containing the following:

        json.dumps(
            {
                "timestamp": datetime.now(timezone.utc).isoformat(),
                "token_usage": token_usage.model_dump(), # token usage for this request
                "input": request.model_dump(), # Input search question (or list of messages) from the user
                "response": final_response.model_dump(), # final response of the user
            }
        )

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

which env var is for the log level in docker logs?

Copy link
Collaborator

@tobiasschweizer tobiasschweizer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works in my local setup.

@vemonet vemonet merged commit f14a4fa into main Nov 28, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants