github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

bugfix: override image.Env with process.Env, rather than be contrary

Browse Source 
Signed-off-by: Yangyang <1032120121@163.com>
This commit is contained in:
Yangyang 2019-08-16 11:57:17 +08:00 committed by YangYang
parent 074b75907b
commit 1c30d4d04c
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
oci/spec_opts.go
View File

@ -326,7 +326,7 @@ func WithImageConfigArgs(image Image, args []string) SpecOpts {
setProcess(s) setProcess(s)
if s.Linux != nil { if s.Linux != nil {
s.Process.Env = replaceOrAppendEnvValues(s.Process.Env, config.Env) s.Process.Env = replaceOrAppendEnvValues(config.Env, s.Process.Env)
cmd := config.Cmd cmd := config.Cmd
if len(args) > 0 { if len(args) > 0 {
cmd = args cmd = args
@ -348,7 +348,7 @@ func WithImageConfigArgs(image Image, args []string) SpecOpts {
// even if there is no specified user in the image config // even if there is no specified user in the image config
return WithAdditionalGIDs("root")(ctx, client, c, s) return WithAdditionalGIDs("root")(ctx, client, c, s)
} else if s.Windows != nil { } else if s.Windows != nil {
s.Process.Env = replaceOrAppendEnvValues(s.Process.Env, config.Env) s.Process.Env = replaceOrAppendEnvValues(config.Env, s.Process.Env)
cmd := config.Cmd cmd := config.Cmd
if len(args) > 0 { if len(args) > 0 {
cmd = args cmd = args

2
oci/spec_opts_test.go
View File

@ -406,7 +406,7 @@ func TestWithImageConfigArgs(t *testing.T) {
WithImageConfigArgs(img, []string{"--boo", "bar"}), WithImageConfigArgs(img, []string{"--boo", "bar"}),
} }
expectedEnv := []string{"x=foo", "y=baz", "z=bar"} expectedEnv := []string{"z=bar", "y=boo", "x=foo"}
expectedArgs := []string{"create", "--namespace=test", "--boo", "bar"} expectedArgs := []string{"create", "--namespace=test", "--boo", "bar"}
for _, opt := range opts { for _, opt := range opts {