Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add experimental replay test to reproduce "couldn't diff the command trees" error #2329

Merged
merged 7 commits into from
Nov 1, 2023
Merged

Add experimental replay test to reproduce "couldn't diff the command trees" error #2329

merged 7 commits into from
Nov 1, 2023

Conversation

nhawke
Copy link
Contributor

@nhawke nhawke commented Oct 31, 2023

This also updates the replayer library to the newest version in go.mod to ensure that the most up-to-date transformations are necessary for this log.

@nhawke nhawke requested review from a team as code owners October 31, 2023 19:55
@OpenConfigBot
Copy link

OpenConfigBot commented Oct 31, 2023 •
edited
Loading

Pull Request Functional Test Report for #2329 / 2160b19

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
 Replay-1.1: Record/replay diff command trees test
 ab5eb02a Log
Cisco 8000E status
 Replay-1.1: Record/replay diff command trees test
 9caa4c0e Log
Cisco XRd status
 Replay-1.1: Record/replay diff command trees test
 3411b087 Log
Juniper cPTX status
 Replay-1.1: Record/replay diff command trees test
 a4f8ab9c Log
Juniper ncPTX status
 Replay-1.1: Record/replay diff command trees test
 6f118a1a Log
Nokia SR Linux status
 Replay-1.1: Record/replay diff command trees test
 34b84d54 Log
Openconfig Lemming status
 Replay-1.1: Record/replay diff command trees test
 20e3d287 Log

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
 Replay-1.1: Record/replay diff command trees test
Cisco 8808 status
 Replay-1.1: Record/replay diff command trees test
Juniper PTX10008 status
 Replay-1.1: Record/replay diff command trees test
Nokia 7250 IXR-10e status
 Replay-1.1: Record/replay diff command trees test

Help

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 31, 2023 •
edited
Loading

Pull Request Test Coverage Report for Build 6720739165

  • 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 48.7%
Totals Coverage Status
Change from base Build 6711171115: 0.0%
Covered Lines: 1536
Relevant Lines: 3154
💛 - Coveralls

greg-dennis
greg-dennis suggested changes Nov 1, 2023
View reviewed changes
.gitattributes Outdated Show resolved Hide resolved
@nhawke nhawke merged commit f9c0878 into main Nov 1, 2023
12 checks passed
frasieroh pushed a commit to aristanetworks/openconfig-featureprofiles that referenced this pull request Nov 1, 2023
@nhawke @frasieroh
Add experimental replay test to reproduce "couldn't diff the command …
e8758f0 
…trees" error (openconfig#2329)

* Add test to reproduce 'couldn't diff the command trees' error

* Update relpayer to latest version
frasieroh pushed a commit to aristanetworks/openconfig-featureprofiles that referenced this pull request Nov 3, 2023
@nhawke @frasieroh
Add experimental replay test to reproduce "couldn't diff the command …
7400f8a 
…trees" error (openconfig#2329)

* Add test to reproduce 'couldn't diff the command trees' error

* Update relpayer to latest version
@dplore dplore deleted the nhawke-diff-command-tree branch September 10, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@greg-dennis greg-dennis greg-dennis approved these changes

Assignees

@greg-dennis greg-dennis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nhawke @OpenConfigBot @greg-dennis