featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

RT-5.7: Aggregate Not Viable All changes incorporated.

Open ampattan opened this issue 3 months ago • 2 comments

  • Introduced a deviation so the script uses interface names and MAC next-hops at the first-level next-hops.
  • Removed vendor specific load-balancing checks. Vendors are expected to manage unsupported values using deviations. Refer the PR comment for details. https://github.com/openconfig/featureprofiles/pull/3550/files/fdab107be3d5b0c617e386d513d33dfb85a5f73a

"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."

ampattan avatar Sep 22 '25 18:09 ampattan

Pull Request Functional Test Report for #4629 / 1f68390f073aa5c9d72828e060e2b3f0193963b6

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 Sep 22 '25 18:09 OpenConfigBot

Pull Request Test Coverage Report for Build 20299659200

Details

  • 0 of 8 (0.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.004%) to 10.03%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/deviations/deviations.go 0 3 0.0%
proto/metadata_go_proto/metadata.pb.go 0 5 0.0%
<!-- Total: 0 8
Totals Coverage Status
Change from base Build 20251926744: -0.004%
Covered Lines: 2227
Relevant Lines: 22203

💛 - Coveralls

coveralls avatar Sep 22 '25 18:09 coveralls

/fptest physical

AmrNJ avatar Nov 14 '25 11:11 AmrNJ

/gemini review

dplore avatar Dec 17 '25 19:12 dplore

@ampattan Please can you resolve Gemini comments?

AmrNJ avatar Dec 18 '25 05:12 AmrNJ