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

[gha][docker] use capacity optimized spot runners #14022

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Conversation

ibalajiarun
Copy link
Contributor

@ibalajiarun ibalajiarun commented Jul 16, 2024

Description

Using capacity-optimized spot instances to limit preemptions.

@ibalajiarun ibalajiarun requested a review from a team as a code owner July 16, 2024 21:56
Copy link

trunk-io bot commented Jul 16, 2024

@ibalajiarun ibalajiarun enabled auto-merge (squash) July 16, 2024 21:59

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite compat success on 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5 ==> 064560d23209ee75a5c0abfd338e16b0e2c53438

Compatibility test results for 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5 ==> 064560d23209ee75a5c0abfd338e16b0e2c53438 (PR)
1. Check liveness of validators at old version: 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5
compatibility::simple-validator-upgrade::liveness-check : committed: 8404.075522442188 txn/s, latency: 3909.0541617161716 ms, (p50: 2700 ms, p90: 6600 ms, p99: 27500 ms), latency samples: 303000
2. Upgrading first Validator to new version: 064560d23209ee75a5c0abfd338e16b0e2c53438
compatibility::simple-validator-upgrade::single-validator-upgrading : committed: 5891.457750982968 txn/s, latency: 4791.673083540432 ms, (p50: 5100 ms, p90: 5700 ms, p99: 5900 ms), latency samples: 104620
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 6921.325665883668 txn/s, latency: 4651.8016429353775 ms, (p50: 4700 ms, p90: 5800 ms, p99: 6500 ms), latency samples: 237380
3. Upgrading rest of first batch to new version: 064560d23209ee75a5c0abfd338e16b0e2c53438
compatibility::simple-validator-upgrade::half-validator-upgrading : committed: 7158.48170816516 txn/s, latency: 3716.0385970105076 ms, (p50: 4100 ms, p90: 4500 ms, p99: 4700 ms), latency samples: 135140
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 6606.867754742805 txn/s, latency: 4575.629365410274 ms, (p50: 4300 ms, p90: 7700 ms, p99: 8200 ms), latency samples: 239840
4. upgrading second batch to new version: 064560d23209ee75a5c0abfd338e16b0e2c53438
compatibility::simple-validator-upgrade::rest-validator-upgrading : committed: 5731.704993324787 txn/s, latency: 4207.995558241426 ms, (p50: 2300 ms, p90: 10600 ms, p99: 12800 ms), latency samples: 132380
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 9720.26226526188 txn/s, latency: 3534.6107264231096 ms, (p50: 3100 ms, p90: 8300 ms, p99: 9000 ms), latency samples: 376640
5. check swarm health
Compatibility test for 1c2ee7082d6eff8c811ee25d6f5a7d00860a75d5 ==> 064560d23209ee75a5c0abfd338e16b0e2c53438 passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 064560d23209ee75a5c0abfd338e16b0e2c53438

two traffics test: inner traffic : committed: 9377.612578526796 txn/s, latency: 4250.167556652456 ms, (p50: 4200 ms, p90: 4800 ms, p99: 9800 ms), latency samples: 3565600
two traffics test : committed: 100.02300926184994 txn/s, latency: 2619.374193548387 ms, (p50: 2100 ms, p90: 4200 ms, p99: 8700 ms), latency samples: 1860
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.252, avg: 0.224", "QsPosToProposal: max: 1.825, avg: 1.561", "ConsensusProposalToOrdered: max: 0.328, avg: 0.294", "ConsensusOrderedToCommit: max: 0.401, avg: 0.385", "ConsensusProposalToCommit: max: 0.698, avg: 0.679"]
Max round gap was 1 [limit 4] at version 1568971. Max no progress secs was 5.85224 [limit 15] at version 1568971.
Test Ok

@ibalajiarun ibalajiarun merged commit 26bbc31 into main Jul 16, 2024
87 of 90 checks passed
@ibalajiarun ibalajiarun deleted the runson-docker branch July 16, 2024 23:47
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