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

use source address for ping to check if the connection is alive #535

Merged

Conversation

glazychev-art
Copy link
Contributor

@glazychev-art glazychev-art commented Nov 14, 2022

Description

Closes: networkservicemesh/integration-k8s-packet#306

Now we specify the source address to determine which interface to use for checking.

Moved the check to goroutines, because with several addresses (src and dst) it is incorrect to divide the timeout.

Signed-off-by: Artem Glazychev artem.glazychev@xored.com

Copy link
Member

@denis-tingaikin denis-tingaikin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few comments.

pkg/kernel/tools/heal/liveness_check.go Show resolved Hide resolved
pkg/kernel/tools/heal/liveness_check.go Outdated Show resolved Hide resolved
pkg/kernel/tools/heal/liveness_check.go Show resolved Hide resolved
pkg/kernel/tools/heal/liveness_check.go Outdated Show resolved Hide resolved
pkg/kernel/tools/heal/liveness_check.go Outdated Show resolved Hide resolved
@glazychev-art glazychev-art force-pushed the livenesscheck_source branch 2 times, most recently from 9527e5a to 64b796a Compare March 21, 2023 10:57
@denis-tingaikin denis-tingaikin marked this pull request as draft April 24, 2023 15:26
@glazychev-art glazychev-art force-pushed the livenesscheck_source branch from 64b796a to bf26737 Compare May 18, 2023 09:23
@glazychev-art glazychev-art self-assigned this May 18, 2023
@glazychev-art glazychev-art marked this pull request as ready for review May 18, 2023 09:25
Copy link
Member

@denis-tingaikin denis-tingaikin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few minor comments.

In general, looks good to me.

pkg/kernel/tools/heal/liveness_check.go Outdated Show resolved Hide resolved
Signed-off-by: Artem Glazychev <artem.glazychev@xored.com>
@glazychev-art glazychev-art force-pushed the livenesscheck_source branch from bf26737 to 26645b0 Compare May 18, 2023 13:52
@denis-tingaikin denis-tingaikin merged commit 6f7d01e into networkservicemesh:main May 18, 2023
nsmbot pushed a commit to networkservicemesh/sdk-sriov that referenced this pull request May 18, 2023
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#535

Commit: 6f7d01e
Author: Denis Tingaikin
Date: 2023-05-18 17:07:02 +0300
Message:
  - Merge pull request #535 from glazychev-art/livenesscheck_source
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-vpp that referenced this pull request May 18, 2023
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#535

Commit: 6f7d01e
Author: Denis Tingaikin
Date: 2023-05-18 17:07:02 +0300
Message:
  - Merge pull request #535 from glazychev-art/livenesscheck_source
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-l7-proxy that referenced this pull request May 18, 2023
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#535

Commit: 6f7d01e
Author: Denis Tingaikin
Date: 2023-05-18 17:07:02 +0300
Message:
  - Merge pull request #535 from glazychev-art/livenesscheck_source
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Datapath healing doesn't work
2 participants