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

TestVl3_dns is not stable #9063

Closed
glazychev-art opened this issue Apr 28, 2023 · 5 comments · Fixed by networkservicemesh/sdk#1479
Closed

TestVl3_dns is not stable #9063

glazychev-art opened this issue Apr 28, 2023 · 5 comments · Fixed by networkservicemesh/sdk#1479
Assignees

Comments

@glazychev-art
Copy link
Contributor

glazychev-art commented Apr 28, 2023

Incident

https://github.com/networkservicemesh/integration-k8s-kind/actions/runs/4826007507/jobs/8597432798?pr=814

Logs

TestRunFeatureSuite.zip

@glazychev-art glazychev-art self-assigned this Apr 28, 2023
@glazychev-art glazychev-art moved this to In Progress in Release v1.9.0 Apr 28, 2023
@denis-tingaikin denis-tingaikin moved this from In Progress to Moved to next release in Release v1.9.0 May 2, 2023
@NikitaSkrynnik
Copy link
Collaborator

@glazychev-art
Copy link
Contributor Author

Blocked by: networkservicemesh/sdk#1457

@denis-tingaikin denis-tingaikin moved this from In Progress to Blocked in Release v1.10.0 May 19, 2023
@glazychev-art
Copy link
Contributor Author

glazychev-art commented Jun 21, 2023

@glazychev-art glazychev-art moved this from Blocked to In Progress in Release v1.10.0 Jun 30, 2023
@glazychev-art
Copy link
Contributor Author

The reason is that if the Request fails on the refresh (Refresh_Requested), another unavailable forwarder (from the previous test) is selected. In this case, the connection to the current one is closed and is not re-created.

Possible solutions:

  1. Increase dial timeout.
  2. Do not select another forwarder until Close comes.

Blocked by: networkservicemesh/sdk#1471

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants