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

Rollover log files on startup #1216

Merged
merged 1 commit into from
Dec 18, 2020
Merged

Rollover log files on startup #1216

merged 1 commit into from
Dec 18, 2020

Conversation

pfink
Copy link
Contributor

@pfink pfink commented Dec 18, 2020

This is something that I use for my setup, as it makes investigating log files much easier because they are rollled over on startup. This means, you really start with a "clean" log file when you restart openHAB. I used to do the rollover manually in the past as I didn't had the idea that it's configurable via log4j, and probably many users often do so as well.

First, I wanted to put it in the docs, but then I thought: Why not using it as default? I think the majority of users will like it more than the current default.

Signed-off-by: Patrick Fink [email protected]

Signed-off-by: Patrick Fink <[email protected]>
@kaikreuzer kaikreuzer added the enhancement An enhancement or new feature label Dec 18, 2020
@kaikreuzer kaikreuzer added this to the 3.0.0.RC2 milestone Dec 18, 2020
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this a lot as well, so let's get that in as the default!

@kaikreuzer kaikreuzer merged commit 201350e into openhab:master Dec 18, 2020
@pfink pfink deleted the patch-17 branch December 18, 2020 21:24
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/display-log-via-tail/140170/2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants