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
Fixed changePodIp func #559
Conversation
Hi @davidefalcone1. Thanks for your PR! I am @adamjensenbot.
Make sure this PR appears in the liqo changelog, adding one of the following labels:
|
/ok-to-test |
Hi @davidefalcone1. Thanks for your PR! I'm waiting for a Liqo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test. Once the patch is verified, the new status will be reflected by the ok-to-test label. |
a31588d
to
cd8f9cf
Compare
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.
/lgtm
/rebase |
newPodIpBuilder.WriteString(s) | ||
newPodIpBuilder.WriteString(".") | ||
} | ||
// Parse newPodCidr |
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.
The following logic may deserve to be in a separate function
Description
Function changePodIp forges a new IP taking the first 2 octets from the newPodCidr and the last 2 from the original Pod Ip. Since Liqo does support networks with random masks, this function needs to be modified: thanks to this PR, the function now takes a number of bits equal to the mask length from the newPodCidr and 32-mask bits from the old Pod IP.