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

Move pinggrouprange to connectioncontext #601

Merged

Conversation

glazychev-art
Copy link
Contributor

Description

If we assign an IP address to the kernel interface (this is done in the connectioncontext), we also should set the pinggrouprange. This element is required for unprivileged livelinesschecker

Signed-off-by: Artem Glazychev <artem.glazychev@xored.com>
@denis-tingaikin denis-tingaikin merged commit f7a0057 into networkservicemesh:main Jul 14, 2023
12 checks passed
nsmbot pushed a commit to networkservicemesh/sdk-sriov that referenced this pull request Jul 14, 2023
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#601

Commit: f7a0057
Author: Denis Tingaikin
Date: 2023-07-14 13:21:59 +0300
Message:
  - Merge pull request #601 from glazychev-art/pinggrouprange_move
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-l7-proxy that referenced this pull request Jul 14, 2023
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#601

Commit: f7a0057
Author: Denis Tingaikin
Date: 2023-07-14 13:21:59 +0300
Message:
  - Merge pull request #601 from glazychev-art/pinggrouprange_move
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-vpp that referenced this pull request Jul 14, 2023
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#601

Commit: f7a0057
Author: Denis Tingaikin
Date: 2023-07-14 13:21:59 +0300
Message:
  - Merge pull request #601 from glazychev-art/pinggrouprange_move
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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants