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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unify timestamp field names #3099

Open
Rohlik opened this issue Apr 24, 2024 · 0 comments
Open

Unify timestamp field names #3099

Rohlik opened this issue Apr 24, 2024 · 0 comments

Comments

@Rohlik
Copy link

Rohlik commented Apr 24, 2024

Hello Grafana guys 馃憢馃徏,
We used the Grafana helm chart and noticed that timestamp representation differs between the grafana-sc-dashboard and the grafana containers. When you have json logging enabled then your logs probably look like this:

# grafana-sc-dashboard
{"time": "2024-04-24T09:09:26.606514+00:00", "taskName": null, "msg": "Found a folder override annotation, placing the metrics-dashboard-15.1.0-dev1 in: /tmp/dashboards/dev1", "level": "INFO"}
# grafana
{"level":"info","logger":"settings","msg":"Config overridden from Environment variable","t":"2024-04-23T06:59:14.858660879Z","var":"GF_PATHS_LOGS=/var/log/grafana"}

You have a t field in one container and for the next one, time. Log parsing is unnecessarily complicated for some log collectors like FluentD when you count with time (default key name for the time field), but you have t 馃槥.

Can you please consider to rename that field t to time 馃檹馃徏?

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

No branches or pull requests

1 participant