-
Notifications
You must be signed in to change notification settings - Fork 370
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
Don't delete the routes which are added for the peer IPv6 gateways #3336
Conversation
b727933
to
c9933e9
Compare
Codecov Report
@@ Coverage Diff @@
## main #3336 +/- ##
==========================================
- Coverage 62.00% 54.34% -7.67%
==========================================
Files 266 373 +107
Lines 26546 51052 +24506
==========================================
+ Hits 16460 27742 +11282
- Misses 8281 20890 +12609
- Partials 1805 2420 +615
Flags with carried forward coverage won't be shown. Click here to find out more.
|
/test-all |
/test-ipv6-all |
c9933e9
to
979ae97
Compare
/test-all |
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.
Thanks @Jexf. LGTM, a minor comment.
IPv6 doesn't support "on-link" route, routes to the peer IPv6 gateways need to be added separately. So don't delete such routes. Signed-off-by: Wu zhengdong <[email protected]>
/test-all |
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, thanks
/test-integration |
/test-integration |
test-integration failure has been fixed in main branch but this commit doesn't rebase on it. Unfortunately the job tests HEAD commit instead of merge commit, so no point to keep retrying. |
IPv6 doesn't support "on-link" route, routes to the peer IPv6 gateways need to be added separately. So don't delete such routes.
Fixes: #3369
Signed-off-by: Wu zhengdong [email protected]