-
Notifications
You must be signed in to change notification settings - Fork 600
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
[NEW] Output logs as JSON #1006
Comments
There was a previous PR in Redis where we discussed some log format:
More information as well
@azuredream I see you opened the old PR to Redis, would you be willing to also contribute your change to Valkey? |
It's very timely for you to open this issue, we talked about this morning as thinking about revamping logs. Can you add a bit more context about what JSON format you want? Either a similar system that outputs in a format that is a bit of a standard. |
Of course you can have any other fields you want in the JSON such as PID , role , etc... the ISO8601 timestamping is particularly important, because TZ Offset , as we have many geographical locations and need to be able to canonize/reconciliate timestamp in a single timezone wherever the logs are consolidated. The ability to also specify the TZ offset in a particular location is also important in case we are asked to log everything in UTC. Also the ability to log to a local file and/or to STDOUT is very important ( in Kubernetes) and thus to be able to choose either or both from the config file of Valkey as an option. Something along these lines |
Would it be possible to get Redis logs output in JSON format with a timestamp that has timezone information as the first field.
This is useful when integrating and using redis in large kubernetes based systems which ingest and demand JSON logs to stdout for all apps.
The text was updated successfully, but these errors were encountered: