featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

RT-5.8: Improve DUT Interface Stats Polling

Open karthikeya-remilla opened this issue 10 months ago • 3 comments

Description: This PR addresses the issue identified in 397859196

Proposed Changes:

  1. Replace the default target-defined subscription with a sampling subscription to enable more proactive polling of the DUT for interface statistics. This change ensures that atleast 100 packets are received on the DUT, verifying that the packets transmitted by the OTG are successfully received.

  2. Implement a sampling subscription instead of using a static time.Sleep for verifying packet reception by the DUT (recvDUTPkts). A timeout of 60 seconds is set to allow sufficient time for the DUT interfaces to report statistics accurately.

Note: This PR diff is similar to the diff of #3536. Was advised to avoid using deviations.InterfaceCountersUpdateDelayed during PR review of #3614

karthikeya-remilla avatar Mar 14 '25 08:03 karthikeya-remilla

Pull Request Functional Test Report for #4002 / 3f48ef86ca4576b0b48dc7e076fac833230cd1c7

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Cisco 8000E status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Cisco XRd status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Juniper ncPTX status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Nokia SR Linux status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Openconfig Lemming status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Cisco 8808 status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Juniper PTX10008 status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local
Nokia 7250 IXR-10e status
status
RT-5.8: IPv6 Link Local
RT-5.8: IPv6 link local

Help

OpenConfigBot avatar Mar 14 '25 08:03 OpenConfigBot

Pull Request Test Coverage Report for Build 13987237845

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.124%

Totals Coverage Status
Change from base Build 13985265235: 0.0%
Covered Lines: 2479
Relevant Lines: 13678

💛 - Coveralls

coveralls avatar Mar 14 '25 08:03 coveralls

The test fails. Updated log in issue tracker.

AmrNJ avatar Apr 11 '25 03:04 AmrNJ

@karthikeya-remilla can you please check as test is failing we can't merge this.

nikki-blr avatar Apr 25 '25 04:04 nikki-blr