-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
tests: iproute2 VRF capability check for topotests #8253
tests: iproute2 VRF capability check for topotests #8253
Conversation
Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
Example usage of iproute2_is_vrf_capable check in bgp_multi_vrf_topo1 and bgp_multi_vrf_topo2 topotests. Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
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.
Thanks for your contribution to FRR!
Pylint found errors in source files changed by this PR:
Pylint report for my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:
************* Module test_bgp_multi_vrf_topo1
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:2677:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:2683:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:4958:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5170:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5449:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5853:8: E1305: Too many arguments for format string (too-many-format-args)
-----------------------------------
Your code has been rated at 9.73/10
Pylint report for my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:
************* Module test_bgp_multi_vrf_topo2
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1572:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1617:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1622:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1862:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:2179:27: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:3388:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:3394:8: E1305: Too many arguments for format string (too-many-format-args)
-----------------------------------
Your code has been rated at 9.58/10
Pylint report for my_frr/tests/topotests/lib/topotest.py:
************* Module lib.topotest
my_frr/tests/topotests/lib/topotest.py:1889:15: E0602: Undefined variable 'unicode' (undefined-variable)
-----------------------------------
Your code has been rated at 9.95/10
If you are a new contributor to FRR, please see our contributing guidelines.
After making changes, you do not need to create a new PR. You should perform an amend or interactive rebase followed by a force push.
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 arm8 part 7: Failed (click for details)Topotests Ubuntu 18.04 arm8 part 7: No useful log foundSuccessful on other platforms/tests
|
Outdated results 💚Basic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Additional example usage of iproute2_is_vrf_capable check in isis-topo1-vrf topotest. Signed-off-by: David Schweizer <dschweizer@opensourcerouting.org>
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.
Thanks for your contribution to FRR!
Pylint found errors in source files changed by this PR:
Pylint report for my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:
************* Module test_bgp_multi_vrf_topo1
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:2677:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:2683:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:4958:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5170:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5449:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5853:8: E1305: Too many arguments for format string (too-many-format-args)
-----------------------------------
Your code has been rated at 9.73/10
Pylint report for my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:
************* Module test_bgp_multi_vrf_topo2
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1572:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1617:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1622:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1862:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:2179:27: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:3388:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:3394:8: E1305: Too many arguments for format string (too-many-format-args)
-----------------------------------
Your code has been rated at 9.58/10
Pylint report for my_frr/tests/topotests/lib/topotest.py:
************* Module lib.topotest
my_frr/tests/topotests/lib/topotest.py:1889:15: E0602: Undefined variable 'unicode' (undefined-variable)
-----------------------------------
Your code has been rated at 9.95/10
If you are a new contributor to FRR, please see our contributing guidelines.
After making changes, you do not need to create a new PR. You should perform an amend or interactive rebase followed by a force push.
Outdated results 💚Basic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 arm8 part 4: Failed (click for details)Topotests Ubuntu 18.04 arm8 part 4: No useful log foundTopotests Ubuntu 16.04 amd64 part 7: Failed (click for details)Topology Test Results are at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-TOPO7U16AMD64-17693/test Topology Tests failed for Topotests Ubuntu 16.04 amd64 part 7:
see full log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-17693/artifact/TOPO7U16AMD64/ErrorLog/log_topotests.txt Topotests Ubuntu 18.04 amd64 part 7: Failed (click for details)Topology Test Results are at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-TOPO7U18AMD64-17693/test Topology Tests failed for Topotests Ubuntu 18.04 amd64 part 7:
see full log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-17693/artifact/TOPO7U18AMD64/ErrorLog/log_topotests.txt Topotests Ubuntu 18.04 arm8 part 7: Failed (click for details)Topotests Ubuntu 18.04 arm8 part 7: No useful log foundSuccessful on other platforms/tests
|
0b04146
to
dd102f6
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.
Thanks for your contribution to FRR!
Pylint found errors in source files changed by this PR:
Pylint report for my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:
************* Module test_bgp_multi_vrf_topo1
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:2677:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:2683:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:4958:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5170:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5449:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py:5853:8: E1305: Too many arguments for format string (too-many-format-args)
-----------------------------------
Your code has been rated at 9.73/10
Pylint report for my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:
************* Module test_bgp_multi_vrf_topo2
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1572:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1617:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1622:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:1862:12: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:2179:27: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:3388:8: E1305: Too many arguments for format string (too-many-format-args)
my_frr/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py:3394:8: E1305: Too many arguments for format string (too-many-format-args)
-----------------------------------
Your code has been rated at 9.58/10
Pylint report for my_frr/tests/topotests/lib/topotest.py:
************* Module lib.topotest
my_frr/tests/topotests/lib/topotest.py:1889:15: E0602: Undefined variable 'unicode' (undefined-variable)
-----------------------------------
Your code has been rated at 9.95/10
If you are a new contributor to FRR, please see our contributing guidelines.
After making changes, you do not need to create a new PR. You should perform an amend or interactive rebase followed by a force push.
Outdated results 💚Basic BGPD CI results: SUCCESS, 0 tests failed
For details, please contact louberger |
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 16.04 i386 part 9: Failed (click for details)Topology Test Results are at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-TOPO9U16I386-17785/test Topology Tests failed for Topotests Ubuntu 16.04 i386 part 9:
see full log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-17785/artifact/TOPO9U16I386/ErrorLog/log_topotests.txt Topotests Ubuntu 16.04 amd64 part 5: Failed (click for details)Topology Test Results are at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-TOPO5U16AMD64-17785/test Topology Tests failed for Topotests Ubuntu 16.04 amd64 part 5:
see full log at https://ci1.netdef.org/browse/FRR-FRRPULLREQ-17785/artifact/TOPO5U16AMD64/ErrorLog/log_topotests.txt Successful on other platforms/tests
|
I understand kernel check is not enough, and iproute2 should be added ? |
dd102f6
to
83d2076
Compare
💚 Basic BGPD CI results: SUCCESS, 0 tests failedResults table
For details, please contact louberger |
Continuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-17906/ This is a comment from an automated CI system. |
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.
looks good
@riw777 Thank you, Russ. |
On older releases of certain distributions (i.e. Ubuntu 16.04 LTS) the
iproute2
utility does not have support for VRFs. This can lead to test failures on systems with backported kernels (Ubuntu HWE, Debian backports, etc.), since the tests are skipped solely based on kernel versions. This PR introduces a check for VRF handling capabilities on theiproute2
utility present in the PATH and example applications in thebgp_multi_vrf_topo1
andbgp_multi_vrf_topo2
topotests.