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

Sync Terraform & Helm #14082

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Sync Terraform & Helm #14082

merged 1 commit into from
Jul 22, 2024

Conversation

aptos-bot
Copy link
Contributor

Sync Terraform & Helm changes

GitOrigin-RevId: ca4c13a1e99bde24c9f6a0f9f10c1a21bb24e4fd
Copy link

trunk-io bot commented Jul 22, 2024

⏱️ 1h 35m total CI duration on this PR
Job Cumulative Duration Recent Runs
forge-e2e-test / forge 16m 🟩
forge-compat-test / forge 15m 🟩
test-target-determinator 9m 🟩🟩
execution-performance / test-target-determinator 9m 🟩🟩
check 7m 🟩🟩
rust-doc-tests 6m 🟩
rust-doc-tests 6m 🟩
rust-cargo-deny 5m 🟩🟩🟩
general-lints 5m 🟩🟩🟩
rust-move-tests 3m 🟩
rust-move-tests 3m 🟩
rust-move-tests 3m
check-dynamic-deps 2m 🟩🟩🟩
indexer-grpc-e2e-tests / test-indexer-grpc-docker-compose 2m 🟩
semgrep/ci 1m 🟩🟩🟩
file_change_determinator 39s 🟩🟩🟩
file_change_determinator 35s 🟩🟩🟩
file_change_determinator 30s 🟩🟩
execution-performance / single-node-performance 19s 🟩🟩
forge-framework-upgrade-test / forge 10s 🟩
permission-check 9s 🟩🟩🟩
permission-check 9s 🟩🟩🟩
permission-check 8s 🟩🟩🟩
permission-check 7s 🟩🟩
Backport PR 7s 🟥🟥
permission-check 7s 🟩🟩
permission-check 7s 🟩🟩🟩
determine-docker-build-metadata 6s 🟩🟩
check-terraform-modifications 2s 🟥

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

Job Duration vs 7d avg Delta
forge-framework-upgrade-test / forge 10s 12m -99%
execution-performance / single-node-performance 9s 17m -99%

settingsfeedbackdocs ⋅ learn more about trunk.io

@sionescu sionescu marked this pull request as ready for review July 22, 2024 19:14
@sionescu sionescu requested a review from a team as a code owner July 22, 2024 19:14
@sionescu sionescu enabled auto-merge (rebase) July 22, 2024 19:16
@sionescu sionescu added the CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR label Jul 22, 2024

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite compat success on 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5 ==> 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca

Compatibility test results for 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5 ==> 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca (PR)
1. Check liveness of validators at old version: 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5
compatibility::simple-validator-upgrade::liveness-check : committed: 7625.370435666094 txn/s, latency: 4541.980203499934 ms, (p50: 3300 ms, p90: 7200 ms, p99: 28900 ms), latency samples: 301720
2. Upgrading first Validator to new version: 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca
compatibility::simple-validator-upgrade::single-validator-upgrading : committed: 6788.113209540302 txn/s, latency: 4119.993494623656 ms, (p50: 4200 ms, p90: 4700 ms, p99: 4800 ms), latency samples: 130200
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 6466.84006062614 txn/s, latency: 4791.198273804789 ms, (p50: 4500 ms, p90: 5400 ms, p99: 8700 ms), latency samples: 251420
3. Upgrading rest of first batch to new version: 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca
compatibility::simple-validator-upgrade::half-validator-upgrading : committed: 7490.339519477782 txn/s, latency: 3610.9661429582447 ms, (p50: 4000 ms, p90: 4300 ms, p99: 4500 ms), latency samples: 141300
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 6762.7719336519895 txn/s, latency: 4471.428217738546 ms, (p50: 4200 ms, p90: 7500 ms, p99: 8100 ms), latency samples: 237900
4. upgrading second batch to new version: 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca
compatibility::simple-validator-upgrade::rest-validator-upgrading : committed: 10514.727358859238 txn/s, latency: 2504.559338107219 ms, (p50: 2500 ms, p90: 3300 ms, p99: 6300 ms), latency samples: 206680
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 9311.714355516355 txn/s, latency: 3403.6400939941404 ms, (p50: 2600 ms, p90: 8400 ms, p99: 9500 ms), latency samples: 327680
5. check swarm health
Compatibility test for 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5 ==> 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 09016a15d68aa4b2ff29d9aa925b958f6a4b95ca

two traffics test: inner traffic : committed: 9321.864630317885 txn/s, submitted: 9572.61132195949 txn/s, expired: 250.74669164160568 txn/s, latency: 2734.5278789075724 ms, (p50: 2700 ms, p90: 3300 ms, p99: 4200 ms), latency samples: 3544400
two traffics test : committed: 100.01074342040641 txn/s, latency: 1990.0715 ms, (p50: 2000 ms, p90: 2200 ms, p99: 2400 ms), latency samples: 2000
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.235, avg: 0.216", "QsPosToProposal: max: 1.291, avg: 0.564", "ConsensusProposalToOrdered: max: 0.340, avg: 0.294", "ConsensusOrderedToCommit: max: 0.389, avg: 0.377", "ConsensusProposalToCommit: max: 0.687, avg: 0.671"]
Max round gap was 1 [limit 4] at version 1235477. Max no progress secs was 7.107122 [limit 15] at version 4042404.
Test Ok

@sionescu sionescu merged commit 6e394fd into main Jul 22, 2024
124 of 140 checks passed
@sionescu sionescu deleted the sync_terraform_helm branch July 22, 2024 19:55
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants