diff options
author | cdoern <cbdoer23@g.holycross.edu> | 2022-05-03 22:26:43 -0400 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2022-06-14 13:08:53 -0400 |
commit | 8a7e280945c182b7feed68f6c9d2166136526040 (patch) | |
tree | 7760cb3bc1d795b3f0718f3a713e0de446e0e25b | |
parent | a1ec21dac104d811d5a22a663181a503177d5187 (diff) | |
download | podman-8a7e280945c182b7feed68f6c9d2166136526040.tar.gz podman-8a7e280945c182b7feed68f6c9d2166136526040.tar.bz2 podman-8a7e280945c182b7feed68f6c9d2166136526040.zip |
play kube log tag handling
currently tags cause a panic due to an uninitialized map. Initialize the map
and add parsing to make sure we are only tagging with journald
resolves #13356
Signed-off-by: cdoern <cbdoer23@g.holycross.edu>
-rw-r--r-- | libpod/oci_conmon_exec_linux.go | 2 | ||||
-rw-r--r-- | pkg/specgen/generate/container_create.go | 1 | ||||
-rw-r--r-- | pkg/specgen/generate/kube/kube.go | 14 | ||||
-rw-r--r-- | test/e2e/play_kube_test.go | 20 |
4 files changed, 34 insertions, 3 deletions
diff --git a/libpod/oci_conmon_exec_linux.go b/libpod/oci_conmon_exec_linux.go index 6d2f13525..70124bec1 100644 --- a/libpod/oci_conmon_exec_linux.go +++ b/libpod/oci_conmon_exec_linux.go @@ -391,7 +391,7 @@ func (r *ConmonOCIRuntime) startExec(c *Container, sessionID string, options *Ex } defer processFile.Close() - args := r.sharedConmonArgs(c, sessionID, c.execBundlePath(sessionID), c.execPidPath(sessionID), c.execLogPath(sessionID), c.execExitFileDir(sessionID), ociLog, define.NoLogging, "") + args := r.sharedConmonArgs(c, sessionID, c.execBundlePath(sessionID), c.execPidPath(sessionID), c.execLogPath(sessionID), c.execExitFileDir(sessionID), ociLog, define.NoLogging, c.config.LogTag) if options.PreserveFDs > 0 { args = append(args, formatRuntimeOpts("--preserve-fds", fmt.Sprintf("%d", options.PreserveFDs))...) diff --git a/pkg/specgen/generate/container_create.go b/pkg/specgen/generate/container_create.go index 19a2b702c..04e24d625 100644 --- a/pkg/specgen/generate/container_create.go +++ b/pkg/specgen/generate/container_create.go @@ -425,7 +425,6 @@ func createContainerOptions(rt *libpod.Runtime, s *specgen.SpecGenerator, pod *l options = append(options, libpod.WithMaxLogSize(s.LogConfiguration.Size)) } if len(s.LogConfiguration.Options) > 0 && s.LogConfiguration.Options["tag"] != "" { - // Note: I'm really guessing here. options = append(options, libpod.WithLogTag(s.LogConfiguration.Options["tag"])) } diff --git a/pkg/specgen/generate/kube/kube.go b/pkg/specgen/generate/kube/kube.go index 4c11e4bff..d56b50fd5 100644 --- a/pkg/specgen/generate/kube/kube.go +++ b/pkg/specgen/generate/kube/kube.go @@ -29,6 +29,7 @@ import ( "github.com/docker/go-units" spec "github.com/opencontainers/runtime-spec/specs-go" "github.com/pkg/errors" + "github.com/sirupsen/logrus" ) func ToPodOpt(ctx context.Context, podName string, p entities.PodCreateOptions, podYAML *v1.PodTemplateSpec) (entities.PodCreateOptions, error) { @@ -153,6 +154,7 @@ func ToSpecGen(ctx context.Context, opts *CtrSpecGenOptions) (*specgen.SpecGener Driver: opts.LogDriver, } + s.LogConfiguration.Options = make(map[string]string) for _, o := range opts.LogOptions { split := strings.SplitN(o, "=", 2) if len(split) < 2 { @@ -170,7 +172,17 @@ func ToSpecGen(ctx context.Context, opts *CtrSpecGenOptions) (*specgen.SpecGener } s.LogConfiguration.Size = logSize default: - s.LogConfiguration.Options[split[0]] = split[1] + switch len(split[1]) { + case 0: + return nil, errors.Wrapf(define.ErrInvalidArg, "invalid log option") + default: + // tags for journald only + if s.LogConfiguration.Driver == "" || s.LogConfiguration.Driver == define.JournaldLogging { + s.LogConfiguration.Options[split[0]] = split[1] + } else { + logrus.Warnf("Can only set tags with journald log driver but driver is %q", s.LogConfiguration.Driver) + } + } } } diff --git a/test/e2e/play_kube_test.go b/test/e2e/play_kube_test.go index 45414ec04..1b223637e 100644 --- a/test/e2e/play_kube_test.go +++ b/test/e2e/play_kube_test.go @@ -3536,4 +3536,24 @@ ENV OPENJ9_JAVA_OPTIONS=%q Expect(kube.ErrorToString()).To(ContainSubstring("ambiguous configuration: the same config map foo is present in YAML and in --configmaps")) }) }) + + It("podman play kube --log-opt = tag test", func() { + pod := getPod() + err := generateKubeYaml("pod", pod, kubeYaml) + Expect(err).To(BeNil()) + + kube := podmanTest.Podman([]string{"play", "kube", kubeYaml, "--log-driver", "journald", "--log-opt", "tag={{.ImageName}}"}) + kube.WaitWithDefaultTimeout() + Expect(kube).Should(Exit(0)) + + start := podmanTest.Podman([]string{"start", getCtrNameInPod(pod)}) + start.WaitWithDefaultTimeout() + Expect(start).Should(Exit(0)) + + inspect := podmanTest.Podman([]string{"inspect", getCtrNameInPod(pod)}) + inspect.WaitWithDefaultTimeout() + Expect(start).Should(Exit(0)) + Expect((inspect.InspectContainerToJSON()[0]).HostConfig.LogConfig.Tag).To(Equal("{{.ImageName}}")) + + }) }) |