Skip to content

Conversation

@FedeDP
Copy link
Contributor

@FedeDP FedeDP commented Nov 24, 2025

Description

Adding a log-file option to broadcast logs to both stderr (or stdout) and the log file, if set.
By default, unset.
Leverages io.MultiWriter.

Note: the new log-file does not support rotation.

Changelog

new: log-file option

@FedeDP FedeDP requested a review from a team as a code owner November 24, 2025 15:53
@FedeDP FedeDP requested a review from jrfastab November 24, 2025 15:53
@netlify
Copy link

netlify bot commented Nov 24, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit d60e81d
🔍 Latest deploy log https://app.netlify.com/projects/tetragon/deploys/6925b088c5c8e50008e84f1e
😎 Deploy Preview https://deploy-preview-4378--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@FedeDP FedeDP marked this pull request as draft November 24, 2025 16:55
@FedeDP FedeDP added the release-note/minor This PR introduces a minor user-visible change label Nov 25, 2025
In that case, a multi slog Logger is set up and logs are
broadcasted to all configured handlers
(ie: both log file and stderr/stdout).

Signed-off-by: Federico Di Pierro <[email protected]>
@FedeDP FedeDP marked this pull request as ready for review November 25, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/minor This PR introduces a minor user-visible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants