diff options
| author | Giuseppe Scrivano <gscrivan@redhat.com> | 2020-05-06 13:20:44 +0200 |
|---|---|---|
| committer | Daniel J Walsh <dwalsh@redhat.com> | 2020-05-07 14:16:50 -0400 |
| commit | 99bdafba999078bb5763c65cf7dfe7df520030f7 (patch) | |
| tree | 916c768fca45b5e809f9e941b5425e760dcfe08c /cmd/podman/common/specgen.go | |
| parent | 1d3cdf9a4642df073becb54d178d6b1959526a47 (diff) | |
| download | podman-99bdafba999078bb5763c65cf7dfe7df520030f7.tar.gz podman-99bdafba999078bb5763c65cf7dfe7df520030f7.tar.bz2 podman-99bdafba999078bb5763c65cf7dfe7df520030f7.zip | |
podman: split env variables in env and overrides
There are three different priorities for applying env variables:
1) environment/config file environment variables
2) image's config
3) user overrides (--env)
The third kind are known to the client, while the default config and image's
config is handled by the backend.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'cmd/podman/common/specgen.go')
| -rw-r--r-- | cmd/podman/common/specgen.go | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/cmd/podman/common/specgen.go b/cmd/podman/common/specgen.go index 3e9772576..9a2345064 100644 --- a/cmd/podman/common/specgen.go +++ b/cmd/podman/common/specgen.go @@ -335,15 +335,12 @@ func FillOutSpecGen(s *specgen.SpecGenerator, c *ContainerCLIOpts, args []string env = envLib.Join(env, fileEnv) } - // env overrides any previous variables - if cmdLineEnv := c.env; len(cmdLineEnv) > 0 { - parsedEnv, err := envLib.ParseSlice(cmdLineEnv) - if err != nil { - return err - } - env = envLib.Join(env, parsedEnv) + parsedEnv, err := envLib.ParseSlice(c.Env) + if err != nil { + return err } - s.Env = env + + s.Env = envLib.Join(env, parsedEnv) // LABEL VARIABLES labels, err := parse.GetAllLabels(c.LabelFile, c.Label) |