featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

Fix P4RT-5.2: Traceroute Packetout regression

Open prakashbadri-arista opened this issue 1 year ago • 3 comments

The test started failing with the following commit https://github.com/openconfig/featureprofiles/commit/e8db60afb7a6e044e5d7f63318f59d55ccdde43f

The problem with the change was that the test was waiting for ARP on the OTG side even before the call to otg.StartProtocols() and as a result the ip address was not configured on the OTG interfaces and the neighbor entry was missing.

prakashbadri-arista avatar Jul 29 '24 06:07 prakashbadri-arista

Pull Request Functional Test Report for #3337 / ec6409e3f25531b39603cf859b582fd1781cf8e3

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
P4RT-5.2: Traceroute Packetout
Cisco 8000E status
P4RT-5.2: Traceroute Packetout
Cisco XRd status
P4RT-5.2: Traceroute Packetout
Juniper ncPTX status
P4RT-5.2: Traceroute Packetout
Nokia SR Linux status
P4RT-5.2: Traceroute Packetout
Openconfig Lemming status
P4RT-5.2: Traceroute Packetout

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
P4RT-5.2: Traceroute Packetout
Cisco 8808 status
P4RT-5.2: Traceroute Packetout
Juniper PTX10008 status
P4RT-5.2: Traceroute Packetout
Nokia 7250 IXR-10e status
P4RT-5.2: Traceroute Packetout

Help

OpenConfigBot avatar Jul 29 '24 06:07 OpenConfigBot

Pull Request Test Coverage Report for Build 10138980991

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 55.5%

Totals Coverage Status
Change from base Build 10138481862: 0.0%
Covered Lines: 1983
Relevant Lines: 3573

💛 - Coveralls

coveralls avatar Jul 29 '24 06:07 coveralls

https://github.com/openconfig/featureprofiles/pull/3251 already addresses that

MarcCharlebois avatar Jul 31 '24 15:07 MarcCharlebois

Changes in this PR is already covered in https://github.com/openconfig/featureprofiles/pull/3251/files which is merged already. Hence closing this PR.

ram-mac avatar Jan 16 '25 02:01 ram-mac

Closing this as the changes are already in the main now.

ram-mac avatar Jan 16 '25 02:01 ram-mac