-
Notifications
You must be signed in to change notification settings - Fork 229
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
Re-add "plain" tunnels in post-DP L3 topology #4280
Conversation
a discussion (no related file): |
Codecov Report
@@ Coverage Diff @@
## master #4280 +/- ##
============================================
+ Coverage 75.49% 75.5% +<.01%
- Complexity 25825 25855 +30
============================================
Files 2106 2107 +1
Lines 102384 102500 +116
Branches 12276 12292 +16
============================================
+ Hits 77293 77390 +97
- Misses 19444 19453 +9
- Partials 5647 5657 +10
|
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.
Reviewed 7 of 15 files at r1.
Reviewable status: 7 of 15 files reviewed, 2 unresolved discussions (waiting on @haverma and @progwriter)
a discussion (no related file):
changes make sense and same as the offline discussion:
- source and destination should match in the tunnels to make them eligible to be in the initial tunnel topology and subsequently in the pruned one.
- traceroute should be done between source addresses of the two tunnel interfaces
Keep a topology of non-IPsec tunnels, re-add edges during dataplane computation.
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.
Reviewable status: 6 of 15 files reviewed, all discussions resolved (waiting on @haverma)
a discussion (no related file):
Previously, haverma (Harsh Verma) wrote…
changes make sense and same as the offline discussion:
- source and destination should match in the tunnels to make them eligible to be in the initial tunnel topology and subsequently in the pruned one.
- traceroute should be done between source addresses of the two tunnel interfaces
Done
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.
Reviewed 3 of 15 files at r1, 6 of 6 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved
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.
once unit tests pass
Reviewable status: complete! all files reviewed, all discussions resolved
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.
Reviewed 6 of 6 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved
Keep a topology of non-IPsec tunnels, re-add edges during dataplane
computation.