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

[v7] Increase missing tunnels check interval #15805

Merged
merged 2 commits into from
Aug 29, 2022

Conversation

rosstimothy
Copy link
Contributor

Backports #15654 to branch/v7

Running the missing tunnel check every 5s can cause contention issues
which affect nodes becoming available. It was originally added to
help diagnose networking issues that would prevent reverse tunnels
from being established. The 5s interval was chosen as something that
would spam the logs a bit to make the debugging process easier. However
doing the check so frequently is causing issues itself.

Bumping the value up to 3m should still ensure that this check runs
frequently enough to detect missing tunnels, yet not often enough
to cause issues.
@rosstimothy rosstimothy enabled auto-merge (squash) August 26, 2022 17:11
@rosstimothy rosstimothy force-pushed the tross/backport-15654-branch/v7 branch from 99cc85e to 993d142 Compare August 29, 2022 13:58
@rosstimothy rosstimothy merged commit 7ca3c17 into branch/v7 Aug 29, 2022
@zmb3 zmb3 deleted the tross/backport-15654-branch/v7 branch September 9, 2022 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants