From dd4f47dd98cfa823cc081cd386b88c00c215c26d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dan=20=C4=8Cerm=C3=A1k?= Date: Tue, 1 Aug 2023 23:53:42 +0200 Subject: [PATCH] Always show RemoteSocket.Exists in json MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The `Exists` field of the `RemoteSocket` struct is marshaled to json with the `omitempty` setting. This has the disadvantage that by default `podman info` shows a `remotePath` entry (the remote path is set in `pkg/domain/infra/abi/systems.go`: `(*ContainerEngine).Info`) but not that this path does not exist: ``` ❯ podman info --format json | jq .host.remoteSocket { "path": "/run/user/1000/podman/podman.sock" } ``` By removing the `omitempty`, we ensure that the existence is always shown: ``` ❯ bin/podman info --format json | jq .host.remoteSocket { "path": "/run/user/1000/podman/podman.sock", "exists": false } ``` Signed-off-by: Dan Čermák --- libpod/define/info.go | 2 +- test/e2e/info_test.go | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libpod/define/info.go b/libpod/define/info.go index 9354ae31a9..4ba718afd6 100644 --- a/libpod/define/info.go +++ b/libpod/define/info.go @@ -68,7 +68,7 @@ type HostInfo struct { // RemoteSocket describes information about the API socket type RemoteSocket struct { Path string `json:"path,omitempty"` - Exists bool `json:"exists,omitempty"` + Exists bool `json:"exists"` } // SlirpInfo describes the slirp executable that is being used diff --git a/test/e2e/info_test.go b/test/e2e/info_test.go index 93f7a46f2a..7714c1abc2 100644 --- a/test/e2e/info_test.go +++ b/test/e2e/info_test.go @@ -118,6 +118,8 @@ var _ = Describe("Podman Info", func() { Expect(session).Should(Exit(0)) if IsRemote() { Expect(session.OutputToString()).To(ContainSubstring("true")) + } else { + Expect(session.OutputToString()).To(ContainSubstring("false")) } })