Disable Apache HTTP Client retry in WaitUtil

WaitUtil has it's own retry mechanism and by disabling the HTTP Client retries it is easier to debug issues with containers starting.

Also HTTP Client retries provide superfluous logging considering it is highly likely the system will be unavailable when the first check occurs.

Signed-off-by: James Baker <james@brighthelix.co.uk>

src/main/java/org/jolokia/docker/maven/util/WaitUtil.java

@@ -10,6 +10,7 @@
 import org.apache.http.client.methods.CloseableHttpResponse;
 import org.apache.http.client.methods.RequestBuilder;
 import org.apache.http.impl.client.CloseableHttpClient;
+import org.apache.http.impl.client.DefaultHttpRequestRetryHandler;
 import org.apache.http.impl.client.HttpClientBuilder;
 
 /**
@@ -34,6 +35,9 @@ public class WaitUtil {
     public static final int DEFAULT_MIN_STATUS = 200;
     public static final int DEFAULT_MAX_STATUS = 399;
 
+    // Disable HTTP client retries by default.
+    public static final int HTTP_CLIENT_RETRIES = 0;
+
 
     private WaitUtil() {}
 
@@ -148,6 +152,7 @@ private boolean ping() throws IOException {
                                  .build();
             CloseableHttpClient httpClient = HttpClientBuilder.create()
                     .setDefaultRequestConfig(requestConfig)
+                    .setRetryHandler(new DefaultHttpRequestRetryHandler(HTTP_CLIENT_RETRIES, false))
                     .build();
             try {
                 CloseableHttpResponse response = httpClient.execute(RequestBuilder.create(method.toUpperCase()).setUri(url).build());