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.

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

Merged
merged 1 commit into from
Apr 28, 2017

Conversation

timoreimann
Copy link
Contributor

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.

@timoreimann timoreimann added this to the 1.3 milestone Mar 24, 2017
@timoreimann
Copy link
Contributor Author

timoreimann commented Mar 24, 2017

The PR currently builds on #1320. Will be rebased once upstream gets merged.

@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch 2 times, most recently from 3c0fa6f to 757b0c2 Compare March 24, 2017 14:25
@timoreimann timoreimann changed the title Add global health check interval parameter. [WIP] Add global health check interval parameter. Mar 24, 2017
@timoreimann
Copy link
Contributor Author

This isn't working yet because time.Duration values are not parsed correctly. We require #1342.

@timoreimann timoreimann changed the title [WIP] Add global health check interval parameter. Add global health check interval parameter. Mar 31, 2017
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch from 73f02f6 to 031c82c Compare April 10, 2017 19:17
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch from 757b0c2 to 70d819e Compare April 10, 2017 22:12
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch from 031c82c to 2c073ad Compare April 12, 2017 23:44
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch 2 times, most recently from c42759f to d273fa0 Compare April 12, 2017 23:59
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch 3 times, most recently from 9c62808 to 6412620 Compare April 19, 2017 21:43
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch from d273fa0 to 34711c9 Compare April 19, 2017 22:03
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch from 6412620 to 501c916 Compare April 19, 2017 22:07
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch from 34711c9 to 8a27d8e Compare April 19, 2017 22:08
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch from 501c916 to 4d7f3d5 Compare April 19, 2017 22:32
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch from 8a27d8e to 620d4db Compare April 19, 2017 22:32
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch from 4d7f3d5 to 8a99789 Compare April 19, 2017 22:40
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch from 620d4db to b2cbb8f Compare April 19, 2017 22:40
@timoreimann
Copy link
Contributor Author

timoreimann commented Apr 22, 2017

Everything's working as expected for quite a while now.

@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch 2 times, most recently from 9b15cf7 to c2f8bcc Compare April 26, 2017 09:31
@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch 4 times, most recently from 06d365d to db3e461 Compare April 27, 2017 11:36
Copy link
Member

@emilevauge emilevauge left a 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

@ldez
Copy link
Contributor

ldez commented Apr 28, 2017

@timoreimann Need rebase 😉

@timoreimann timoreimann force-pushed the configure-healthcheck-via-marathon-label branch 3 times, most recently from 8023297 to 5d43b9e Compare April 28, 2017 16:17
The new parameter allows to set a health check interval valid for all
backends. Custom values set per provider may override the global one.
@timoreimann timoreimann force-pushed the add-global-option-for-healthcheck-interval branch from b2cbb8f to 2534542 Compare April 28, 2017 18:54
@timoreimann timoreimann changed the base branch from configure-healthcheck-via-marathon-label to master April 28, 2017 18:54
@timoreimann timoreimann removed the request for review from vdemeester April 28, 2017 19:03
@timoreimann timoreimann merged commit e2fdc27 into master Apr 28, 2017
@timoreimann timoreimann deleted the add-global-option-for-healthcheck-interval branch April 28, 2017 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/healthcheck kind/enhancement a new or improved feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants