Skip to content

Draft: Optionally emit priority level markers on bubblewrap messages

Simon McVittie requested to merge wip/smcv/level-prefix into main
  • https://github.com/containers/bubblewrap/pull/646 (Reviewed-by: @refi64 upstream)

  • bwrap: Check availability of --level-prefix option

    We cannot rely on bwrap always supporting this option, even if our vendored copy (srt-bwrap) is known to have it, because we might be using a setuid-root system copy of bwrap on some host OSs.

  • pv-wrap: Run bwrap with --level-prefix if available

    This will let us highlight warnings and fatal errors from bwrap as such. In particular, if bwrap fails with a fatal error, putting the <3> prefix on it will ensure that it's highlighted as an error.

  • DO NOT MERGE: Patch some mock log messages into bubblewrap

    This makes steamrt/tasks#444 easier to test, but is not intended to be upstreamed or included in production releases!


Previously this MR also included similar changes for libcapsule, but I moved those to !742 (merged).

/cc @refi64 @denittis

Edited by Simon McVittie

Merge request reports