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
Cumulus/FRR: split NCLU and FRR #6630
Conversation
- NCLU from FRR side - Concat, Frr, Interfaces, Ports from NCLU side * Rewrite some VS tests with the appropriate vendor * Delete InterfaceConverter in NCLU side
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 108 of 113 files at r1.
Reviewable status: 96 of 118 files reviewed, all discussions resolved (waiting on @dhalperi)
They were always unused, just became exposed by this work
Codecov Report
@@ Coverage Diff @@
## master #6630 +/- ##
============================================
- Coverage 73.41% 73.37% -0.04%
- Complexity 35871 36365 +494
============================================
Files 2847 2916 +69
Lines 144720 146708 +1988
Branches 17514 17686 +172
============================================
+ Hits 106249 107650 +1401
- Misses 30047 30567 +520
- Partials 8424 8491 +67
|
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 113 files at r1, 25 of 27 files at r2, 23 of 23 files at r4, 23 of 24 files at r5, 4 of 4 files at r6, 1 of 1 files at r7.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @anothermattbrown and @sfraint)
These are two completely different implementations, and the weird things we've done to
make them one are holding us back.
cumulus_nclu
package.CumulusNodeConfiguration
abstraction