Adding MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop Test
Readme Location: https://github.com/openconfig/featureprofiles/blob/main/feature/mpls/otg_tests/static_bgp_nexthop/README.md
Since this is the first time I have raised an issue with deviation, I'm not sure if the metadata pb file should be added or will be generated automatically. Please let me know if the generated pb file should be added as part of the PR.
Pull Request Functional Test Report for #4131 / c9384b912111f5b0446fb6165f04fee3efbc8364
Virtual Devices
| Device | Test | Test Documentation | Job | Raw Log |
|---|---|---|---|---|
| Arista cEOS | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
|||
| Cisco 8000E | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
|||
| Cisco XRd | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
|||
| Juniper ncPTX | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
|||
| Nokia SR Linux | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
|||
| Openconfig Lemming | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
Hardware Devices
| Device | Test | Test Documentation | Raw Log |
|---|---|---|---|
| Arista 7808 | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
||
| Cisco 8808 | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
||
| Juniper PTX10008 | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
||
| Nokia 7250 IXR-10e | MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop. |
I think the deviation for static LSP's is already defined, please reuse that, and modify your cfgplugin to return a gnmi.BatchReplace. Then a gnmi.Set (example) to push the config to the DUT in your _test.go code.
Updated gnmi batch.
Pull Request Test Coverage Report for Build 16666337916
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
- For more information on this, see Tracking coverage changes with pull request builds.
- To avoid this issue with future PRs, see these Recommended CI Configurations.
- For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
Details
- 0 of 87 (0.0%) changed or added relevant lines in 3 files are covered.
- 16 unchanged lines in 1 file lost coverage.
- Overall coverage decreased (-0.06%) to 13.553%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
|---|---|---|---|
| internal/deviations/deviations.go | 0 | 3 | 0.0% |
| internal/cfgplugins/mpls.go | 0 | 11 | 0.0% |
| proto/metadata_go_proto/metadata.pb.go | 0 | 73 | 0.0% |
| <!-- | Total: | 0 | 87 |
| Files with Coverage Reduction | New Missed Lines | % |
|---|---|---|
| internal/cfgplugins/sflow.go | 16 | 0.0% |
| <!-- | Total: | 16 |
| Totals | |
|---|---|
| Change from base Build 16610268795: | -0.06% |
| Covered Lines: | 2093 |
| Relevant Lines: | 15443 |