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

High tps test #13573

Closed
wants to merge 74 commits into from
Closed

High tps test #13573

wants to merge 74 commits into from

Conversation

sitalkedia
Copy link
Contributor

Description

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Performance improvement
  • Refactoring
  • Dependency update
  • Documentation update
  • Tests

Which Components or Systems Does This Change Impact?

  • Validator Node
  • Full Node (API, Indexer, etc.)
  • Move/Aptos Virtual Machine
  • Aptos Framework
  • Aptos CLI/SDK
  • Developer Infrastructure
  • Other (specify)

How Has This Been Tested?

Key Areas to Review

Checklist

  • I have read and followed the CONTRIBUTING doc
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I identified and added all stakeholders and component owners affected by this change as reviewers
  • I tested both happy and unhappy path of the functionality
  • I have made corresponding changes to the documentation

Copy link

trunk-io bot commented Jun 5, 2024

⏱️ 79h 53m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-smoke-tests 14h 1m 🟥🟥🟥🟩 (+23 more)
execution-performance / single-node-performance 11h 23m 🟩🟩🟥🟩🟩 (+23 more)
rust-targeted-unit-tests 9h 47m 🟥🟥🟥🟥🟩 (+23 more)
forge-e2e-test / forge 8h 9m 🟥🟥🟥🟥 (+23 more)
rust-images / rust-all 6h 31m 🟩🟩🟩🟩 (+24 more)
forge-compat-test / forge 6h 11m 🟥🟥🟥🟩 (+22 more)
rust-move-tests 5h 42m 🟩🟩🟩🟩 (+24 more)
rust-lints 2h 55m 🟩🟩🟥🟥 (+23 more)
cli-e2e-tests / run-cli-tests 2h 45m 🟩🟩🟩🟩🟩 (+21 more)
run-tests-main-branch 2h 10m 🟩🟩🟩🟩🟩 (+24 more)
rust-build-cached-packages 2h 8m 🟩🟩🟥🟥🟩 (+22 more)
check 1h 40m 🟩🟩🟩🟥 (+22 more)
test-target-determinator 1h 33m 🟩🟩🟩🟩 (+23 more)
execution-performance / test-target-determinator 1h 33m 🟩🟩🟩🟩🟩 (+22 more)
general-lints 50m 🟩🟩🟩🟩🟩 (+22 more)
indexer-grpc-e2e-tests / test-indexer-grpc-docker-compose 46m 🟥🟩🟩🟩🟩 (+22 more)
check-dynamic-deps 43m 🟩🟩🟩🟩🟩 (+24 more)
node-api-compatibility-tests / node-api-compatibility-tests 29m 🟩🟩🟩🟩🟩 (+21 more)
semgrep/ci 14m 🟩🟩🟩🟩🟩 (+24 more)
file_change_determinator 5m 🟩🟩🟩🟩🟩 (+24 more)
file_change_determinator 5m 🟩🟩🟩🟩🟩 (+24 more)
file_change_determinator 5m 🟩🟩🟩🟩🟩 (+24 more)
permission-check 2m 🟩🟩🟩🟩🟩 (+25 more)
permission-check 1m 🟩🟩🟩🟩🟩 (+24 more)
permission-check 1m 🟩🟩🟩🟩🟩 (+24 more)
permission-check 1m 🟩🟩🟩🟩🟩 (+24 more)
permission-check 1m 🟩🟩🟩🟩🟩 (+24 more)
determine-docker-build-metadata 1m 🟩🟩🟩🟩🟩 (+24 more)

settingsfeedbackdocs ⋅ learn more about trunk.io

@sitalkedia sitalkedia added the CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR label Jun 5, 2024

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on d5cfd8d68b5e1644be81af024cc65a20f4742cad

two traffics test: inner traffic : committed: 7593.007098921517 txn/s, latency: 2386.4312679965838 ms, (p50: 2000 ms, p90: 3700 ms, p99: 5700 ms), latency samples: 3278400
two traffics test : committed: 99.96374808578653 txn/s, latency: 1160.8193548387096 ms, (p50: 1100 ms, p90: 1500 ms, p99: 4300 ms), latency samples: 1860
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.240, avg: 0.224", "QsPosToProposal: max: 0.172, avg: 0.139", "ConsensusProposalToOrdered: max: 0.323, avg: 0.295", "ConsensusOrderedToCommit: max: 0.402, avg: 0.365", "ConsensusProposalToCommit: max: 0.700, avg: 0.660"]
Max round gap was 0 [limit 4] at version 0. Max no progress secs was 0.87049997 [limit 15] at version 1689627.
Test Ok

Copy link
Contributor

✅ Forge suite compat success on f648076a280621dbfd4e73b1ca83e3a3f52878ed ==> d5cfd8d68b5e1644be81af024cc65a20f4742cad

Compatibility test results for f648076a280621dbfd4e73b1ca83e3a3f52878ed ==> d5cfd8d68b5e1644be81af024cc65a20f4742cad (PR)
1. Check liveness of validators at old version: f648076a280621dbfd4e73b1ca83e3a3f52878ed
compatibility::simple-validator-upgrade::liveness-check : committed: 249.69115211635165 txn/s, submitted: 309.6350371824861 txn/s, expired: 59.94388506613444 txn/s, latency: 3985.172621832906 ms, (p50: 2800 ms, p90: 4500 ms, p99: 14900 ms), latency samples: 160281
2. Upgrading first Validator to new version: d5cfd8d68b5e1644be81af024cc65a20f4742cad
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 921.107710212362 txn/s, submitted: 1286.8658519641963 txn/s, failed submission: 110.8701884805132 txn/s, expired: 365.75814175183433 txn/s, latency: 8910.647488573224 ms, (p50: 9900 ms, p90: 11500 ms, p99: 12000 ms), latency samples: 81169
3. Upgrading rest of first batch to new version: d5cfd8d68b5e1644be81af024cc65a20f4742cad
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 897.8176106194843 txn/s, submitted: 1196.8111163405943 txn/s, expired: 298.99350572111 txn/s, latency: 9307.864820066561 ms, (p50: 10700 ms, p90: 11500 ms, p99: 13800 ms), latency samples: 77223
4. upgrading second batch to new version: d5cfd8d68b5e1644be81af024cc65a20f4742cad
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 1385.8285492541993 txn/s, submitted: 1524.5612025862124 txn/s, expired: 138.73265333201317 txn/s, latency: 13545.151033857526 ms, (p50: 5500 ms, p90: 48000 ms, p99: 54600 ms), latency samples: 114716
5. check swarm health
Compatibility test for f648076a280621dbfd4e73b1ca83e3a3f52878ed ==> d5cfd8d68b5e1644be81af024cc65a20f4742cad passed
Test Ok

Copy link
Contributor

This issue is stale because it has been open 45 days with no activity. Remove the stale label, comment or push a commit - otherwise this will be closed in 15 days.

@github-actions github-actions bot added the Stale label Aug 14, 2024
@github-actions github-actions bot closed this Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants