Begin merge connection strategy #1231
Labels
bug
Something isn't working
Comments
|
Hmm. Can this be related to networkservicemesh/cmd-forwarder-vpp#441? |
|
/cc @edwarnicke |
|
/cc @glazychev-art |
|
yes, from what I understood it seems related. So far, the only blocking element is the begin chain component. I tried with the changes in my environment and it worked fine (my NSC re-request with updated source IPs and policies) |
LionelJouin
added a commit
to Nordix/nsm-sdk
that referenced
this issue
Feb 23, 2022
LionelJouin
added a commit
to Nordix/nsm-sdk
that referenced
this issue
Feb 23, 2022
Using the Client chain, the network service client cannot update the IPContext without the mergeConnectionContext function implemented. A NSC might want to update it if it needs to add, update or delete routes, src IP addresses (VIPs...), policies...
9 tasks
LionelJouin
added a commit
to Nordix/nsm-sdk
that referenced
this issue
Feb 23, 2022
Using the Client chain, the network service client cannot update the IPContext without the mergeConnectionContext function implemented. A NSC might want to update it if it needs to add, update or delete routes, src IP addresses (VIPs...), policies...
LionelJouin
added a commit
to Nordix/nsm-sdk
that referenced
this issue
Mar 3, 2022
Using the Client chain, the network service client cannot update the IPContext without the mergeConnectionContext function implemented. A NSC might want to update it if it needs to add, update or delete routes, src IP addresses (VIPs...), policies... Signed-off-by: Lionel Jouin <[email protected]>
LionelJouin
added a commit
to Nordix/nsm-sdk
that referenced
this issue
Mar 3, 2022
Using the Client chain, the network service client cannot update the IPContext without the mergeConnectionContext function implemented. A NSC might want to update it if it needs to add, update or delete routes, src IP addresses (VIPs...), policies... Signed-off-by: Lionel Jouin <[email protected]>
LionelJouin
added a commit
to Nordix/nsm-sdk
that referenced
this issue
Mar 3, 2022
Using the Client chain, the network service client cannot update the IPContext without the mergeConnectionContext function implemented. A NSC might want to update it if it needs to add, update or delete routes, src IP addresses (VIPs...), policies... Signed-off-by: Lionel Jouin <[email protected]>
This was referenced Mar 10, 2022
|
Solved: #1238 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Overview
Together with the new source routing feature, we would like our NSCs to have the ability to re-configure their connections on the fly, so without having to close, and re-request (mostly to update the IP context). But currently, the begin client chain component is preventing this due a merge function supporting only the extracontext for now.
Here is the link to the function, my proposal would be to not merge the ip contexts and only keep the requested one.
References
networkservicemesh/api#119
https://github.com/networkservicemesh/sdk/blob/main/pkg/networkservice/common/begin/merge.go#L39
The text was updated successfully, but these errors were encountered: