Skip to content

Add timeout to task state calls#3444

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
crosbymichael:state-timeout
Jul 25, 2019
Merged

Add timeout to task state calls#3444
crosbymichael merged 1 commit intocontainerd:masterfrom
crosbymichael:state-timeout

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

Fixes #3440

This also returns the task that times out or has an error on the state
call in an UNKNOWN status so that the user can manually kill and remove
the task.

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

Comment thread runtime/v1/shim/service.go Outdated
Fixes containerd#3440

This also returns the task that times out or has an error on the state
call in an UNKNOWN status so that the user can manually kill and remove
the task.

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

theopenlab-ci Bot commented Jul 23, 2019

Build succeeded.

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

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3444 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #3444      +/-   ##
=========================================
- Coverage   44.11%   44.1%   -0.01%     
=========================================
  Files         124     124              
  Lines       13760   13760              
=========================================
- Hits         6070    6069       -1     
- Misses       6759    6760       +1     
  Partials      931     931
Flag Coverage Δ
#linux 47.87% <ø> (ø) ⬆️
#windows 39.75% <ø> (-0.01%) ⬇️
Impacted Files Coverage Δ
runtime/v2/shim/shim_windows.go 42.85% <0%> (-0.65%) ⬇️

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 f776141...101d4b7. Read the comment docs.

Comment thread services/tasks/local.go
Copy link
Copy Markdown
Contributor

@jterry75 jterry75 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 crosbymichael merged commit 41e1723 into containerd:master Jul 25, 2019
@crosbymichael crosbymichael deleted the state-timeout branch July 25, 2019 16:55
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.

container runtime is down after kubelet is restarted

4 participants