TE3.7 - Added remove config end of testImplementDrain and addconfig at start of testBaseHierarchialNHGwithVrfPolW test.
Added remove config end of testImplementDrain and addconfig at start of testBaseHierarchialNHGwithVrfPolW for failure seen in the latter test.
Failure was due to configuration of testImplementDrain (configuration is different compared to 1st and 3rd test) still being present during testBaseHierarchialNHGwithVrfPolW (3rd) test run.
Pull Request Functional Test Report for #4082 / 5f7261026e3fc00fc127f0f713ed538507a8fc9c
Virtual Devices
| Device | Test | Test Documentation | Job | Raw Log |
|---|---|---|---|---|
| Arista cEOS | TE-3.7: Base Hierarchical NHG Update |
|||
| Cisco 8000E | TE-3.7: Base Hierarchical NHG Update |
|||
| Cisco XRd | TE-3.7: Base Hierarchical NHG Update |
|||
| Juniper ncPTX | TE-3.7: Base Hierarchical NHG Update |
|||
| Nokia SR Linux | TE-3.7: Base Hierarchical NHG Update |
|||
| Openconfig Lemming | TE-3.7: Base Hierarchical NHG Update |
Hardware Devices
| Device | Test | Test Documentation | Raw Log |
|---|---|---|---|
| Arista 7808 | TE-3.7: Base Hierarchical NHG Update |
||
| Cisco 8808 | TE-3.7: Base Hierarchical NHG Update |
||
| Juniper PTX10008 | TE-3.7: Base Hierarchical NHG Update |
||
| Nokia 7250 IXR-10e | TE-3.7: Base Hierarchical NHG Update |
Pull Request Test Coverage Report for Build 16451360477
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
- For more information on this, see Tracking coverage changes with pull request builds.
- To avoid this issue with future PRs, see these Recommended CI Configurations.
- For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
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 13.461%
| Totals | |
|---|---|
| Change from base Build 16442435584: | 0.0% |
| Covered Lines: | 2065 |
| Relevant Lines: | 15341 |
💛 - Coveralls
/fptest all
The logic and code looks good and I have validated it. @Swetha-haridasula Please can you approve?