featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

corrected deviation usage and isis await timeout

Open bkreddy143 opened this issue 1 year ago • 2 comments

  1. Corrected use of deviation as per PR-2953 , i.e deviation is need to skip ecmp config .
  2. Modified gnmi Await timeout to 90 seconds to verify ISIS session state , this to accommodated 3 hello packets drop interval
  3. Using LACP System ID as destination MAC in flow creation is not reliable so changed to get interface MAC while creating traffic flow.
  4. Added additional check to verify either session DOWN or no adjacencies presence.

"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

bkreddy143 avatar Jul 17 '24 03:07 bkreddy143

Pull Request Functional Test Report for #3304 / f07fbfc4c1486e5c7dbb8225cff892a86b42ea00

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
RT-5.7: Aggregate Not Viable All
Cisco 8000E status
RT-5.7: Aggregate Not Viable All
Cisco XRd status
RT-5.7: Aggregate Not Viable All
Juniper ncPTX status
RT-5.7: Aggregate Not Viable All
Nokia SR Linux status
RT-5.7: Aggregate Not Viable All
Openconfig Lemming status
RT-5.7: Aggregate Not Viable All

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
RT-5.7: Aggregate Not Viable All
Cisco 8808 status
RT-5.7: Aggregate Not Viable All
Juniper PTX10008 status
RT-5.7: Aggregate Not Viable All
Nokia 7250 IXR-10e status
RT-5.7: Aggregate Not Viable All

Help

OpenConfigBot avatar Jul 17 '24 03:07 OpenConfigBot

Pull Request Test Coverage Report for Build 10087116237

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 10085646249: 0.0%
Covered Lines: 1983
Relevant Lines: 3573

💛 - Coveralls

coveralls avatar Jul 17 '24 03:07 coveralls