-
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
Cisco interface parsing: allow secondary HSRP address #4186
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4186 +/- ##
============================================
- Coverage 75.32% 75.31% -0.01%
+ Complexity 24947 24944 -3
============================================
Files 2046 2046
Lines 99604 99604
Branches 11839 11839
============================================
- Hits 75022 75016 -6
- Misses 19209 19212 +3
- Partials 5373 5376 +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 4 of 4 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @progwriter)
a discussion (no related file):
What impact does this have? I see no data model changes nor tests. Do we not process HSRP currently? Are you ignoring distinction between primary and secondary HSRP addresses?
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: all files reviewed, 1 unresolved discussion (waiting on @arifogel)
a discussion (no related file):
Previously, arifogel (Ari Fogel) wrote…
What impact does this have? I see no data model changes nor tests. Do we not process HSRP currently? Are you ignoring distinction between primary and secondary HSRP addresses?
The extractor has no logic of any if_hsrp
stanzas; we only handle standby
stanzas. Cursory search suggests it is an ios/nxos difference.
Happy to flesh out extraction/conversion, but best in separate PR.
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: complete! all files reviewed, all discussions resolved
No description provided.