Ensure that we don't add balancees with no health check runs if there…

… is a health check defined on it Change test behavior
jangie · Feb 3, 2017 · ad12a72 · ad12a72
1 parent 29059b7
commit ad12a72
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/provider/marathon.go b/provider/marathon.go
@@ -264,6 +264,9 @@ func (provider *Marathon) taskFilter(task marathon.Task, applications *marathon.
 					return false
 				}
 			}
+		} else {
+			log.Debugf("Filtering marathon task %s with defined healthcheck as no healthcheck has run yet", task.AppID)
+			return false
 		}
 	}
 	return true

diff --git a/provider/marathon_test.go b/provider/marathon_test.go
@@ -629,7 +629,7 @@ func TestMarathonTaskFilter(t *testing.T) {
 					},
 				},
 			},
-			expected:         true,
+			expected:         false,
 			exposedByDefault: true,
 		},
 		{