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
Batfish: unusableVlanShutdown should only count active interfaces #6368
Conversation
If an interface is shutdown, it will not keep a Vlan up. (The autostate flag is still supported below).
Codecov Report
@@ Coverage Diff @@
## master #6368 +/- ##
============================================
- Coverage 72.98% 72.97% -0.02%
+ Complexity 35164 35155 -9
============================================
Files 2829 2829
Lines 142779 142779
Branches 17160 17160
============================================
- Hits 104212 104191 -21
- Misses 30336 30351 +15
- Partials 8231 8237 +6 |
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 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.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dhalperi)
a discussion (no related file):
test
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: 1 of 4 files reviewed, all discussions resolved
a discussion (no related file):
Previously, dhalperi (Dan Halperin) wrote…
test
Done.
If an interface is shutdown, it will not keep a Vlan up. (The autostate flag is
still supported below).