-
Notifications
You must be signed in to change notification settings - Fork 240
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
add prefix length from mtpnc to podipinfo & gatewayip #2828
Conversation
a164ec8
to
256d1d3
Compare
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Kshitija Murudi <kmurudi@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there is too much copy/pasted between the linux and windows versions. Let's keep one common function for the majority of the code, and then create windows/linux versions only for the part that is different.
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
02fca1e
to
f4185ac
Compare
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
Reason for Change:
AKS Middleware for Swiftv2 requires us to add the correct subnet-prefix length which was missing & set the gateway IP as well in CNS for pod connectivity to work.
Issue Fixed:
Requirements:
Notes: