-
Notifications
You must be signed in to change notification settings - Fork 228
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
Added support for parsing NX-OS NetFlow stanzas #5646
Added support for parsing NX-OS NetFlow stanzas #5646
Conversation
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 8 of 8 files at r1, 1 of 1 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @agember)
projects/batfish/src/test/resources/org/batfish/grammar/cisco_nxos/testconfigs/nxos_flow, line 8 at r1 (raw file):
source Ethernet1/1
Add as interface reference?
@arifogel is it obvious the right way to do this? There's no previous keyword to look at...
Codecov Report
@@ Coverage Diff @@
## master #5646 +/- ##
============================================
- Coverage 73.78% 73.78% -0.01%
- Complexity 32899 32905 +6
============================================
Files 2667 2667
Lines 131612 131660 +48
Branches 15704 15707 +3
============================================
+ Hits 97106 97139 +33
- Misses 26840 26852 +12
- Partials 7666 7669 +3
|
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 3 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved
Parsing support for this was motivated by the inclusion of a version statement in a flow exporter stanza. Without this support, parser recovery treats the version statement as a top-level version statement (which I believe negatively impacts vendor detection). As an added benefit, there is now missing reference tracking for NetFlow-related configuration.