mesosphere / marathon-lb

Marathon-lb is a service discovery & load balancing tool for DC/OS
Apache License 2.0
449 stars 300 forks source link

Fix bug in checking healthChecks at first (#655) #656

Closed chozo99 closed 4 years ago

chozo99 commented 4 years ago

Suppose you have an app that needs to warming up at first. ( e.g. http response code: 503 -> 200 )

When start a app that has healthChecks task have healthChecks Results property but don't have any element in array at first

But it add backend server in haproxy

Eventually, Before healthChecks run once at first request's packet will flow in a app that don't check healthChecks

655

mesosphere-ci commented 4 years ago

Can one of the admins verify this patch?

jkoelker commented 4 years ago

ok to test

jkoelker commented 4 years ago

Thanks!