lxc: Fix reporting of startup errors with debug logging enabled

[PATCH 0/3] lxc: Fix reporting of startup errors with debug logging enabled

Peter Krempa posted 3 patches 9 months, 1 week ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1690961331.git.pkrempa@redhat.com

src/lxc/lxc_controller.c |  8 +++++---
src/lxc/lxc_process.c    | 12 ++++++++++++
2 files changed, 17 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH 0/3] lxc: Fix reporting of startup errors with debug logging enabled

Posted by Peter Krempa 9 months, 1 week ago

Few issues with propagating error to the user when debug logging is
enabled,

Peter Krempa (3):
  virLXCControllerSetupUsernsMap: Modify debug logging for clean startup
    errors
  virLXCProcessReadLogOutputData: Refill buffer after filtering out
    noise
  virLXCProcessReportStartupLogError: Strip trailing newline from error

 src/lxc/lxc_controller.c |  8 +++++---
 src/lxc/lxc_process.c    | 12 ++++++++++++
 2 files changed, 17 insertions(+), 3 deletions(-)

-- 
2.41.0

Re: [PATCH 0/3] lxc: Fix reporting of startup errors with debug logging enabled

Posted by Martin Kletzander 9 months, 1 week ago

On Wed, Aug 02, 2023 at 09:29:39AM +0200, Peter Krempa wrote:
>Few issues with propagating error to the user when debug logging is
>enabled,
>
>Peter Krempa (3):
>  virLXCControllerSetupUsernsMap: Modify debug logging for clean startup
>    errors
>  virLXCProcessReadLogOutputData: Refill buffer after filtering out
>    noise
>  virLXCProcessReportStartupLogError: Strip trailing newline from error
>
> src/lxc/lxc_controller.c |  8 +++++---
> src/lxc/lxc_process.c    | 12 ++++++++++++
> 2 files changed, 17 insertions(+), 3 deletions(-)
>

Reviewed-by: Martin Kletzander <mkletzan@redhat.com>

>-- 
>2.41.0
>