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

Increase dial timeout #420

Merged
merged 1 commit into from
Nov 19, 2021

Conversation

glazychev-art
Copy link
Contributor

Signed-off-by: Artem Glazychev [email protected]

50 ms is not enough for some cases.
For example, we caught it on dial.Close():

...
Nov 19 10:58:10.566[37m [TRAC] [id:0eafd1f2-ecfa-408e-9110-afbfbbcfe96c] [name:nsmgr-42wfj] [type:NetworkService] [0m(23)                       ⎆ sdk/pkg/networkservice/common/dial/dialClient.Close() span=08facc14b33840b5:1189cf3b633a1bb6:130bab6676d637ae:1
Nov 19 10:58:10.642[37m [TRAC] [id:0eafd1f2-ecfa-408e-9110-afbfbbcfe96c] [name:nsmgr-42wfj] [type:NetworkService] [0m(24)                        ⎆ sdk/pkg/networkservice/common/mechanisms/recvfd/recvFDClient.Close() span=08facc14b33840b5:39e0be67363268f5:1189cf3b633a1bb6:1
...

Signed-off-by: Artem Glazychev <[email protected]>
@denis-tingaikin denis-tingaikin merged commit f669cf1 into networkservicemesh:main Nov 19, 2021
nsmbot pushed a commit to networkservicemesh/deployments-k8s that referenced this pull request Nov 19, 2021
…d-nsmgr@main

PR link: networkservicemesh/cmd-nsmgr#420

Commit: f669cf1
Author: Artem Glazychev
Date: 2021-11-19 19:26:51 +0700
Message:
  - Increase dial timeout (#420)
Signed-off-by: NSMBot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants