Skip to content

Update runc to 96ec2177ae841256168fcf76954f7177af#2857

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
crosbymichael:runc-kill-paused
Dec 4, 2018
Merged

Update runc to 96ec2177ae841256168fcf76954f7177af#2857
dmcgowan merged 1 commit intocontainerd:masterfrom
crosbymichael:runc-kill-paused

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

This fixes a regression in runc that didn't allow signals being sent to
paused containers.

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

This fixes a regression in runc that didn't allow signals being sent to
paused containers.

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

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2857      +/-   ##
==========================================
- Coverage   43.75%   40.91%   -2.84%     
==========================================
  Files         100       69      -31     
  Lines       10751     9409    -1342     
==========================================
- Hits         4704     3850     -854     
+ Misses       5317     5002     -315     
+ Partials      730      557     -173
Flag Coverage Δ
#linux ?
#windows 40.91% <ø> (ø) ⬆️
Impacted Files Coverage Δ
snapshots/native/native.go 1.78% <0%> (-41.52%) ⬇️
archive/tar.go 16.99% <0%> (-26.15%) ⬇️
metadata/snapshot.go 21.53% <0%> (-24.28%) ⬇️
content/local/writer.go 56.86% <0%> (-0.99%) ⬇️
gc/scheduler/scheduler.go 66.34% <0%> (-0.97%) ⬇️
oci/spec_opts.go 20.49% <0%> (-0.28%) ⬇️
mount/temp_unix.go
sys/reaper_linux.go
services/server/server_linux.go
sys/env.go
... and 27 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 4a8acf5...66c20f2. 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

@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Dec 4, 2018

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.

5 participants