featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

RT-1.53 fix

Open MarcCharlebois opened this issue 1 year ago • 2 comments

  • delete pfx2 from var as it was never "replaced" - see below Before the change, we can see that the var was never replaced and always carried pfx2 which seems to not be the intended behavior by the author.
    prefix_set_test.go:65: Printing the value after re-assignment, &{out-of-range E_PrefixSet_Mode enum value: 0 0xc001e93e80 map[{10.240.31.48/28 exact}:0xc001e93eb0 {173.36.128.0/20 exact}:0xc001e93ee0]}
    prefix_set_test.go:69: Printing the value after L68, &{out-of-range E_PrefixSet_Mode enum value: 0 0xc001e93e80 map[{10.240.31.48/28 exact}:0xc001e93eb0 {173.36.128.0/20 exact}:0xc001e93ee0 {173.36.144.0/20 exact}:0xc002171a90]}
  • add deviation

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.

MarcCharlebois avatar Jul 25 '24 19:07 MarcCharlebois

Pull Request Functional Test Report for #3327 / 22d373c96ae63678ab5c9156e6ddea7b2317a74e

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
RT-1.53: prefix-list test
Cisco 8000E status
RT-1.53: prefix-list test
Cisco XRd status
RT-1.53: prefix-list test
Juniper ncPTX status
RT-1.53: prefix-list test
Nokia SR Linux status
RT-1.53: prefix-list test
Openconfig Lemming status
RT-1.53: prefix-list test

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
RT-1.53: prefix-list test
Cisco 8808 status
RT-1.53: prefix-list test
Juniper PTX10008 status
RT-1.53: prefix-list test
Nokia 7250 IXR-10e status
RT-1.53: prefix-list test

Help

OpenConfigBot avatar Jul 25 '24 19:07 OpenConfigBot

Pull Request Test Coverage Report for Build 10100456332

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

💛 - Coveralls

coveralls avatar Jul 25 '24 19:07 coveralls