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

Update devent vkey and test proof for keyless #12413

Merged
merged 7 commits into from
Mar 11, 2024

Conversation

heliuchuan
Copy link
Contributor

Description

Test Plan

Copy link

trunk-io bot commented Mar 6, 2024

⏱️ 10h 2m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-unit-tests 3h 15m 🟩🟩🟩 (+2 more)
windows-build 2h 27m 🟩🟩🟩🟩🟩 (+3 more)
rust-lints 51m 🟩🟩🟩🟩🟩 (+2 more)
run-tests-main-branch 34m 🟩🟩🟩🟩🟩 (+3 more)
rust-smoke-tests 31m 🟩
check 29m 🟩🟩🟩🟩 (+3 more)
execution-performance / single-node-performance 22m 🟩
rust-images / rust-all 17m 🟩
check-dynamic-deps 16m 🟩🟩🟩🟩🟩 (+3 more)
general-lints 15m 🟩🟩🟩🟩🟩 (+2 more)
forge-compat-test / forge 15m 🟩
forge-e2e-test / forge 14m 🟩
cli-e2e-tests / run-cli-tests 7m 🟩
semgrep/ci 4m 🟩🟩🟩🟩🟩 (+3 more)
file_change_determinator 1m 🟩🟩🟩🟩🟩 (+3 more)
file_change_determinator 1m 🟩🟩🟩🟩🟩 (+3 more)
node-api-compatibility-tests / node-api-compatibility-tests 52s 🟩
permission-check 24s 🟩🟩🟩🟩🟩 (+3 more)
permission-check 23s 🟩🟩🟩🟩🟩 (+3 more)
permission-check 22s 🟩🟩🟩🟩🟩 (+3 more)
permission-check 19s 🟩🟩🟩🟩🟩 (+3 more)
file_change_determinator 9s 🟩
execution-performance / file_change_determinator 9s 🟩
permission-check 7s 🟩
determine-docker-build-metadata 1s 🟩

🚨 2 jobs on the last run were significantly faster/slower than expected

Job Duration vs 7d avg Delta
check-dynamic-deps 3m 2m +76%
rust-images / rust-all 17m 14m +23%

settingsfeedbackdocs ⋅ learn more about trunk.io

@heliuchuan heliuchuan changed the title add script Update devent vkey and test proof for keyless Mar 8, 2024
@heliuchuan heliuchuan marked this pull request as ready for review March 11, 2024 18:31
@heliuchuan heliuchuan enabled auto-merge (squash) March 11, 2024 19:01

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 20c3c7fca426711a53710d4f8a1dca50e7b866d2

two traffics test: inner traffic : committed: 7630 txn/s, latency: 5139 ms, (p50: 4800 ms, p90: 6000 ms, p99: 12300 ms), latency samples: 3296340
two traffics test : committed: 100 txn/s, latency: 1886 ms, (p50: 1800 ms, p90: 2100 ms, p99: 3600 ms), latency samples: 1840
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.236, avg: 0.203", "QsPosToProposal: max: 0.360, avg: 0.288", "ConsensusProposalToOrdered: max: 0.479, avg: 0.455", "ConsensusOrderedToCommit: max: 0.296, avg: 0.268", "ConsensusProposalToCommit: max: 0.763, avg: 0.723"]
Max round gap was 1 [limit 4] at version 1360642. Max no progress secs was 5.65746 [limit 15] at version 1360642.
Test Ok

Copy link
Contributor

✅ Forge suite compat success on aptos-node-v1.9.5 ==> 20c3c7fca426711a53710d4f8a1dca50e7b866d2

Compatibility test results for aptos-node-v1.9.5 ==> 20c3c7fca426711a53710d4f8a1dca50e7b866d2 (PR)
1. Check liveness of validators at old version: aptos-node-v1.9.5
compatibility::simple-validator-upgrade::liveness-check : committed: 5939 txn/s, latency: 4920 ms, (p50: 4900 ms, p90: 7800 ms, p99: 8400 ms), latency samples: 237560
2. Upgrading first Validator to new version: 20c3c7fca426711a53710d4f8a1dca50e7b866d2
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 717 txn/s, latency: 34660 ms, (p50: 36600 ms, p90: 55800 ms, p99: 57100 ms), latency samples: 56700
3. Upgrading rest of first batch to new version: 20c3c7fca426711a53710d4f8a1dca50e7b866d2
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 402 txn/s, submitted: 592 txn/s, expired: 189 txn/s, latency: 38678 ms, (p50: 43700 ms, p90: 58700 ms, p99: 59400 ms), latency samples: 36234
4. upgrading second batch to new version: 20c3c7fca426711a53710d4f8a1dca50e7b866d2
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 1684 txn/s, latency: 17374 ms, (p50: 16600 ms, p90: 25000 ms, p99: 25400 ms), latency samples: 72420
5. check swarm health
Compatibility test for aptos-node-v1.9.5 ==> 20c3c7fca426711a53710d4f8a1dca50e7b866d2 passed
Test Ok

@heliuchuan heliuchuan merged commit cd811bf into aptos-labs:main Mar 11, 2024
79 of 80 checks passed
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.

3 participants