diff options
author | Valentin Rothberg <vrothberg@redhat.com> | 2022-06-10 12:50:27 +0200 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2022-06-23 09:11:57 -0400 |
commit | 15188dce0566ffcba9e7da6fbde69625f49e0e16 (patch) | |
tree | fa06a1f4c3251bca781f204dbe63a70f5a20654b /libpod | |
parent | 8e88abda85f7bf44b6857ad5d62c8ef58206fce5 (diff) | |
download | podman-15188dce0566ffcba9e7da6fbde69625f49e0e16.tar.gz podman-15188dce0566ffcba9e7da6fbde69625f49e0e16.tar.bz2 podman-15188dce0566ffcba9e7da6fbde69625f49e0e16.zip |
conmon: silence json-file error
We should just silently fall through. The log was flooding the
system-service logs when running Gitlab runner.
Signed-off-by: Valentin Rothberg <vrothberg@redhat.com>
Diffstat (limited to 'libpod')
-rw-r--r-- | libpod/oci_conmon_linux.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libpod/oci_conmon_linux.go b/libpod/oci_conmon_linux.go index fde8624b0..b07660937 100644 --- a/libpod/oci_conmon_linux.go +++ b/libpod/oci_conmon_linux.go @@ -1354,8 +1354,6 @@ func (r *ConmonOCIRuntime) sharedConmonArgs(ctr *Container, cuuid, bundlePath, p logDriverArg = define.NoLogging case define.PassthroughLogging: logDriverArg = define.PassthroughLogging - case define.JSONLogging: - fallthrough //lint:ignore ST1015 the default case has to be here default: //nolint:stylecheck,gocritic // No case here should happen except JSONLogging, but keep this here in case the options are extended @@ -1365,6 +1363,8 @@ func (r *ConmonOCIRuntime) sharedConmonArgs(ctr *Container, cuuid, bundlePath, p // to get here, either a user would specify `--log-driver ""`, or this came from another place in libpod // since the former case is obscure, and the latter case isn't an error, let's silently fallthrough fallthrough + case define.JSONLogging: + fallthrough case define.KubernetesLogging: logDriverArg = fmt.Sprintf("%s:%s", define.KubernetesLogging, logPath) } |