diff options
author | Peter Hunt <pehunt@redhat.com> | 2019-05-20 13:58:31 -0400 |
---|---|---|
committer | Peter Hunt <pehunt@redhat.com> | 2019-05-28 11:10:57 -0400 |
commit | 51bdf29f0493827ce3eb278a193d0a7402add896 (patch) | |
tree | 685eb00a699d96bc0f5174ee7b9b7d51d6a851b9 /libpod/options.go | |
parent | 02f971131a3bb71615d15a45df808edd0fa88266 (diff) | |
download | podman-51bdf29f0493827ce3eb278a193d0a7402add896.tar.gz podman-51bdf29f0493827ce3eb278a193d0a7402add896.tar.bz2 podman-51bdf29f0493827ce3eb278a193d0a7402add896.zip |
Address comments
Signed-off-by: Peter Hunt <pehunt@redhat.com>
Diffstat (limited to 'libpod/options.go')
-rw-r--r-- | libpod/options.go | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/libpod/options.go b/libpod/options.go index d6eb97609..20aa51981 100644 --- a/libpod/options.go +++ b/libpod/options.go @@ -985,10 +985,12 @@ func WithLogDriver(driver string) CtrCreateOption { if ctr.valid { return ErrCtrFinalized } - if driver == "" { + switch driver { + case "": return errors.Wrapf(ErrInvalidArg, "log driver must be set") - } - if driver != "journald" && driver != "k8s-file" && driver != "json-file" { + case JournaldLogging, KubernetesLogging, JSONLogging: + break + default: return errors.Wrapf(ErrInvalidArg, "invalid log driver") } |