From fdc20de500452d16d9611e42d473fbd7dc1aa0ae Mon Sep 17 00:00:00 2001 From: Rodrigo Campos Date: Fri, 19 Jan 2024 11:33:55 +0100 Subject: [PATCH] kubelet/userns: Wrap error message Most error messages are properly wrapped already, but this was missing. The kubelet logs will show something like this now: E0201 12:00:03.505680 3007049 run.go:74] "command failed" err="failed to run Kubelet: failed to create kubelet: record pod mappings: create user namespace store: mkdir XXX: permission denied" Before this commit, the message was not so clear: E0120 16:02:40.484404 474711 run.go:74] "command failed" err="failed to run Kubelet: failed to create kubelet: mkdir XXX: permission denied" Signed-off-by: Rodrigo Campos --- pkg/kubelet/userns/userns_manager.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/kubelet/userns/userns_manager.go b/pkg/kubelet/userns/userns_manager.go index 95b08184c95..662479e6d96 100644 --- a/pkg/kubelet/userns/userns_manager.go +++ b/pkg/kubelet/userns/userns_manager.go @@ -157,7 +157,7 @@ func MakeUserNsManager(kl userNsPodsManager) (*UsernsManager, error) { for _, podUID := range found { klog.V(5).InfoS("reading pod from disk for user namespace", "podUID", podUID) if err := m.recordPodMappings(podUID); err != nil { - return nil, err + return nil, fmt.Errorf("record pod mappings: %w", err) } }