RT-6.1: LLDP testcase changes for cisco
PR to incorporate LLDP behavior change from ios-xr version 25.1.1: interface needs to be added to OC datastore explicitly before enabling LLDP on it
Proposed changes: leafref fix + interface operation remove the deviation "lldp_interface_config_override_global" which is not applicable anymore
Note: go test flag "lldpInterfaceConfigOverrideGlobal" has been added for backward compatibility - if consuming IOS-XR 2441 images for test, set this flag to true
Pull Request Functional Test Report for #4035 / 4a2e1077d48a455178843619a26671782f4c98a6
Virtual Devices
| Device | Test | Test Documentation | Job | Raw Log |
|---|---|---|---|---|
| Arista cEOS | RT-6.1: Core LLDP TLV Population |
|||
| Cisco 8000E | RT-6.1: Core LLDP TLV Population |
|||
| Cisco XRd | RT-6.1: Core LLDP TLV Population |
|||
| Juniper ncPTX | RT-6.1: Core LLDP TLV Population |
|||
| Nokia SR Linux | RT-6.1: Core LLDP TLV Population |
|||
| Openconfig Lemming | RT-6.1: Core LLDP TLV Population |
Hardware Devices
| Device | Test | Test Documentation | Raw Log |
|---|---|---|---|
| Arista 7808 | RT-6.1: Core LLDP TLV Population |
||
| Cisco 8808 | RT-6.1: Core LLDP TLV Population |
||
| Juniper PTX10008 | RT-6.1: Core LLDP TLV Population |
||
| Nokia 7250 IXR-10e | RT-6.1: Core LLDP TLV Population |
Pull Request Test Coverage Report for Build 13975421627
Details
- 0 of 5 (0.0%) changed or added relevant lines in 1 file are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage decreased (-0.003%) to 18.121%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
|---|---|---|---|
| internal/deviations/byexceptions.go | 0 | 5 | 0.0% |
| <!-- | Total: | 0 | 5 |
| Totals | |
|---|---|
| Change from base Build 13959151785: | -0.003% |
| Covered Lines: | 2479 |
| Relevant Lines: | 13680 |
💛 - Coveralls
@karthikeya-remilla Please resolve conflicts
@rohit-rp is the ATE version of LLDP not used anymore? I think it is deleted?
If so, we can close this PR, as it is not applicable anymore as the changes are applicable only to ATE
Yes right it was deleted by me in /pull/4094