-
Notifications
You must be signed in to change notification settings - Fork 36
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 DualStack IPPool that support both IPv4 and IPv6 addresses #1659
Add DualStack IPPool that support both IPv4 and IPv6 addresses #1659
Conversation
Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
9c598fe
to
3c049f0
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1659 +/- ##
=======================================
Coverage ? 67.78%
=======================================
Files ? 265
Lines ? 12626
Branches ? 0
=======================================
Hits ? 8559
Misses ? 3537
Partials ? 530 ☔ View full report in Codecov by Sentry. |
Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
3c049f0
to
8eb6043
Compare
Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
60f895d
to
188832f
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.
As I know @Ex4amp1e has found a few problems with these changes during testing networkservicemesh/deployments-k8s#11371
So @Ex4amp1e is planned to continue this PR.
Taken the main part from here and implemented in the scope of the #1679 PR. |
Done in #1679 |
Description
Add a dual stack ippool that support both IPv4 and IPv6 addresses. It fixes some errors in
strictipam
chain elementIssue link
How Has This Been Tested?
Types of changes