From 50e470fab4ebdffff74bafeefa2f7fb8c21da13e Mon Sep 17 00:00:00 2001
From: Sebastiaan van Stijn <github@gone.nl>
Date: Fri, 3 Jun 2016 13:28:08 +0200
Subject: [PATCH] Healthcheck: set default retries to 3

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
---
 builder/dockerfile/parser/testfiles/health/Dockerfile | 2 +-
 builder/dockerfile/parser/testfiles/health/result     | 2 +-
 daemon/health.go                                      | 6 +++++-
 docs/reference/builder.md                             | 2 +-
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/builder/dockerfile/parser/testfiles/health/Dockerfile b/builder/dockerfile/parser/testfiles/health/Dockerfile
index 6534ce17d4..081e442882 100644
--- a/builder/dockerfile/parser/testfiles/health/Dockerfile
+++ b/builder/dockerfile/parser/testfiles/health/Dockerfile
@@ -2,7 +2,7 @@ FROM debian
 ADD check.sh main.sh /app/
 CMD /app/main.sh
 HEALTHCHECK
-HEALTHCHECK --interval=5s --timeout=3s --retries=1 \
+HEALTHCHECK --interval=5s --timeout=3s --retries=3 \
   CMD /app/check.sh --quiet
 HEALTHCHECK CMD
 HEALTHCHECK   CMD   a b
diff --git a/builder/dockerfile/parser/testfiles/health/result b/builder/dockerfile/parser/testfiles/health/result
index bfa846c917..092924f88c 100644
--- a/builder/dockerfile/parser/testfiles/health/result
+++ b/builder/dockerfile/parser/testfiles/health/result
@@ -2,7 +2,7 @@
 (add "check.sh" "main.sh" "/app/")
 (cmd "/app/main.sh")
 (healthcheck)
-(healthcheck ["--interval=5s" "--timeout=3s" "--retries=1"] "CMD" "/app/check.sh --quiet")
+(healthcheck ["--interval=5s" "--timeout=3s" "--retries=3"] "CMD" "/app/check.sh --quiet")
 (healthcheck "CMD")
 (healthcheck "CMD" "a b")
 (healthcheck ["--timeout=3s"] "CMD" "foo")
diff --git a/daemon/health.go b/daemon/health.go
index ec9843561a..781ba76239 100644
--- a/daemon/health.go
+++ b/daemon/health.go
@@ -28,6 +28,10 @@ const (
 	// than this, the check is considered to have failed.
 	defaultProbeTimeout = 30 * time.Second
 
+	// Default number of consecutive failures of the health check
+	// for the container to be considered unhealthy.
+	defaultProbeRetries = 3
+
 	// Shut down a container if it becomes Unhealthy.
 	defaultExitOnUnhealthy = true
 
@@ -111,7 +115,7 @@ func handleProbeResult(d *Daemon, c *container.Container, result *types.Healthch
 
 	retries := c.Config.Healthcheck.Retries
 	if retries <= 0 {
-		retries = 1 // Default if unset or set to an invalid value
+		retries = defaultProbeRetries
 	}
 
 	h := c.State.Health
diff --git a/docs/reference/builder.md b/docs/reference/builder.md
index f460338b22..efe8d820c5 100644
--- a/docs/reference/builder.md
+++ b/docs/reference/builder.md
@@ -1491,7 +1491,7 @@ The options that can appear before `CMD` are:
 
 * `--interval=DURATION` (default: `30s`)
 * `--timeout=DURATION` (default: `30s`)
-* `--retries=N` (default: `1`)
+* `--retries=N` (default: `3`)
 
 The health check will first run **interval** seconds after the container is
 started, and then again **interval** seconds after each previous check completes.