Skip to content

update cri vendor#2383

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
AkihiroSuda:vendor-cri-20180605
Jun 5, 2018
Merged

update cri vendor#2383
crosbymichael merged 1 commit intocontainerd:masterfrom
AkihiroSuda:vendor-cri-20180605

Conversation

@AkihiroSuda
Copy link
Copy Markdown
Member

@AkihiroSuda AkihiroSuda commented Jun 5, 2018

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

Codecov Report

Merging #2383 into master will increase coverage by 3.73%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2383      +/-   ##
==========================================
+ Coverage    41.3%   45.03%   +3.73%     
==========================================
  Files          66       92      +26     
  Lines        7842     9390    +1548     
==========================================
+ Hits         3239     4229     +990     
- Misses       4095     4478     +383     
- Partials      508      683     +175
Flag Coverage Δ
#linux 49.28% <ø> (?)
#windows 41.3% <ø> (ø) ⬆️
Impacted Files Coverage Δ
oci/spec_opts_unix.go 21.77% <0%> (ø)
oci/spec_unix.go 98.37% <0%> (ø)
content/local/store_unix.go 75% <0%> (ø)
mount/mountinfo_linux.go 53.06% <0%> (ø)
sys/env.go 100% <0%> (ø)
sys/fds.go 0% <0%> (ø)
sys/reaper.go 0% <0%> (ø)
snapshots/btrfs/btrfs.go 57.39% <0%> (ø)
snapshots/overlay/check.go 0% <0%> (ø)
cio/io.go 35.44% <0%> (ø)
... and 23 more

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 84bebdd...135a4d9. Read the comment docs.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 0d7fd3f into containerd:master Jun 5, 2018
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