openconfig
TE-16.2: encapsulation FRR scenarios changes incorporated.
Updated metadata.textproto
- Added ExplicitInterfaceInDefaultVRF
- Added NoEcmpWithEncapDecapNhMix
Updated gribi_route_test.go
- Add the 0/0 route in encap_te_vrf_a as part of gribi base config so that it will be part of all tests.
- Deviation NoEcmpWithEncapDecapNhMix added for TC2-primaryBackupSingle & TC3-primaryBackupRoutingSingle.
Updated README.md
- fallback route to redirect to DEFAULT vrf.
Updated Deviation.go ,metadata.proto & metadata.pb.go
A new deviation has been included due to mixed encap and decap next hops are not supported.
"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 #4270 / 69b0b406089b3f5b64a6d069dc8365434bf4e332
Virtual Devices
| Device | Test | Test Documentation | Job | Raw Log |
|---|---|---|---|---|
| Arista cEOS |  |
TE-16.2: encapsulation FRR scenarios |
||
| Cisco 8000E |  |
TE-16.2: encapsulation FRR scenarios |
||
| Cisco XRd |  |
TE-16.2: encapsulation FRR scenarios |
||
| Juniper ncPTX |  |
TE-16.2: encapsulation FRR scenarios |
||
| Nokia SR Linux |  |
TE-16.2: encapsulation FRR scenarios |
||
| Openconfig Lemming |  |
TE-16.2: encapsulation FRR scenarios |
Hardware Devices
| Device | Test | Test Documentation | Raw Log |
|---|---|---|---|
| Arista 7808 |  |
TE-16.2: encapsulation FRR scenarios |
|
| Cisco 8808 |  |
TE-16.2: encapsulation FRR scenarios |
|
| Juniper PTX10008 |  |
TE-16.2: encapsulation FRR scenarios |
|
| Nokia 7250 IXR-10e |  |
TE-16.2: encapsulation FRR scenarios |
Pull Request Test Coverage Report for Build 19360368611
Details
- 0 of 12 (0.0%) changed or added relevant lines in 3 files are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage decreased (-0.004%) to 10.288%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
|---|---|---|---|
| internal/vrfpolicy/vrfpolicy.go | 0 | 1 | 0.0% |
| proto/metadata_go_proto/metadata.pb.go | 0 | 5 | 0.0% |
| internal/deviations/deviations.go | 0 | 6 | 0.0% |
| <!-- | Total: | 0 | 12 |
| Totals | |
|---|---|
| Change from base Build 19357318501: | -0.004% |
| Covered Lines: | 2224 |
| Relevant Lines: | 21618 |
💛 - Coveralls
PR in draft state, please share the changes for review once the changes are pushed
@nikki-blr Please assign @openconfig/featureprofiles-owner-gribi to review the PR
Thank you ampattan@! there is one more unintentional change I think?
Just to point out - the main question I have is around finding a reference to the underlying discussion for the new deviation - I want to understand whether this is planned to be fixed in the future images and the timeline for the fix (see issue 437880737). Thank you!
@gggsmith Nokia team is currently discussing this. We’ll provide an update along with the respective Buganizer. Thank you!
