Skip to content

v2: Fix missing ns when openShimLog on windows#3929

Merged
fuweid merged 1 commit intocontainerd:masterfrom
darfux:v2_fix_win_ctx_ns
Jan 6, 2020
Merged

v2: Fix missing ns when openShimLog on windows#3929
fuweid merged 1 commit intocontainerd:masterfrom
darfux:v2_fix_win_ctx_ns

Conversation

@darfux
Copy link
Contributor

@darfux darfux commented Jan 5, 2020

Related to #3921 (comment)

/cc @TBBle
Signed-off-by: Li Yuxuan liyuxuan04@baidu.com

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

@darfux darfux force-pushed the v2_fix_win_ctx_ns branch from 586c619 to a87118c Compare January 5, 2020 04:13
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

@darfux darfux force-pushed the v2_fix_win_ctx_ns branch from a87118c to 5c059dd Compare January 5, 2020 05:00
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

@darfux darfux force-pushed the v2_fix_win_ctx_ns branch from 5c059dd to aef1a8f Compare January 5, 2020 05:25
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

@darfux darfux force-pushed the v2_fix_win_ctx_ns branch from aef1a8f to 6a22539 Compare January 5, 2020 06:05
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

@darfux darfux force-pushed the v2_fix_win_ctx_ns branch from 6a22539 to 12ab4c0 Compare January 5, 2020 07:57
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

Related to
containerd#3921 (comment)

Signed-off-by: Li Yuxuan <liyuxuan04@baidu.com>
@darfux darfux force-pushed the v2_fix_win_ctx_ns branch from 12ab4c0 to 1fb1d93 Compare January 5, 2020 11:42
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 5, 2020

Build succeeded.

@codecov-io
Copy link

codecov-io commented Jan 5, 2020

Codecov Report

Merging #3929 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3929      +/-   ##
==========================================
- Coverage   42.42%   42.42%   -0.01%     
==========================================
  Files         130      130              
  Lines       14710    14711       +1     
==========================================
  Hits         6241     6241              
- Misses       7548     7549       +1     
  Partials      921      921
Flag Coverage Δ
#linux 45.81% <0%> (-0.01%) ⬇️
#windows 37.94% <0%> (-0.01%) ⬇️
Impacted Files Coverage Δ
runtime/v2/binary.go 0% <0%> (ø) ⬆️

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 5fc0f30...1fb1d93. Read the comment docs.

Copy link
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

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.

4 participants