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

[Feature]: Add an command line option to enable stdout log ouput #670

Closed
1 task done
Phoenix500526 opened this issue Mar 4, 2024 · 1 comment · Fixed by #686
Closed
1 task done

[Feature]: Add an command line option to enable stdout log ouput #670

Phoenix500526 opened this issue Mar 4, 2024 · 1 comment · Fixed by #686
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@Phoenix500526
Copy link
Collaborator

Description about the feature

Currently, Xline only supports redirecting logs to log files. The location of the log file and its rotation settings are determined by the command line parameters --log-file and --log-rotate. However, this approach may not be a good practice in a Kubernetes environment. The reason is that Kubernetes containers use overlayfs, and frequent logging operations can lead to unnecessary performance impacts. Additionally, in Kubernetes environments, kubectl logs typically retrieves logs from stdout and stderr. Redirecting logs to log files can affect the convenience of maintenance and debugging. Therefore, we need to support redirecting log information to stdout.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Phoenix500526 Phoenix500526 added enhancement New feature or request good first issue Good for newcomers labels Mar 4, 2024
@GFX9
Copy link
Contributor

GFX9 commented Mar 4, 2024

Let me try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants