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

forge: enable optqs #15794

Merged
merged 1 commit into from
Jan 22, 2025
Merged

forge: enable optqs #15794

merged 1 commit into from
Jan 22, 2025

Conversation

ibalajiarun
Copy link
Contributor

Description

Re-enabling optqs in Forge.

Copy link

trunk-io bot commented Jan 22, 2025

⏱️ 31m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-doc-tests 6m 🟩
execution-performance / test-target-determinator 5m 🟩
test-target-determinator 4m 🟩
general-lints 4m 🟩🟩
check-dynamic-deps 4m 🟩🟩
rust-cargo-deny 3m 🟩🟩
fetch-last-released-docker-image-tag 2m 🟩
semgrep/ci 44s 🟩🟩
file_change_determinator 22s 🟩🟩
file_change_determinator 15s 🟩
execution-performance / single-node-performance 11s 🟩
permission-check 6s 🟩🟩
permission-check 5s 🟩🟩
permission-check 2s 🟩
determine-docker-build-metadata 2s 🟩

🚨 1 job on the last run was significantly faster/slower than expected

Job Duration vs 7d avg Delta
general-lints 4m 27s +781%

settingsfeedbackdocs ⋅ learn more about trunk.io

@ibalajiarun ibalajiarun enabled auto-merge (squash) January 22, 2025 21:22

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite compat success on bb609ba7417025d8af1e199c345631c8b4b2c412 ==> 6916969f13c138d3a44e8589d1eb90db705c16b6

Compatibility test results for bb609ba7417025d8af1e199c345631c8b4b2c412 ==> 6916969f13c138d3a44e8589d1eb90db705c16b6 (PR)
1. Check liveness of validators at old version: bb609ba7417025d8af1e199c345631c8b4b2c412
compatibility::simple-validator-upgrade::liveness-check : committed: 14577.20 txn/s, latency: 2103.72 ms, (p50: 2200 ms, p70: 2300, p90: 2500 ms, p99: 2800 ms), latency samples: 482340
2. Upgrading first Validator to new version: 6916969f13c138d3a44e8589d1eb90db705c16b6
compatibility::simple-validator-upgrade::single-validator-upgrading : committed: 3556.83 txn/s, latency: 8443.11 ms, (p50: 9100 ms, p70: 10100, p90: 10800 ms, p99: 11000 ms), latency samples: 79420
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 3523.00 txn/s, latency: 9779.24 ms, (p50: 10700 ms, p70: 11000, p90: 11000 ms, p99: 11200 ms), latency samples: 129200
3. Upgrading rest of first batch to new version: 6916969f13c138d3a44e8589d1eb90db705c16b6
compatibility::simple-validator-upgrade::half-validator-upgrading : committed: 4285.70 txn/s, latency: 7294.37 ms, (p50: 8100 ms, p70: 8700, p90: 9000 ms, p99: 9300 ms), latency samples: 92100
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 4393.49 txn/s, latency: 7734.08 ms, (p50: 8700 ms, p70: 8800, p90: 9000 ms, p99: 9100 ms), latency samples: 155360
4. upgrading second batch to new version: 6916969f13c138d3a44e8589d1eb90db705c16b6
compatibility::simple-validator-upgrade::rest-validator-upgrading : committed: 8137.79 txn/s, latency: 3751.42 ms, (p50: 4200 ms, p70: 4500, p90: 4700 ms, p99: 5500 ms), latency samples: 155120
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 7762.60 txn/s, latency: 4397.97 ms, (p50: 4800 ms, p70: 4800, p90: 4900 ms, p99: 5000 ms), latency samples: 260980
5. check swarm health
Compatibility test for bb609ba7417025d8af1e199c345631c8b4b2c412 ==> 6916969f13c138d3a44e8589d1eb90db705c16b6 passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 6916969f13c138d3a44e8589d1eb90db705c16b6

two traffics test: inner traffic : committed: 14767.23 txn/s, latency: 2681.60 ms, (p50: 2700 ms, p70: 2700, p90: 3000 ms, p99: 3300 ms), latency samples: 5614840
two traffics test : committed: 99.99 txn/s, latency: 1413.74 ms, (p50: 1400 ms, p70: 1400, p90: 1500 ms, p99: 1800 ms), latency samples: 1700
Latency breakdown for phase 0: ["MempoolToBlockCreation: max: 1.446, avg: 1.385", "ConsensusProposalToOrdered: max: 0.294, avg: 0.291", "ConsensusOrderedToCommit: max: 0.419, avg: 0.407", "ConsensusProposalToCommit: max: 0.710, avg: 0.697"]
Max non-epoch-change gap was: 0 rounds at version 0 (avg 0.00) [limit 4], 0.70s no progress at version 63282 (avg 0.19s) [limit 15].
Max epoch-change gap was: 0 rounds at version 0 (avg 0.00) [limit 4], 0.63s no progress at version 2699745 (avg 0.63s) [limit 16].
Test Ok

@ibalajiarun ibalajiarun merged commit 622e2ec into main Jan 22, 2025
90 checks passed
@ibalajiarun ibalajiarun deleted the balaji/forge-optqs branch January 22, 2025 21:56
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