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

[Not for review] forge test with event translation #15434

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

junkil-park
Copy link
Contributor

@junkil-park junkil-park commented Dec 1, 2024

with enabling the event v2 translation

Copy link

trunk-io bot commented Dec 1, 2024

⏱️ 10h 30m total CI duration on this PR
Slowest 15 Jobs Cumulative Duration Recent Runs
execution-performance / single-node-performance 6h 10m 🟩🟩🟩🟩🟩 (+6 more)
execution-performance / test-target-determinator 47m 🟩🟩🟩🟩🟩 (+6 more)
test-target-determinator 27m 🟩🟩🟩🟩🟩 (+2 more)
rust-images-performance / rust-all 23m 🟩
check 21m 🟩🟩🟩🟩🟩 (+1 more)
check-dynamic-deps 17m 🟩🟩🟩🟩🟩 (+8 more)
rust-move-tests 13m 🟩
rust-move-tests 13m 🟩
rust-move-tests 12m 🟩
rust-move-tests 12m 🟩
rust-cargo-deny 12m 🟩🟩🟩🟩🟩 (+3 more)
rust-move-tests 11m
fetch-last-released-docker-image-tag 9m 🟩🟩🟩🟩🟩 (+1 more)
rust-doc-tests 5m 🟥
rust-doc-tests 5m 🟩

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

Job Duration vs 7d avg Delta
execution-performance / single-node-performance 46m 19m +139%

settingsfeedbackdocs ⋅ learn more about trunk.io

@junkil-park junkil-park marked this pull request as draft December 1, 2024 22:02
@junkil-park junkil-park added the CICD:run-forge-e2e-perf Run the e2e perf forge only label Dec 1, 2024

This comment has been minimized.

@junkil-park junkil-park added CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR CICD:build-images when this label is present github actions will start build+push rust images from the PR. CICD:build-failpoints-images Build failpoints docker image CICD:experimental-build CICD:build-indexer-images CICD:build-performance-images build performance docker image variants labels Dec 1, 2024

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@junkil-park junkil-park added CICD:run-execution-performance-test Run execution performance test CICD:run-execution-performance-full-test Run execution performance test (full version) labels Dec 2, 2024

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@junkil-park junkil-park force-pushed the jpark/event-v2-translation-forge-test branch 2 times, most recently from 8a3033f to 799b1f7 Compare December 2, 2024 18:02

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@@ -238,6 +238,8 @@ impl ForgeConfig {
["enable_storage_sharding"] = true.into();
helm_values["validator"]["config"]["indexer_db_config"]["enable_event"] = true.into();
helm_values["fullnode"]["config"]["indexer_db_config"]["enable_event"] = true.into();
helm_values["validator"]["config"]["indexer_db_config"]["enable_event_v2_translation"] = true.into();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will fail compat because this field doesn't exist in the old version of binary.

@junkil-park junkil-park force-pushed the jpark/event-v2-translation-forge-test branch from 799b1f7 to 6bb129a Compare December 2, 2024 20:45

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

github-actions bot commented Dec 2, 2024

✅ Forge suite realistic_env_max_load success on 6bb129ae99f4a5da4c03a6b31560275d2e4d32ff

two traffics test: inner traffic : committed: 13641.32 txn/s, latency: 2913.88 ms, (p50: 2700 ms, p70: 2900, p90: 3000 ms, p99: 3300 ms), latency samples: 5186760
two traffics test : committed: 99.96 txn/s, latency: 2414.04 ms, (p50: 2200 ms, p70: 2400, p90: 2500 ms, p99: 18300 ms), latency samples: 1700
Latency breakdown for phase 0: ["MempoolToBlockCreation: max: 1.922, avg: 1.207", "ConsensusProposalToOrdered: max: 0.330, avg: 0.300", "ConsensusOrderedToCommit: max: 0.398, avg: 0.382", "ConsensusProposalToCommit: max: 0.696, avg: 0.682"]
Max non-epoch-change gap was: 0 rounds at version 0 (avg 0.00) [limit 4], 0.91s no progress at version 2682765 (avg 0.21s) [limit 15].
Max epoch-change gap was: 0 rounds at version 0 (avg 0.00) [limit 4], 15.72s no progress at version 2682763 (avg 15.72s) [limit 16].
Test Ok

Copy link
Contributor

github-actions bot commented Dec 2, 2024

❌ Forge suite compat failure on 010570d3b7aa20889fb5ad0e5b23800aa33f5634 ==> 6bb129ae99f4a5da4c03a6b31560275d2e4d32ff

Forge test runner terminated:
Trailing Log Lines:
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:552:40
  34: std::panicking::try
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:516:19
  35: std::panic::catch_unwind
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panic.rs:146:14
  36: std::rt::lang_start_internal
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/rt.rs:148:20
  37: main
  38: __libc_start_main
  39: _start
Debugging output:
NAME                                    READY   STATUS      RESTARTS      AGE
aptos-node-0-validator-0                0/1     Error       2 (23s ago)   27s
aptos-node-1-validator-0                0/1     Error       3 (30s ago)   63s
aptos-node-2-validator-0                0/1     Error       2 (17s ago)   24s
aptos-node-3-validator-0                0/1     Error       1 (11s ago)   31s
forge-testnet-deployer-mpw2n            0/1     Error       0             20m
genesis-aptos-genesis-eforge110-ch8ft   0/1     Completed   0             20m

@junkil-park junkil-park changed the title [Do not merge] This is for running the forge test [Not for review] forge test with event translation Dec 3, 2024
Copy link
Contributor

This issue is stale because it has been open 45 days with no activity. Remove the stale label, comment or push a commit - otherwise this will be closed in 15 days.

@github-actions github-actions bot added the Stale label Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CICD:build-failpoints-images Build failpoints docker image CICD:build-images when this label is present github actions will start build+push rust images from the PR. CICD:build-indexer-images CICD:build-performance-images build performance docker image variants CICD:experimental-build CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR CICD:run-execution-performance-full-test Run execution performance test (full version) CICD:run-execution-performance-test Run execution performance test CICD:run-forge-e2e-perf Run the e2e perf forge only Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants