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.

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

06651 Added missing rehash for VirtualMap leaves #6653

Merged
merged 1 commit into from May 18, 2023

Conversation

imalygin
Copy link
Member

Description:

This PR adds fullLeafRehash call to VirtualMap instances created by VirtualMerkleStateInitializer in PlatformTestingTool

Related issue(s):

Fixes #6651

Notes for reviewer:
This test should fix https://github.com/swirlds/swirlds-platform-regression/issues/3411

@imalygin imalygin requested a review from a team as a code owner May 17, 2023 20:04
@imalygin imalygin requested a review from poulok May 17, 2023 20:04
@imalygin imalygin force-pushed the 06651-test-failure-due-to-missing-hashes branch 2 times, most recently from 49d3a92 to a01c317 Compare May 18, 2023 01:21
@imalygin imalygin self-assigned this May 18, 2023
@imalygin imalygin added this to the v0.39 milestone May 18, 2023
@imalygin imalygin force-pushed the 06651-test-failure-due-to-missing-hashes branch from a01c317 to c786bf4 Compare May 18, 2023 15:27
cody-littley
cody-littley previously approved these changes May 18, 2023
Signed-off-by: Ivan Malygin <ivan@swirldslabs.com>
@github-actions
Copy link

Node: Unit Test Results

    1 383 files      1 383 suites   1h 13m 2s ⏱️
  98 456 tests   98 449 ✔️ 7 💤 0
103 621 runs  103 614 ✔️ 7 💤 0

Results for commit 1afb9ab.

@sonarcloud
Copy link

sonarcloud bot commented May 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@github-actions
Copy link

Node: Integration Test Results

    4 files      4 suites   13m 45s ⏱️
170 tests 170 ✔️ 0 💤 0
172 runs  172 ✔️ 0 💤 0

Results for commit 1afb9ab.

@github-actions
Copy link

Node: E2E Test Results

    1 files      1 suites   18m 0s ⏱️
310 tests 310 ✔️ 0 💤 0
328 runs  328 ✔️ 0 💤 0

Results for commit 1afb9ab.

@github-actions
Copy link

github-actions bot commented May 18, 2023

Platform: JUnit Test Report

     522 files           1 errors  521 suites   21m 56s ⏱️
13 819 tests 13 785 ✔️ 34 💤 0
15 804 runs  15 770 ✔️ 34 💤 0

Results for commit 1afb9ab.

♻️ This comment has been updated with latest results.

@sonarcloud
Copy link

sonarcloud bot commented May 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

80.0% 80.0% Coverage
0.0% 0.0% Duplication

@imalygin imalygin merged commit dceadbf into develop May 18, 2023
22 of 23 checks passed
@imalygin imalygin deleted the 06651-test-failure-due-to-missing-hashes branch May 18, 2023 18:39
nickpoorman pushed a commit that referenced this pull request May 18, 2023
Signed-off-by: Ivan Malygin <ivan@swirldslabs.com>
iwsimon pushed a commit that referenced this pull request May 19, 2023
Signed-off-by: Ivan Malygin <ivan@swirldslabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VMerkleSC-Saved-State-5k-20m is failing due to missing hashes
3 participants