RT 7.4 & RT 7.11: Updated the script with deviation for ASpath set
RT 7.4 & RT 7.11: Updated the script with deviation for ASpath set
Added policy result as ACCEPT for policy "match_community_regex"
Pull Request Functional Test Report for #3365 / ee6f9df9690eb902af57fa1251468042562b8acc
Virtual Devices
| Device | Test | Test Documentation | Job | Raw Log |
|---|---|---|---|---|
| Arista cEOS | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
|||
| Cisco 8000E | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
|||
| Cisco XRd | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
|||
| Juniper ncPTX | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
|||
| Nokia SR Linux | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
|||
| Openconfig Lemming | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
Hardware Devices
| Device | Test | Test Documentation | Raw Log |
|---|---|---|---|
| Arista 7808 | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
||
| Cisco 8808 | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
||
| Juniper PTX10008 | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
||
| Nokia 7250 IXR-10e | RT-7.11: BGP Policy - Import/Export Policy Action Using Multiple Criteria |
Pull Request Test Coverage Report for Build 13814392929
Details
- 7 of 24 (29.17%) changed or added relevant lines in 2 files are covered.
- 3 unchanged lines in 1 file lost coverage.
- Overall coverage increased (+0.04%) to 18.213%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
|---|---|---|---|
| internal/deviations/deviations.go | 0 | 3 | 0.0% |
| proto/metadata_go_proto/metadata.pb.go | 7 | 21 | 33.33% |
| <!-- | Total: | 7 | 24 |
| Files with Coverage Reduction | New Missed Lines | % |
|---|---|---|
| proto/metadata_go_proto/metadata.pb.go | 3 | 5.48% |
| <!-- | Total: | 3 |
| Totals | |
|---|---|
| Change from base Build 13814371155: | 0.04% |
| Covered Lines: | 2477 |
| Relevant Lines: | 13600 |
💛 - Coveralls
Fixes #3415
@s-nandu please let us know if you will be resolving the comments.
@s-nandu there are still conflicts to be resolved
@s-nandu - Can you please resolve the conflicts?
@juniper - Please resolve the conflicts in this PR so i can validate it. The conflicts are in the testcode itself.
@ram-mac , resolved all the conflicts. Please merge the code after review @dplore , all comments have been resolved but the pull request still says there are unresolved comments. Please help rectify
@s-nandu - i validated this change and looks like it is breaking one vendor during gnmi.Replace(ln502) is breaking with the following error "An unexpected element "match-as-path-set" is present".
Please resolve the conflicts; So it can go for final approval and merge it soon.