miry
commented
8 years ago I found this bug very harmful. My consul has a invalid keys. And now dns discovery is not working well, because pointed to old instances instead of varnish old key. I think to add verify how many instances per task do you have in marathon.
Can @owners of the repo merge this fix?
ChrisAubuchon
Fixes a bug in deregistration, where the process aborts if a single deregistration fails. This happens most commonly if a host is down, and means that any deregistrations after those on the downed host will not be performed.
This PR changes that behaviour so that a message will be thrown if a deregistration request fails (at info level) but degregistration will continue.