RT-1.27 : Fixing script issues seen after recent changes
Following changes are made -
- Fixing the usage of
deviations.SetMetricAsPreference, i.e. when this deviation is set true, script should configure the Preference with Metric value. - Restoring the original static route as per these steps in readme, which make traffic validation possible - https://github.com/openconfig/featureprofiles/blob/66586143b8e4b24bf65b2db76e3f2dabfd5fdb48/feature/bgp/static_route_bgp_redistribution/otg_tests/static_route_bgp_redistribution_test/README.md?plain=1#L64 https://github.com/openconfig/featureprofiles/blob/66586143b8e4b24bf65b2db76e3f2dabfd5fdb48/feature/bgp/static_route_bgp_redistribution/otg_tests/static_route_bgp_redistribution_test/README.md?plain=1#L72
- Fixing
validateLearnedIPv4PrefixandvalidateLearnedIPv6Prefixto validate rejected-route whenshouldBePresentis set to false for some tests.
"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."
Pull Request Functional Test Report for #3444 / fd146d57d05d853564661710a2eb39b78d08c393
Virtual Devices
| Device | Test | Test Documentation | Job | Raw Log |
|---|---|---|---|---|
| Arista cEOS | RT-1.27: Static route to BGP redistribution |
de1d06a0 | Log | |
| Cisco 8000E | RT-1.27: Static route to BGP redistribution |
56d676bd | Log | |
| Cisco XRd | RT-1.27: Static route to BGP redistribution |
c6af4ac5 | Log | |
| Juniper ncPTX | RT-1.27: Static route to BGP redistribution |
d652251d | Log | |
| Nokia SR Linux | RT-1.27: Static route to BGP redistribution |
f5403500 | Log | |
| Openconfig Lemming | RT-1.27: Static route to BGP redistribution |
a75291fc | Log |
Hardware Devices
| Device | Test | Test Documentation | Raw Log |
|---|---|---|---|
| Arista 7808 | RT-1.27: Static route to BGP redistribution |
||
| Cisco 8808 | RT-1.27: Static route to BGP redistribution |
||
| Juniper PTX10008 | RT-1.27: Static route to BGP redistribution |
||
| Nokia 7250 IXR-10e | RT-1.27: Static route to BGP redistribution |
Pull Request Test Coverage Report for Build 14621484863
Details
- 0 of 0 changed or added relevant lines in 0 files are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage remained the same at 18.016%
| Totals | |
|---|---|
| Change from base Build 14593131142: | 0.0% |
| Covered Lines: | 2481 |
| Relevant Lines: | 13771 |
💛 - Coveralls
/fptest virtual
This test is not passing on any virtual platforms. Please check the log at least for srlinux:
I am getting Forbidden Error 403 while accessing the logs. Can you please share the logs or enable the access to above url ?
/fptest virtual
/fptest virtual
This test is not passing on any virtual platforms. Please check the log at least for srlinux: Log
I am getting
Forbidden Error 403while accessing the logs. Can you please share the logs or enable the access to above url ?
Apologies, looks like this log is only accessible to googlers over github.
HW log with these changes are available at - b/380289416
Hi @trathod1 May I know the status of this PR?
Hi @trathod1 May I know the status of this PR?
This PR is ready for review.
Hi @trathod1 May I know the status of this PR?
This PR is ready for review.
@nsadhasivam - Your attention is requested here
@dplore , can you review this please
/fptest virtual