Skip to content

Add timeouts in case of hung operations#7884

Merged
mhagen-vmware merged 3 commits intomasterfrom
feature/add-timeouts
May 3, 2018
Merged

Add timeouts in case of hung operations#7884
mhagen-vmware merged 3 commits intomasterfrom
feature/add-timeouts

Conversation

@mhagen-vmware
Copy link
Contributor

[skip unit]
The most recent nightly hung on a docker pull within the ops user test, this adds a timeout to prevent it from hanging indefinitely again.

Copy link
Contributor

@rajanashok rajanashok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

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

codecov-io commented May 3, 2018

Codecov Report

Merging #7884 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7884   +/-   ##
=======================================
  Coverage   26.34%   26.34%           
=======================================
  Files          37       37           
  Lines        5189     5189           
=======================================
  Hits         1367     1367           
  Misses       3715     3715           
  Partials      107      107

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 fe040e0...2b84088. Read the comment docs.

@mhagen-vmware mhagen-vmware merged commit 72391e5 into master May 3, 2018
@mhagen-vmware mhagen-vmware deleted the feature/add-timeouts branch May 3, 2018 16:55
zjs pushed a commit to zjs/vic that referenced this pull request Jul 19, 2018
zjs pushed a commit to zjs/vic that referenced this pull request Jul 27, 2018
zjs pushed a commit that referenced this pull request Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants