-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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.
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
Add global health check interval parameter. #1338
Add global health check interval parameter. #1338
Conversation
The PR currently builds on #1320. Will be rebased once upstream gets merged. |
3c0fa6f
to
757b0c2
Compare
This isn't working yet because |
73f02f6
to
031c82c
Compare
757b0c2
to
70d819e
Compare
031c82c
to
2c073ad
Compare
c42759f
to
d273fa0
Compare
9c62808
to
6412620
Compare
d273fa0
to
34711c9
Compare
6412620
to
501c916
Compare
34711c9
to
8a27d8e
Compare
501c916
to
4d7f3d5
Compare
8a27d8e
to
620d4db
Compare
4d7f3d5
to
8a99789
Compare
620d4db
to
b2cbb8f
Compare
Everything's working as expected for quite a while now. |
9b15cf7
to
c2f8bcc
Compare
06d365d
to
db3e461
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @timoreimann, great PR :)
LGTM
@timoreimann Need rebase 😉 |
8023297
to
5d43b9e
Compare
The new parameter allows to set a health check interval valid for all backends. Custom values set per provider may override the global one.
b2cbb8f
to
2534542
Compare
The new parameter allows to set a health check interval valid for all backends. Custom values set per provider may override the global one.
@containous/traefik Please review.