8000 Vendor go-runc ed1cbe1fc31f5fb2359d3a54b6330d1a097858b7 by mlaventure · Pull Request #1657 · containerd/containerd · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Vendor go-runc ed1cbe1fc31f5fb2359d3a54b6330d1a097858b7 #1657

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 18, 2017

Conversation

mlaventure
Copy link
Contributor

This fixes possible panics in Run & Restore when done without IOs

Signed-off-by: Kenfe-Mickael Laventure mickael.laventure@gmail.com

This fixes possible panics in Run & Restore when done without IOs

Signed-off-by: Kenfe-Mickael Laventure <mickael.laventure@gmail.com>
@codecov-io
Copy link

Codecov Report

Merging #1657 into master will decrease coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #1657     +/-   ##
=========================================
- Coverage   49.06%   48.87%   -0.2%     
=========================================
  Files          27       27             
  Lines        4080     4080             
=========================================
- Hits         2002     1994      -8     
- Misses       1663     1667      +4     
- Partials      415      419      +4
Impacted Files Coverage Δ
metadata/db.go 65.5% <0%> (-1.5%) ⬇️
metadata/snapshot.go 49.34% <0%> (-0.94%) ⬇️

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 84fa396...90c5355. Read the comment docs.

@crosbymichael
Copy link
Member

LGTM

@crosbymichael crosbymichael merged commit 87596ad into containerd:master Oct 18, 2017
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.

3 participants
0