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

feat: turbo sync #10785

Draft
wants to merge 44 commits into
base: main
Choose a base branch
from
Draft

feat: turbo sync #10785

wants to merge 44 commits into from

Conversation

cody-littley
Copy link
Contributor

closes #10691

Cody Littley added 23 commits December 30, 2023 14:04
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Copy link

github-actions bot commented Jan 4, 2024

Node: E2E Test Results

    1 files  ±    0      1 suites  ±0   22m 44s ⏱️ + 22m 44s
311 tests +310  311 ✔️ +311  0 💤 ±0  0  - 1 
333 runs  +332  333 ✔️ +333  0 💤 ±0  0  - 1 

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

This pull request removes 1 and adds 311 tests. Note that renamed tests count towards both.
EndToEndTests ‑ initializationError
EndToEndTests ‑ ADDRESS_BOOK_CONTROLCanUpdateADDRESS_BOOK
EndToEndTests ‑ ADDRESS_BOOK_CONTROLCanUpdateNODE_DETAILS
EndToEndTests ‑ AccountsGetPayerRecordsIfSoConfigured
EndToEndTests ‑ Acct57CanMakeSmallChanges
EndToEndTests ‑ Acct57CantMakeLargeChanges
EndToEndTests ‑ AddingSignaturesToExecutedTxFails
EndToEndTests ‑ AddingSignaturesToExecutedTxFailsWithLongTermEnabled
EndToEndTests ‑ AddingSignaturesToNonExistingTxFails
EndToEndTests ‑ AddingSignaturesToNonExistingTxFailsWithLongTermEnabled
EndToEndTests ‑ AddressAliasIdFuzzing
…

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 4, 2024

Node: HAPI Test (Crypto) Results

210 tests  ±0   210 ✔️ ±0   33m 59s ⏱️ + 13m 15s
  22 suites ±0       0 💤 ±0 
  22 files   ±0       0 ±0 

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 4, 2024

Node: HAPI Test (Token) Results

189 tests  ±0   188 ✔️ +1   31m 33s ⏱️ + 12m 4s
  13 suites ±0       0 💤 ±0 
  13 files   ±0       1  - 1 

For more details on these failures, see this check.

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 4, 2024

Node: Unit Test Results

    2 293 files  +1      2 293 suites  +1   51m 32s ⏱️ + 3m 12s
118 795 tests +3  118 750 ✔️  - 3  44 💤 +5  1 +1 
127 318 runs  +5  127 273 ✔️  - 1  44 💤 +5  1 +1 

For more details on these failures, see this check.

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Jan 4, 2024

Codecov Report

Attention: 312 lines in your changes are missing coverage. Please review.

Comparison is base (3df5a2a) 63.27% compared to head (780079c) 63.12%.
Report is 5 commits behind head on develop.

Files Patch % Lines
...ds/platform/gossip/sync/turbo/TurboSyncRunner.java 0.00% 202 Missing ⚠️
.../platform/gossip/sync/turbo/TurboSyncProtocol.java 0.00% 48 Missing ⚠️
...swirlds/platform/gossip/shadowgraph/SyncUtils.java 0.00% 18 Missing ⚠️
...java/com/swirlds/platform/metrics/SyncMetrics.java 9.09% 10 Missing ⚠️
...a/com/swirlds/platform/gossip/sync/SyncGossip.java 0.00% 9 Missing ⚠️
...irlds/platform/gossip/shadowgraph/ShadowGraph.java 0.00% 8 Missing ⚠️
...m/gossip/shadowgraph/LatestEventTipsetTracker.java 0.00% 4 Missing ⚠️
...irlds/platform/gossip/FallenBehindManagerImpl.java 75.00% 2 Missing and 1 partial ⚠️
.../platform/gossip/sync/turbo/TurboSyncDataSent.java 0.00% 3 Missing ⚠️
...n/java/com/swirlds/platform/event/GossipEvent.java 0.00% 2 Missing ⚠️
... and 5 more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop   #10785      +/-   ##
=============================================
- Coverage      63.27%   63.12%   -0.16%     
+ Complexity     31313    31310       -3     
=============================================
  Files           3390     3394       +4     
  Lines         137465   137782     +317     
  Branches       14404    14445      +41     
=============================================
- Hits           86985    86977       -8     
- Misses         46894    47217     +323     
- Partials        3586     3588       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Cody Littley <cody@swirldslabs.com>
Copy link

github-actions bot commented Jan 5, 2024

Node: HAPI Test (Time Consuming) Results

20 tests  ±0   20 ✔️ ±0   54m 12s ⏱️ +26s
  2 suites ±0     0 💤 ±0 
  2 files   ±0     0 ±0 

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 5, 2024

Node: HAPI Test (Misc) Results

423 tests  ±0   406 ✔️  - 3   53m 47s ⏱️ + 19m 29s
  73 suites ±0     14 💤 ±0 
  73 files   ±0       3 +3 

For more details on these failures, see this check.

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 5, 2024

Node: Integration Test Results

    5 files  ±0      5 suites  ±0   1h 6m 42s ⏱️ +41s
318 tests ±0  318 ✔️ ±0  0 💤 ±0  0 ±0 
330 runs  ±0  330 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 5, 2024

Node: HAPI Test (Smart Contract) Results

445 tests  ±0   444 ✔️ ±0   1h 36m 1s ⏱️ + 31m 22s
  52 suites ±0       0 💤 ±0 
  52 files   ±0       1 ±0 

For more details on these failures, see this check.

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Cody Littley added 3 commits January 12, 2024 14:52
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Copy link

github-actions bot commented Jan 12, 2024

Node: HAPI Test (Restart) Results

1 tests  ±0   1 ✔️ ±0   4m 56s ⏱️ -39s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 12, 2024

Node: HAPI Test (Node Death Reconnect) Results

1 tests  ±0   1 ✔️ ±0   5m 59s ⏱️ +34s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit 780079c. ± Comparison against base commit 3df5a2a.

♻️ This comment has been updated with latest results.

Cody Littley added 17 commits January 12, 2024 16:41
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@swirldslabs.com>
Signed-off-by: Cody Littley <cody@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.

Turbo Sync
1 participant