Merge pull request #30683 from milindchawre/fix_30503
Adding support for docker max restart time Upstream-commit: 67dffe2741a1f56de0a8225988765bc5baa543fc Component: engine
This commit is contained in:
@ -12,6 +12,7 @@ import (
|
||||
const (
|
||||
backoffMultiplier = 2
|
||||
defaultTimeout = 100 * time.Millisecond
|
||||
maxRestartTimeout = 1 * time.Minute
|
||||
)
|
||||
|
||||
// ErrRestartCanceled is returned when the restart manager has been
|
||||
@ -70,11 +71,15 @@ func (rm *restartManager) ShouldRestart(exitCode uint32, hasBeenManuallyStopped
|
||||
if executionDuration.Seconds() >= 10 {
|
||||
rm.timeout = 0
|
||||
}
|
||||
if rm.timeout == 0 {
|
||||
switch {
|
||||
case rm.timeout == 0:
|
||||
rm.timeout = defaultTimeout
|
||||
} else {
|
||||
case rm.timeout < maxRestartTimeout:
|
||||
rm.timeout *= backoffMultiplier
|
||||
}
|
||||
if rm.timeout > maxRestartTimeout {
|
||||
rm.timeout = maxRestartTimeout
|
||||
}
|
||||
|
||||
var restart bool
|
||||
switch {
|
||||
|
||||
Reference in New Issue
Block a user