Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

404 responses when a new Marathon leader is elected #653

Closed
gomes opened this issue Aug 29, 2016 · 1 comment
Closed

404 responses when a new Marathon leader is elected #653

gomes opened this issue Aug 29, 2016 · 1 comment
Labels
area/provider/marathon kind/bug/confirmed a confirmed bug (reproducible). status/5-frozen-due-to-age

Comments

@gomes
Copy link
Contributor

gomes commented Aug 29, 2016 •
edited
Loading

The problem: I am running a lot of apps in my Marathon cluster, +200 and when a leader is elected it actually runs healthchecks for all tasks, so it starts with 0 healthchecked apps and starts to check every task and populating data about the results. Meantime, Traefik thinks I don't have my apps anymore, because it assumes if I don't have healthchecks, it's unhealthy. That's untrue, because my apps are still there, Marathon just didn't had enough time to check all of then.

So, Traefik should not consider the absence of a healthcheck the same as not healthy.
@gomes gomes mentioned this issue Aug 29, 2016
Closed
@emilevauge emilevauge added the bug label Aug 30, 2016
@gomes gomes mentioned this issue Sep 5, 2016
Merged
emilevauge pushed a commit that referenced this issue Sep 14, 2016
@gomes @emilevauge
If Marathon doesn't have healthcheck, assume it's ok (#665)
1e324ad 
Healthcheck are not mandatory, so if a result is not present, assume it
is ok to continue. Fixes the case when a new leader is elected and
don't have any healthcheck result's, returning 404 to all requests.
#653
@gomes
Copy link
Contributor Author

gomes commented Nov 21, 2016

The PR (#665) fixing this issue was merged in version 1.1.0, so I will close this issue.

@gomes gomes closed this as completed Nov 21, 2016
This was referenced Feb 23, 2017
Merged
Closed
@ldez ldez added the kind/bug/confirmed a confirmed bug (reproducible). label Apr 29, 2017
@timoreimann timoreimann mentioned this issue Mar 3, 2019
Open
@traefik traefik locked and limited conversation to collaborators Sep 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area/provider/marathon kind/bug/confirmed a confirmed bug (reproducible). status/5-frozen-due-to-age
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@errm @gomes @ldez @emilevauge @traefiker