Skip to content
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

FRR: Parse IPv6 BGP neighbors #6587

Merged
merged 2 commits into from
Jan 25, 2021
Merged

FRR: Parse IPv6 BGP neighbors #6587

merged 2 commits into from
Jan 25, 2021

Conversation

ratulm
Copy link
Member

@ratulm ratulm commented Jan 23, 2021

Fixes #6586

@batfish-bot
Copy link

This change is Reviewable

@codecov
Copy link

codecov bot commented Jan 23, 2021

Codecov Report

Merging #6587 (0f169cd) into master (b7639c5) will decrease coverage by 0.01%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master    #6587      +/-   ##
============================================
- Coverage     73.39%   73.38%   -0.02%     
+ Complexity    35768    35762       -6     
============================================
  Files          2838     2839       +1     
  Lines        144381   144390       +9     
  Branches      17475    17475              
============================================
- Hits         105971   105958      -13     
- Misses        30004    30020      +16     
- Partials       8406     8412       +6     
Impacted Files Coverage Δ Complexity Δ
...ish/representation/cumulus/CumulusConversions.java 91.77% <0.00%> (ø) 191.00 <0.00> (ø)
...atfish/representation/cumulus/BgpIpv6Neighbor.java 75.00% <75.00%> (ø) 1.00 <1.00> (?)
...ar/cumulus_frr/CumulusFrrConfigurationBuilder.java 75.19% <100.00%> (+0.19%) 170.00 <3.00> (+2.00)
...rg/batfish/identifiers/StorageBasedIdResolver.java 85.29% <0.00%> (-5.89%) 22.00% <0.00%> (ø%)
.../org/batfish/dataplane/rib/RouteAdvertisement.java 83.67% <0.00%> (-4.09%) 18.00% <0.00%> (-1.00%)
...org/batfish/datamodel/flow/BidirectionalTrace.java 81.81% <0.00%> (-2.28%) 14.00% <0.00%> (-1.00%)
...src/main/java/org/batfish/coordinator/PoolMgr.java 59.52% <0.00%> (-1.20%) 15.00% <0.00%> (-1.00%)
...ain/java/org/batfish/storage/FileBasedStorage.java 86.19% <0.00%> (-0.68%) 249.00% <0.00%> (ø%)
...java/org/batfish/dataplane/ibdp/VirtualRouter.java 90.45% <0.00%> (-0.32%) 206.00% <0.00%> (-1.00%)
...a/org/batfish/representation/aws/LoadBalancer.java 82.18% <0.00%> (-0.32%) 70.00% <0.00%> (-1.00%)
... and 3 more

Copy link
Member

@dhalperi dhalperi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 6 of 6 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @arifogel and @ratulm)


projects/batfish/src/main/antlr4/org/batfish/grammar/cumulus_frr/CumulusFrr_bgp.g4, line 211 at r1 (raw file):

  (
     ip = IP_ADDRESS
     | ip6 = IPV6_ADDRESS

probably worth factoring out into sbn_ip6 instead of combining here.

@ratulm
Copy link
Member Author

ratulm commented Jan 25, 2021

The last commit (0f169cd) adds parsing for some more IPv6 related stuff to make the user file behave alright.

Copy link
Member

@dhalperi dhalperi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 6 of 6 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @arifogel)

@dhalperi dhalperi merged commit 612b9e1 into master Jan 25, 2021
@dhalperi dhalperi deleted the frr-bgp-v6-neighbors branch January 25, 2021 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FRR: NPE with IPv6 BGP neighbor
3 participants