featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

Adding MPLS-2.2: MPLS forwarding via static LSP to BGP next-hop Test

Open keysightgems opened this issue 9 months ago • 1 comments

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.

keysightgems avatar Apr 15 '25 05:04 keysightgems

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.

keysightgems avatar Jul 29 '25 15:07 keysightgems

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.

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 Coverage Status
Change from base Build 16610268795: -0.06%
Covered Lines: 2093
Relevant Lines: 15443

💛 - Coveralls

coveralls avatar Aug 01 '25 05:08 coveralls