Skip to content

[release/1.2] backport bugfix: override image.Env with process.Env, rather than be contrary#3546

Merged
fuweid merged 2 commits intocontainerd:release/1.2from
estesp:cp-3542
Aug 16, 2019
Merged

[release/1.2] backport bugfix: override image.Env with process.Env, rather than be contrary#3546
fuweid merged 2 commits intocontainerd:release/1.2from
estesp:cp-3542

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Aug 16, 2019

Signed-off-by: Yangyang [email protected]
Signed-off-by: Andrey Kolomentsev [email protected]
Signed-off-by: Phil Estes [email protected]

includes backport of some test framework code due to cherry-pick conflicts around the env/image config test

Cherry-pick/backport of #3542
Backport of #2887

@estesp estesp added this to the 1.2.8 milestone Aug 16, 2019
Signed-off-by: Andrey Kolomentsev <[email protected]>

Signed-off-by: akolomentsev <[email protected]>
@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 16, 2019

Codecov Report

Merging #3546 into release/1.2 will increase coverage by 0.3%.
The diff coverage is 94.44%.

Impacted file tree graph

@@              Coverage Diff               @@
##           release/1.2    #3546     +/-   ##
==============================================
+ Coverage        43.66%   43.96%   +0.3%     
==============================================
  Files              101      101             
  Lines            10816    10824      +8     
==============================================
+ Hits              4723     4759     +36     
+ Misses            5357     5326     -31     
- Partials           736      739      +3
Flag Coverage Δ
#linux 47.58% <93.75%> (+0.33%) ⬆️
#windows 41.12% <94.44%> (+0.34%) ⬆️
Impacted Files Coverage Δ
oci/spec_opts.go 25.68% <94.44%> (+4.68%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a052634...c8d75ca. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

crosbymichael commented Aug 16, 2019

LGTM

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fuweid fuweid merged commit b21e4f4 into containerd:release/1.2 Aug 16, 2019
@estesp estesp deleted the cp-3542 branch August 21, 2019 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants