-
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
XR: parse and warn on as-path-set passes-through #6008
Conversation
To get this implemented correctly, we need to do a similar overhaul to AS matching as we did to Community matching. But this prevents a conversion error.
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 4 of 4 files at r1.
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 1 of 1 files at r2.
Reviewable status:complete! all files reviewed, all discussions resolved
Codecov Report
@@ Coverage Diff @@
## master #6008 +/- ##
============================================
- Coverage 72.23% 72.22% -0.01%
+ Complexity 34329 34324 -5
============================================
Files 2810 2810
Lines 140882 140887 +5
Branches 16928 16928
============================================
- Hits 101761 101759 -2
- Misses 31159 31164 +5
- Partials 7962 7964 +2
|
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 1 of 1 files at r2.
Reviewable status:complete! all files reviewed, all discussions resolved
To get this implemented correctly, we need to do a similar overhaul to AS matching as we did to Community matching. But this prevents a conversion error. Fix batfish#6005
To get this implemented correctly, we need to do a similar overhaul to
AS matching as we did to Community matching. But this prevents a conversion error.
Fix #6005.