featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

Adding TE-18.3: MPLS in UDP Encapsulation Scale Test

Open keysightgems opened this issue 4 months ago • 7 comments

Readme Location: https://github.com/openconfig/featureprofiles/blob/main/feature/gribi/mpls_in_udp/otg_tests/mpls_in_udp_scale/README.md

The log is attached here: https://partnerissuetracker.corp.google.com/issues/415458482

keysightgems avatar Aug 23 '25 21:08 keysightgems

Pull Request Test Coverage Report for Build 20291377695

Details

  • 0 of 54 (0.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 10.012%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/gribi/gribi.go 0 2 0.0%
internal/cfgplugins/networkinstance.go 0 6 0.0%
internal/iputil/iputil.go 0 6 0.0%
internal/cfgplugins/dut_initialize.go 0 20 0.0%
internal/cfgplugins/interface.go 0 20 0.0%
<!-- Total: 0 54
Totals Coverage Status
Change from base Build 20251926744: -0.02%
Covered Lines: 2227
Relevant Lines: 22243

💛 - Coveralls

coveralls avatar Aug 23 '25 21:08 coveralls

@henryhu-G, @cpubob Addressed all the comments and committed the code. Please find the latest updated logs here https://partnerissuetracker.corp.google.com/issues/415458482

keysightgems avatar Sep 23 '25 17:09 keysightgems

@cpubob, @abansod-dev Addressed all the comments and fixed.

keysightgems avatar Oct 06 '25 16:10 keysightgems

@abansod-dev has filed a bug b/450015270 and created PR #4683 to update the README for single VRF configurations. The update clarifies that IP entries must be in a non-default VRF, while the Next-Hop (NH) and Next-Hop Group (NHG) must be in the default VRF. This is necessary to facilitate a change for Keysight in PR #4515. The new PR is currently awaiting approval.

ksgireesha avatar Oct 08 '25 05:10 ksgireesha

@abansod-dev, @cpubob Updated the code as per the Readme update (PR https://github.com/openconfig/featureprofiles/pull/4683) and addressed the above review comments as well.

keysightgems avatar Oct 13 '25 12:10 keysightgems

@abansod-dev Addressed all the comments and fixed.

Noticed that the traffic is not flowing through the non-default VRF and is always taking the default VRF path. I have raised issue 455792904 and informed the vendor team. I will apply the fix once a solution is provided.

keysightgems avatar Oct 28 '25 16:10 keysightgems

Fixed the traffic issue but not able to capture the complete test pass logs due to the gRIBI connection timeout (mentioned details in defect 455792904).

Attached individual test case logs here https://partnerissuetracker.corp.google.com/u/1/issues/415458482

keysightgems avatar Dec 02 '25 10:12 keysightgems

@cpubob Addressed all the comments and fixed.

keysightgems avatar Dec 11 '25 10:12 keysightgems