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

meta-lxatac-bsp/rauc: Enable event log #142

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

SmithChart
Copy link
Member

This enables rauc event logging for all types of events known to rauc. With this log we can examine boots, mark-good decisions after boot and a history of installed bundles.

The all-events.log file is created inside the data directory in /srv/rauc/.
There is no rotation configured for this file - so history is stored forever. This will add around 350 Bytes of log per boot and 900 Bytes of log per install.
Thus, for a reasonable use-case the log file should stay well below a few MB.

@SmithChart SmithChart requested a review from hnez June 7, 2024 07:41
Copy link
Collaborator

@ejoerns ejoerns left a comment

Choose a reason for hiding this comment

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

There is a slight discrepancy between the logger name ('install', which is a distinct event type) and the file name used ('all' events). Apart from this, change looks good to me. 👍

This enables rauc event logging for all types of events known to rauc.
With this log we can examine boots, mark-good decisions after boot and a
history of installed bundles.

The `all-events.log` file is created inside the data directory in
`/srv/rauc/`.
There is no rotation configured for this file - so history is stored
forever. This will add around 350 Bytes of log per boot and 900 Bytes of
log per install.
Thus, for a reasonable use-case the log file should stay well below a
few MB.

Signed-off-by: Chris Fiege <[email protected]>
@SmithChart
Copy link
Member Author

There is a slight discrepancy between the logger name ('install', which is a distinct event type) and the file name used ('all' events). Apart from this, change looks good to me. 👍

Thanks for catching that. I've changed the name of the logger to log.all-log.

@hnez hnez merged commit 366a2cd into linux-automation:scarthgap Jun 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants