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

rabbit: raise failure-timeout #2065

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

rabbit: raise failure-timeout #2065

wants to merge 1 commit into from

Conversation

dirkmueller
Copy link
Contributor

The failure timeout of 30s is far too low. Essentially it means that a
failed node is considered ready after 30s. Given that any start or
stop operation takes considerably more than 30s. We should only expire
failures after around 30 minutes to prevent flapping services.

@dirkmueller @JanZerebecki
rabbit: raise failure-timeout (noref)
88ddf9f 
The failure timeout of 30s is far too low. Essentially it means that a
failed node is considered ready after 30s. Given that any start or
stop operation takes considerably more than 30s. We should only expire
failures after around 30 minutes to prevent flapping services.
@JanZerebecki
Copy link
Contributor

Changed commit message to match regex checked by travis.
Sounds good to me, but I don't know the finer details of these timings, really needs someone with rabitmq clustering knowledge.

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

@aspiers aspiers Awaiting requested review from aspiers

@Gsantomaggio Gsantomaggio Awaiting requested review from Gsantomaggio

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
needs backport to SOC7 needs backport to SOC8
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dirkmueller @JanZerebecki