-
Notifications
You must be signed in to change notification settings - Fork 157
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
Add export policy test cases to link bandwidth test #3522
Conversation
Pull Request Functional Test Report for #3522 / ed7765cVirtual Devices
Hardware Devices
|
Pull Request Test Coverage Report for Build 11851311028Details
💛 - Coveralls |
a2bf73a
to
0fe0d54
Compare
This PR messed up formating of README.md. Can you please fix it. |
7558e45
to
b6cb543
Compare
3904be6
to
ed1643a
Compare
feature/bgp/policybase/otg_tests/link_bandwidth_test/link_bandwidth_test.go
Show resolved
Hide resolved
feature/bgp/policybase/otg_tests/link_bandwidth_test/link_bandwidth_test.go
Outdated
Show resolved
Hide resolved
6ce75d1
to
672acc6
Compare
672acc6
to
2ef277b
Compare
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.
Export-policy mixup has been resolved. Can we remove fptest.LogQuery(t, "", gnmi.OC().RoutingPolicy().Config(), root) at line 844 which prints detailed routing-policy on console, if it is not necessary.
/cpbrun |
We think this is needed for troubleshooting if the test fails. It avoids needing to rerun and manual step debugging for bug reports. |
* Add export policy test cases to link bandwidth test * update not_match_100_set_linkbw_1M policy to add statement to remove any LBw before adding the new LBw * added both kbps and KBps comparison * removed old import and export policies --------- Co-authored-by: rszarecki <[email protected]>
* Add export policy test cases to link bandwidth test * update not_match_100_set_linkbw_1M policy to add statement to remove any LBw before adding the new LBw * added both kbps and KBps comparison * removed old import and export policies --------- Co-authored-by: rszarecki <[email protected]>
* Add export policy test cases to link bandwidth test * update not_match_100_set_linkbw_1M policy to add statement to remove any LBw before adding the new LBw * added both kbps and KBps comparison * removed old import and export policies --------- Co-authored-by: rszarecki <[email protected]>
No description provided.