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

Support a file logging option #6821

Closed
1 of 2 tasks
dnjscksdn98 opened this issue Dec 10, 2024 · 3 comments
Closed
1 of 2 tasks

Support a file logging option #6821

dnjscksdn98 opened this issue Dec 10, 2024 · 3 comments
Labels
I5-enhancement An additional feature request. I10-unconfirmed Issue might be valid, but it's not yet known.

Comments

@dnjscksdn98
Copy link

dnjscksdn98 commented Dec 10, 2024

Is there an existing issue?

  • I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

  • This is not a support question.

Motivation

Currently, there is no execution option to enable file logging. For example storing stdout or stderr to a certain text file. Without this option, it's quite difficult to examine past logs. It seems that Geth already have an option like this. https://geth.ethereum.org/docs/fundamentals/logs

Request

I suggest an execution option that takes a path to a certain text file, for example, --log-path <file path>. When enabled it will sequentially store logs occurred by the node. Log rotation should also be included to prevent massive log file size.

Solution

No response

Are you willing to help with this request?

Yes!

@dnjscksdn98 dnjscksdn98 added the I5-enhancement An additional feature request. label Dec 10, 2024
@github-actions github-actions bot added the I10-unconfirmed Issue might be valid, but it's not yet known. label Dec 10, 2024
@alindima
Copy link
Contributor

https://geth.ethereum.org/docs/fundamentals/logs

Geth here just suggests redirecting the output to some file of your choice using the classic >> terminal syntax.

I think doing all of this, especially rotating logs, is out of scope for polkadot and can be easily done with external tooling

@bkchr
Copy link
Member

bkchr commented Dec 10, 2024

Yeah why not just use the normal tooling that your distribution etc provides for this? It already supports all the features you are requesting.

@dnjscksdn98
Copy link
Author

Thanks for quick responses. I'll look around for it then 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I5-enhancement An additional feature request. I10-unconfirmed Issue might be valid, but it's not yet known.
Projects
None yet
Development

No branches or pull requests

3 participants