-
Notifications
You must be signed in to change notification settings - Fork 22
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
cmd-forwarder-vpp should apply changes for the datapath in case of changing connection proto on refresh #441
Comments
/cc @edwarnicke |
@denis-tingaikin Do we know why the endpoint changed the Src/Dst IPs assigned? |
@edwarnicke At this moment we did not face something similar in our examples. The question is about to do we allow endpoints to change datapath config in general on refresh. |
@denis-tingaikin Ah... then yes, yes it should :) |
OK, then we should fix it. |
@edwarnicke Should this be considered before release |
@edwarnicke confirmed internally that the issue should be considered.
|
…k-k8s@main PR link: networkservicemesh/sdk-k8s#441 Commit: 83122c3 Author: Denis Tingaikin Date: 2023-06-29 22:09:02 +0300 Message: - Merge pull request #441 from Nordix/go120 Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
Description
Client or Endpoint could change connection proto-state on refresh requests. For example, change Mac addresses or IP addresses or change routes, and so on.
The question is: Should forwarder-vpp apply the change?
Use-case
Actual: client still have old IPs
Expected: ?
The text was updated successfully, but these errors were encountered: