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.

Sign up for GitHub

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

Jump to bottom

Update Docker images #13227

Merged
merged 1 commit into from
May 9, 2024
Merged

Update Docker images #13227

merged 1 commit into from
May 9, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented May 7, 2024 •
edited

Update Docker images. Generated by the Docker Update Images workflow.

@github-actions github-actions bot requested a review from a team as a code owner May 7, 2024 18:06
@github-actions github-actions bot added the CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR label May 7, 2024
@trunk-io Trunk.io
Copy link

trunk-io bot commented May 7, 2024 •
edited

⏱️ 10h 41m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-smoke-tests 2h 24m 🟩 🟩 
rust-lints 1h 20m 🟩 🟩 
rust-build-cached-packages 1h 16m 🟩 🟩 
test-target-determinator 1h 14m 🟩 🟩 
rust-targeted-unit-tests 1h 14m 🟩 🟩 
windows-build 45m 🟩 🟩 
rust-images / rust-all 32m 🟩 🟩 
forge-e2e-test / forge 27m 🟩 🟩 
forge-compat-test / forge 24m 🟥 🟩 
rust-move-tests 15m 🟩 🟩 
cli-e2e-tests / run-cli-tests 12m 🟩 🟩 
run-tests-main-branch 8m 🟩 🟩 
check 8m 🟩 🟩 
execution-performance / test-target-determinator 7m 🟩 🟩 
general-lints 4m 🟩 🟩 
indexer-grpc-e2e-tests / test-indexer-grpc-docker-compose 4m 🟩 🟩 
check-dynamic-deps 3m 🟩 🟩 
node-api-compatibility-tests / node-api-compatibility-tests 2m 🟩 🟩 
semgrep/ci 59s 🟩 🟩 
file_change_determinator 23s 🟩 🟩 
file_change_determinator 23s 🟩 🟩 
file_change_determinator 21s 🟩 🟩 
forge-framework-upgrade-test / forge 20s 🟩 🟩 
execution-performance / single-node-performance 16s 🟩 🟩 
permission-check 7s 🟩 🟩 
permission-check 6s 🟩 🟩 
permission-check 6s 🟩 🟩 
permission-check 5s 🟩 🟩 
determine-docker-build-metadata 4s 🟩 🟩 
permission-check 4s 🟩 🟩 

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

Job Duration vs 7d avg Delta
rust-build-cached-packages 8m 5m +56%
rust-lints 10m 8m +30%
rust-targeted-unit-tests 7m 17m -60%
execution-performance / single-node-performance 8s 23m -99%

settings ⋅ feedback ⋅ docs ⋅ learn more about trunk.io

@gedigi gedigi enabled auto-merge (squash) May 7, 2024 18:26
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@gedigi gedigi added CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR and removed CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR labels May 9, 2024
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions
Copy link
Contributor Author

github-actions bot commented May 9, 2024

✅ Forge suite realistic_env_max_load success on 4b94bd346d4f7532228614823437229fdf209172

two traffics test: inner traffic : committed: 7927 txn/s, latency: 4943 ms, (p50: 4700 ms, p90: 5700 ms, p99: 10900 ms), latency samples: 3424520
two traffics test : committed: 100 txn/s, latency: 1796 ms, (p50: 1800 ms, p90: 2100 ms, p99: 3800 ms), latency samples: 1840
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.206, avg: 0.201", "QsPosToProposal: max: 0.258, avg: 0.221", "ConsensusProposalToOrdered: max: 0.444, avg: 0.408", "ConsensusOrderedToCommit: max: 0.363, avg: 0.342", "ConsensusProposalToCommit: max: 0.766, avg: 0.751"]
Max round gap was 1 [limit 4] at version 1657514. Max no progress secs was 4.882733 [limit 15] at version 1657514.
Test Ok

@github-actions GitHub Actions
Copy link
Contributor Author

github-actions bot commented May 9, 2024

✅ Forge suite compat success on 01b24e7e3548382dd25440b39a0438a993387f12 ==> 4b94bd346d4f7532228614823437229fdf209172

Compatibility test results for 01b24e7e3548382dd25440b39a0438a993387f12 ==> 4b94bd346d4f7532228614823437229fdf209172 (PR)
1. Check liveness of validators at old version: 01b24e7e3548382dd25440b39a0438a993387f12
compatibility::simple-validator-upgrade::liveness-check : committed: 5860 txn/s, latency: 5620 ms, (p50: 5200 ms, p90: 7700 ms, p99: 12100 ms), latency samples: 205120
2. Upgrading first Validator to new version: 4b94bd346d4f7532228614823437229fdf209172
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 1530 txn/s, latency: 19078 ms, (p50: 18400 ms, p90: 29900 ms, p99: 30200 ms), latency samples: 90300
3. Upgrading rest of first batch to new version: 4b94bd346d4f7532228614823437229fdf209172
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 1850 txn/s, latency: 15721 ms, (p50: 18300 ms, p90: 22300 ms, p99: 22600 ms), latency samples: 90680
4. upgrading second batch to new version: 4b94bd346d4f7532228614823437229fdf209172
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 3362 txn/s, latency: 8819 ms, (p50: 9700 ms, p90: 11700 ms, p99: 12900 ms), latency samples: 144600
5. check swarm health
Compatibility test for 01b24e7e3548382dd25440b39a0438a993387f12 ==> 4b94bd346d4f7532228614823437229fdf209172 passed
Test Ok

@gedigi gedigi merged commit d0d77dc into main May 9, 2024
101 of 102 checks passed
@gedigi gedigi deleted the docker-update-images branch May 9, 2024 18:14
@gedigi gedigi mentioned this pull request May 9, 2024
Merged
21 tasks
georgemitenkov pushed a commit that referenced this pull request May 15, 2024
@github-actions @gedigi @georgemitenkov
Update Docker images (#13227)
b03f961 
Co-authored-by: gedigi <19227040+gedigi@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sionescu sionescu sionescu approved these changes

@gedigi gedigi gedigi approved these changes

Assignees
No one assigned
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sionescu @gedigi