featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

RT-14.2: gRIBI route test changes incorporated.

Open ampattan opened this issue 11 months ago • 4 comments

  • Updated metadata.textproto

    • Removed explicit_port_speed
    • Added no_fallback_route_in_transit_vrf
  • Updated gribi_route_test.go

    • A deviation has been included to skip TC-14.2.4, as fallback route support is not available in Transit network instances.
    • Removed explicit port speed check.
  • Updated Deviation.go,metadata.proto & metadata.pb.go

    • A new deviation has been included due to the unavailability of fallback route support in Transit network instances.

"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 Feb 13 '25 14:02 ampattan

Pull Request Functional Test Report for #3813 / e1e8985b861896f9e147fb9da06cbb508315519c

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
RT-14.2: GRIBI Route Test
Cisco 8000E status
RT-14.2: GRIBI Route Test
Cisco XRd status
RT-14.2: GRIBI Route Test
Juniper ncPTX status
RT-14.2: GRIBI Route Test
Nokia SR Linux status
RT-14.2: GRIBI Route Test
Openconfig Lemming status
RT-14.2: GRIBI Route Test

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
RT-14.2: GRIBI Route Test
Cisco 8808 status
RT-14.2: GRIBI Route Test
Juniper PTX10008 status
RT-14.2: GRIBI Route Test
Nokia 7250 IXR-10e status
RT-14.2: GRIBI Route Test

Help

OpenConfigBot avatar Feb 13 '25 14:02 OpenConfigBot

HW log with these changes are available at - b/396343715

ampattan avatar Feb 13 '25 14:02 ampattan

@rohit-rp -- please check this PR based on a deviation being added to skip a test.

robshakir avatar Feb 16 '25 07:02 robshakir

@rohit-rp -- please check this PR based on a deviation being added to skip a test.

robshakir avatar Feb 16 '25 07:02 robshakir

Pull Request Test Coverage Report for Build 14924487408

Details

  • 1 of 10 (10.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 18.04%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/deviations/deviations.go 0 3 0.0%
proto/metadata_go_proto/metadata.pb.go 1 7 14.29%
<!-- Total: 1 10
Totals Coverage Status
Change from base Build 14921966186: -0.02%
Covered Lines: 2478
Relevant Lines: 13736

💛 - Coveralls

coveralls avatar May 09 '25 08:05 coveralls

This PR included only a deviation to skip TC-14.2.4. As the deviation is no longer required, we are closing the PR

ampattan avatar Jun 24 '25 12:06 ampattan