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

Troubleshooting: Support for JSON formatted logs #2909

Closed
jinleileiking opened this issue Feb 11, 2022 · 3 comments
Closed

Troubleshooting: Support for JSON formatted logs #2909

jinleileiking opened this issue Feb 11, 2022 · 3 comments
Assignees
Labels
Feature It's a new feature. TransByAI Translated by AI/GPT. Won't fix We won't fix it.

Comments

@jinleileiking
Copy link

jinleileiking commented Feb 11, 2022

The current srs.log is in text format. Let's see if there is a need to output it as JSON. The company is quite supportive of providing feedback to the community. If you agree, I will mention it.

TRANS_BY_GPT3

@winlinvip

This comment was marked as resolved.

@winlinvip winlinvip self-assigned this Feb 12, 2022
@winlinvip winlinvip added the Deleting Will deleted in future. label Feb 12, 2022
@jinleileiking

This comment was marked as off-topic.

@winlinvip
Copy link
Member

winlinvip commented Feb 18, 2022

Sorry, it is #2899 supporting Prometheus's API for troubleshooting implementation issues.

In troubleshooting, the main approach is to collect data through instrumentation like Prometheus, and then search for logs only when detailed logs are needed.

When searching for logs, the most suitable method is to use the grep command with the context ID. It is not suitable for JSON logs, but rather for text logs.

Generally, JSON-formatted logs are key logs for tracking, suitable for integration with Prometheus system, rather than directly converting all logs into JSON and filtering them.

Log troubleshooting is definitely necessary. I welcome discussing this solution and currently, I am more inclined towards using Prometheus with the current text log troubleshooting. Could you please look into it and see if it is suitable? I welcome your collaboration in implementing it.

TRANS_BY_GPT3

@winlinvip winlinvip added Feature It's a new feature. Won't fix We won't fix it. and removed Deleting Will deleted in future. labels Feb 18, 2022
@winlinvip winlinvip changed the title FEAT: log输出为json 问题排查:支持JSON格式的日志 Feb 18, 2022
@winlinvip winlinvip changed the title 问题排查:支持JSON格式的日志 Troubleshooting: Support for JSON formatted logs Jul 28, 2023
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature It's a new feature. TransByAI Translated by AI/GPT. Won't fix We won't fix it.
Projects
None yet
Development

No branches or pull requests

2 participants