Retry check interval #1268

Closed
andronux-zz opened this Issue Oct 1, 2015 · 3 comments

Comments

Projects
None yet
4 participants
@andronux-zz

It'll be nice to have a retry_check_interval along with a retry_check_count option so a service can be marked as being down if, for instance, the check fails x times.

"retry_check_interval" : 3s,
"retry_check_count": 3

[check] ...wait 3s ... [check] ... wait 3s ... [check] === {warning or critical}

@ryanuber

This comment has been minimized.

Show comment
Hide comment
@ryanuber

ryanuber Oct 2, 2015

Member

One way you can have this functionality now using a script/interval check type is to take a longer interval, and do the retries within the script. It's possible we could add some quiescence for checks, marking as a thinking ticket for now. Thanks.

Member

ryanuber commented Oct 2, 2015

One way you can have this functionality now using a script/interval check type is to take a longer interval, and do the retries within the script. It's possible we could add some quiescence for checks, marking as a thinking ticket for now. Thanks.

@ryanuber ryanuber added the thinking label Oct 2, 2015

@linyows linyows referenced this issue in linyows/consul-cookbook Nov 18, 2015

Merged

Support retry for consul check #8

@thedebugger

This comment has been minimized.

Show comment
Hide comment
@thedebugger

thedebugger Jan 27, 2017

We have been using script checks but we want to use native checks if this feature is supported. We don't want to manage script checks as they are hard to get right on first few attempts. So it would be nice to have this support in the native checks.

We have been using script checks but we want to use native checks if this feature is supported. We don't want to manage script checks as they are hard to get right on first few attempts. So it would be nice to have this support in the native checks.

@slackpad

This comment has been minimized.

Show comment
Hide comment
@slackpad

slackpad May 3, 2017

Contributor

Closing as a duplicate of #2890.

Contributor

slackpad commented May 3, 2017

Closing as a duplicate of #2890.

@slackpad slackpad closed this May 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment