Skip to content

server: only warn on failed OOM score adjust#1906

Merged
estesp merged 1 commit intocontainerd:masterfrom
stevvooe:change-on-oom-failure
Dec 12, 2017
Merged

server: only warn on failed OOM score adjust#1906
estesp merged 1 commit intocontainerd:masterfrom
stevvooe:change-on-oom-failure

Conversation

@stevvooe
Copy link
Copy Markdown
Member

Signed-off-by: Stephen J Day [email protected]

@n4ss
Copy link
Copy Markdown

n4ss commented Dec 11, 2017

LGTM :D

@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 11, 2017

Codecov Report

Merging #1906 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1906   +/-   ##
=======================================
  Coverage   47.29%   47.29%           
=======================================
  Files          89       89           
  Lines        8830     8830           
=======================================
  Hits         4176     4176           
  Misses       3968     3968           
  Partials      686      686
Flag Coverage Δ
#linux 50.66% <0%> (ø) ⬆️
#windows 42.24% <ø> (ø) ⬆️
Impacted Files Coverage Δ
server/server_linux.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 6393165...af5d03e. Read the comment docs.

@crosbymichael
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

@estesp estesp merged commit 03bc5e9 into containerd:master Dec 12, 2017
@stevvooe stevvooe deleted the change-on-oom-failure branch December 12, 2017 18:58
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