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

Update Scripts to count tx state dumps diff #73

Merged
merged 7 commits into from
Oct 28, 2024
Merged

Update Scripts to count tx state dumps diff #73

merged 7 commits into from
Oct 28, 2024

Conversation

pefontana
Copy link
Member

Update Scripts to count tx state dumps diff: (scripts/cmp_state_dumps.sh & scripts/delta_state_dumps.sh)

  • Now we iterate over the state_dumps/native dumps
  • If a state dump is missing in state_dumps/vm, we skip the check

@pefontana pefontana linked an issue Oct 21, 2024 that may be closed by this pull request
Fix delta_state_dumps.sh compare order in Linux #70
Open
@pefontana pefontana marked this pull request as draft October 21, 2024 16:55
@pefontana pefontana marked this pull request as ready for review October 23, 2024 17:59
@pefontana pefontana removed a link to an issue Oct 23, 2024
Fix delta_state_dumps.sh compare order in Linux #70
Open
@pefontana pefontana linked an issue Oct 23, 2024 that may be closed by this pull request
Add missing counter to cmp_state_dumps.sh #71
Closed
@JulianGCalderon JulianGCalderon added this pull request to the merge queue Oct 28, 2024
Merged via the queue into main with commit 4b32b84 Oct 28, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@edg-l edg-l edg-l approved these changes

@JulianGCalderon JulianGCalderon JulianGCalderon approved these changes

@FrancoGiachetta FrancoGiachetta FrancoGiachetta approved these changes

@igaray igaray Awaiting requested review from igaray igaray is a code owner

@azteca1998 azteca1998 Awaiting requested review from azteca1998 azteca1998 is a code owner

@fmoletta fmoletta Awaiting requested review from fmoletta fmoletta is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add missing counter to cmp_state_dumps.sh
4 participants
@pefontana @edg-l @JulianGCalderon @FrancoGiachetta