Skip to content

Update runc to 00dc70017d222b178a002ed30e9321b126#2674

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
crosbymichael:runc
Sep 21, 2018
Merged

Update runc to 00dc70017d222b178a002ed30e9321b126#2674
crosbymichael merged 1 commit intocontainerd:masterfrom
crosbymichael:runc

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

Signed-off-by: Michael Crosby [email protected]

@dmcgowan dmcgowan added this to the 1.2 milestone Sep 20, 2018
@dmcgowan
Copy link
Copy Markdown
Member

LGTM

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; seems like this has no material code changes; only comments and docs. Is this just to line up with another vendoring commit?

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #2674      +/-   ##
=========================================
- Coverage   45.04%     45%   -0.04%     
=========================================
  Files          92      92              
  Lines       10106   10114       +8     
=========================================
  Hits         4552    4552              
- Misses       4834    4842       +8     
  Partials      720     720
Flag Coverage Δ
#linux 48.74% <ø> (-0.04%) ⬇️
#windows 41.76% <ø> (-0.04%) ⬇️
Impacted Files Coverage Δ
oci/spec_opts.go 20.89% <0%> (-0.24%) ⬇️

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 2623241...806a8f9. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member Author

Not just vendoring, this is how we pin to a runc version. Most of the changes and fixes are in the runc binary build.

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