Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logfire displays time on the terminal that does not match the system time. #710

Open
dlivxpr opened this issue Dec 20, 2024 · 2 comments
Open
Labels
bug Bug related to the Logfire Python SDK documentation Improvements or additions to documentation Feature Request

Comments

@dlivxpr
Copy link

dlivxpr commented Dec 20, 2024

Description

As the title suggests, I am using logfire while using pydantic-ai. However, when executing the code, the time displayed by logfire in the terminal is 8 hours later than the system time. I suspect it is a time zone issue, but I have checked the logfire documentation and found no time zone related settings. Is there a way to make logfire display log time in the local terminal as the system time?

Python, Logfire & OS Versions, related packages (not required)

logfire="2.8.0"
platform="Windows-11-10.0.26100-SP0"
python="3.13.1 (main, Dec  6 2024, 18:07:41) [MSC v.1942 64 bit (AMD64)]"
[related_packages]
requests="2.32.3"
pydantic="2.10.3"
fastapi="0.115.6"
openai="1.58.1"
protobuf="5.29.1"
rich="13.9.4"
executing="2.1.0"
opentelemetry-api="1.29.0"
opentelemetry-exporter-otlp-proto-common="1.29.0"
opentelemetry-exporter-otlp-proto-http="1.29.0"
opentelemetry-instrumentation="0.50b0"
opentelemetry-proto="1.29.0"
opentelemetry-sdk="1.29.0"
opentelemetry-semantic-conventions="0.50b0"
@alexmojaki
Copy link
Contributor

It intentionally uses the UTC timezone. Maybe we should make that configurable.

@Kludex
Copy link
Member

Kludex commented Dec 24, 2024

Maybe we should make that configurable.

A timezone parameter to the SimpleConsoleSpanExporter?


I don't think this should be a UTC tho, the deployment environments are usually configured to UTC. I think it should just be local time.

@Kludex Kludex added bug Bug related to the Logfire Python SDK documentation Improvements or additions to documentation Feature Request labels Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug related to the Logfire Python SDK documentation Improvements or additions to documentation Feature Request
Projects
None yet
Development

No branches or pull requests

3 participants