summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGiuseppe Scrivano <gscrivan@redhat.com>2021-12-17 14:35:57 +0100
committerGiuseppe Scrivano <gscrivan@redhat.com>2021-12-17 14:47:18 +0100
commita837984c46a326e85a030da3f15eda57e45d6709 (patch)
treeac748935b6ad5639d5da64856279e4b03b6d5d7c
parentd984fec351c06a95f8a51c7e6ced819c6b17245f (diff)
downloadpodman-a837984c46a326e85a030da3f15eda57e45d6709.tar.gz
podman-a837984c46a326e85a030da3f15eda57e45d6709.tar.bz2
podman-a837984c46a326e85a030da3f15eda57e45d6709.zip
rootless: include the args in the debug message
include the arguments used to create the user namespace to help debugging. [NO NEW TESTS NEEDED] it changes a debug message Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
-rw-r--r--pkg/rootless/rootless_linux.go4
1 files changed, 2 insertions, 2 deletions
diff --git a/pkg/rootless/rootless_linux.go b/pkg/rootless/rootless_linux.go
index 3e81d5c14..92725adc7 100644
--- a/pkg/rootless/rootless_linux.go
+++ b/pkg/rootless/rootless_linux.go
@@ -145,8 +145,8 @@ func tryMappingTool(uid bool, pid int, hostID int, mappings []idtools.IDMap) err
}
if output, err := cmd.CombinedOutput(); err != nil {
- logrus.Debugf("error from %s: %s", tool, output)
- return errors.Wrapf(err, "cannot setup namespace using %s", tool)
+ logrus.Errorf("error running `%s`: %s", strings.Join(args, " "), output)
+ return errors.Wrapf(err, "cannot setup namespace using %q", path)
}
return nil
}