New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updater: Decrease default monitoring period #1967

Merged
merged 1 commit into from Feb 24, 2017
Jump to file or symbol
Failed to load files and symbols.
+1 −1
Diff settings

Always

Just for now

updater: Decrease default monitoring period

The updater monitors a tasks after they start to make sure they don't
fail immediately. The default time interval for this is 30s. However,
this is a bit long for synchronous updates to wait. Reduce it to 5s.
Users who would like a longer monitoring period should specify their own
value, or ideally use a health check in the service definition.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
  • Loading branch information...
aaronlehmann committed Feb 17, 2017
commit bc74a253b31bb1007ceace5f1aba00233991afcd
@@ -21,7 +21,7 @@ import (
gogotypes "github.com/gogo/protobuf/types"
)
const defaultMonitor = 30 * time.Second
const defaultMonitor = 5 * time.Second
// Supervisor supervises a set of updates. It's responsible for keeping track of updates,
// shutting them down and replacing them.
ProTip! Use n and p to navigate between commits in a pull request.