monogon-dev / monogon

The Monogon Monorepo. May contain traces of peanuts and a ✨pure Go Linux userland✨. Work in progress!
https://monogon.tech
Apache License 2.0
378 stars 8 forks source link

supervisor: log runnable restarts into their own log DNs #296

Open lorenz opened 3 months ago

lorenz commented 3 months ago

Currently the supervisor logs runnable restarts into its own supervisor DN. This hides runnable restarts from DN-filtered logs, potentially confusing people who look at the logs as a lot of our runnables do not log errors, instead relying on the supervisor to log them as the reason for restarting the runnable. It would be a much better experience if these were inline with the actual runnable logs. For example #295 can only really be understood by looking at both the original DN logs as well as the supervisor's.

q3k commented 3 months ago

Yeah, I've been meaning to fix that for a while. We should just change this behaviour.