From 6c356a5c75208d970f886b2f14b7dc5b58a0ed86 Mon Sep 17 00:00:00 2001 From: Rodrigo Campos Date: Wed, 7 Feb 2024 10:54:53 +0100 Subject: [PATCH] cri: Expose userns in PodSandboxStatus rpc We added support for userns but we weren't showing it in the podSandboxStatus. Let's just show the whole nsOpts, so we don't forget in the future either if something else inside there changes. Please note that this will expose the content of nsOpts.TargetId that we weren't exposing before. But that seemed like a bug to me. Signed-off-by: Rodrigo Campos Maybe this is better? --- internal/cri/server/sandbox_status.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/internal/cri/server/sandbox_status.go b/internal/cri/server/sandbox_status.go index b0c1f728f..0f4da8a31 100644 --- a/internal/cri/server/sandbox_status.go +++ b/internal/cri/server/sandbox_status.go @@ -134,11 +134,7 @@ func toCRISandboxStatus(meta sandboxstore.Metadata, status string, createdAt tim }, Linux: &runtime.LinuxPodSandboxStatus{ Namespaces: &runtime.Namespace{ - Options: &runtime.NamespaceOption{ - Network: nsOpts.GetNetwork(), - Pid: nsOpts.GetPid(), - Ipc: nsOpts.GetIpc(), - }, + Options: nsOpts, }, }, Labels: meta.Config.GetLabels(),