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

Preview randomness #12327

Merged
merged 11 commits into from
Mar 3, 2024
Merged

Preview randomness #12327

merged 11 commits into from
Mar 3, 2024

Conversation

zjma
Copy link
Contributor

@zjma zjma commented Mar 1, 2024

Description

Test Plan

Copy link

trunk-io bot commented Mar 1, 2024

⏱️ 22h 11m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-smoke-coverage 3h 🟩
windows-build 2h 19m 🟩🟩🟩🟩🟩 (+2 more)
forge-framework-upgrade-test / forge 2h 13m 🟥🟥🟥 (+1 more)
rust-unit-tests 1h 47m 🟥🟥🟥🟥🟥 (+1 more)
rust-move-tests 1h 47m 🟥🟥🟥🟥
execution-performance / single-node-performance 1h 45m 🟩🟩🟥🟩🟩 (+1 more)
rust-move-unit-coverage 1h 27m 🟩🟩🟩🟩🟩
rust-images / rust-all 1h 19m 🟩🟩🟩🟩🟩 (+1 more)
forge-compat-test / forge 1h 13m 🟩🟩🟩 (+1 more)
forge-e2e-test / forge 1h 8m 🟥🟥🟥 (+1 more)
rust-smoke-tests 1h 1m 🟥🟥🟥🟥🟥
cli-e2e-tests / run-cli-tests 35m 🟩🟩🟩🟩 (+1 more)
rust-lints 33m 🟥🟥🟩🟩🟩 (+1 more)
run-tests-main-branch 25m 🟩🟩🟩🟩🟩 (+2 more)
indexer-grpc-e2e-tests / test-indexer-grpc-docker-compose 25m 🟥🟩🟥🟩🟩 (+1 more)
check 22m 🟩🟩🟩🟩🟩 (+1 more)
rust-unit-coverage 14m 🟥
check-dynamic-deps 13m 🟩🟩🟩🟩🟩 (+2 more)
general-lints 10m 🟩🟩🟩🟩🟩 (+1 more)
node-api-compatibility-tests / node-api-compatibility-tests 5m 🟩🟩🟩🟩🟩 (+1 more)
semgrep/ci 3m 🟩🟩🟩🟩🟩 (+2 more)
file_change_determinator 1m 🟩🟩🟩🟩🟩 (+2 more)
execution-performance / file_change_determinator 1m 🟩🟩🟩🟩🟩 (+1 more)
file_change_determinator 1m 🟩🟩🟩🟩🟩 (+1 more)
file_change_determinator 1m 🟩🟩🟩🟩🟩 (+1 more)
permission-check 35s 🟩🟩🟩🟩🟩 (+3 more)
permission-check 27s 🟩🟩🟩🟩🟩 (+3 more)
permission-check 25s 🟩🟩🟩🟩🟩 (+3 more)
permission-check 19s 🟩🟩🟩🟩🟩 (+2 more)
permission-check 16s 🟩🟩🟩🟩🟩 (+2 more)
determine-docker-build-metadata 14s 🟩🟩🟩🟩🟩 (+1 more)

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

Job Duration vs 7d avg Delta
rust-move-tests 23m 18m +32%
rust-unit-tests 38m 31m +22%

settingsfeedbackdocs ⋅ learn more about trunk.io

@zjma zjma added CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR CICD:run-framework-upgrade-test labels Mar 1, 2024
@zjma zjma marked this pull request as ready for review March 1, 2024 21:54
zjma and others added 3 commits March 1, 2024 21:35
* framework update from randomnet

Squashed commit of the following to fix jwk smoke tests:

commit 3bd0154
Author: zhoujun.ma <[email protected]>
Date:   Tue Feb 27 02:47:57 2024 -0800

    update

commit 2eb6add
Author: zhoujun.ma <[email protected]>
Date:   Tue Feb 27 02:12:27 2024 -0800

    update

commit 9d82151
Author: zhoujun.ma <[email protected]>
Date:   Tue Feb 27 01:51:08 2024 -0800

    debug

fix doc test

fix spec

fix doc

update initialization in genesis

update features.move

initialize randomness in genesis

update golden files

private entry fun check and vm updates

* postpone release builder changes

* update goldenfiles

* fix is_safe_call spec
* smoke test deps and 1st case from randomnet

* update

* more smoke tests
* [move] fixes to `randomness.move` (#12250)

* [move] fixes to randomness.move

* Fixed the Prover spec

Fixed the spec to unblock the PR.

Need to prove the introduced assumptions with proper loop invariants, which should be provable.

* lint

---------

Co-authored-by: Junkil Park <[email protected]>
Co-authored-by: danielxiangzl <[email protected]>

* fix specs

---------

Co-authored-by: Alin Tomescu <[email protected]>
Co-authored-by: Junkil Park <[email protected]>
Co-authored-by: danielxiangzl <[email protected]>
@zjma zjma force-pushed the preview-randomness branch from 6181a74 to cfcc59f Compare March 2, 2024 05:36

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

* dkg/randomness failure injection

* update

* smoke test

* update

* update

* fix scripts
@zjma zjma merged commit cc977b4 into preview Mar 3, 2024
33 of 36 checks passed
@zjma zjma deleted the preview-randomness branch March 3, 2024 01:20

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

github-actions bot commented Mar 3, 2024

❌ Forge suite realistic_env_max_load failure on 19fc1ee7f9a84c7450d0656f9937428e22664991

Forge test runner terminated:
Trailing Log Lines:
             at ./testsuite/forge/src/runner.rs:598:30
  21: forge::run_forge
             at ./testsuite/forge-cli/src/main.rs:424:11
  22: forge::main
             at ./testsuite/forge-cli/src/main.rs:350:21
  23: core::ops::function::FnOnce::call_once
             at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
{"level":"INFO","source":{"package":"aptos_forge","file":"testsuite/forge/src/backend/k8s/cluster_helper.rs:292"},"thread_name":"main","hostname":"forge-e2e-pr-12327-1709429382-19fc1ee7f9a84c7450d0656f9937428e2","timestamp":"2024-03-03T01:42:39.376529Z","message":"Deleting namespace forge-e2e-pr-12327: Some(NamespaceStatus { conditions: None, phase: Some(\"Terminating\") })"}
{"level":"INFO","source":{"package":"aptos_forge","file":"testsuite/forge/src/backend/k8s/cluster_helper.rs:400"},"thread_name":"main","hostname":"forge-e2e-pr-12327-1709429382-19fc1ee7f9a84c7450d0656f9937428e2","timestamp":"2024-03-03T01:42:39.376573Z","message":"aptos-node resources for Forge removed in namespace: forge-e2e-pr-12327"}
Debugging output:
NAME                                   READY   STATUS      RESTARTS   AGE
aptos-node-0-fullnode-eforge65-0       1/1     Running     0          11m
aptos-node-0-validator-0               1/1     Running     0          11m
aptos-node-1-fullnode-eforge65-0       1/1     Running     0          11m
aptos-node-1-validator-0               1/1     Running     0          11m
aptos-node-2-fullnode-eforge65-0       1/1     Running     0          11m
aptos-node-2-validator-0               1/1     Running     0          11m
aptos-node-3-fullnode-eforge65-0       1/1     Running     0          11m
aptos-node-3-validator-0               1/1     Running     0          11m
aptos-node-4-fullnode-eforge65-0       1/1     Running     0          11m
aptos-node-4-validator-0               1/1     Running     0          11m
aptos-node-5-validator-0               1/1     Running     0          11m
aptos-node-6-validator-0               1/1     Running     0          11m
genesis-aptos-genesis-eforge65-8mpbn   0/1     Completed   0          11m

Copy link
Contributor

github-actions bot commented Mar 3, 2024

✅ Forge suite compat success on aptos-node-v1.9.5 ==> 19fc1ee7f9a84c7450d0656f9937428e22664991

Compatibility test results for aptos-node-v1.9.5 ==> 19fc1ee7f9a84c7450d0656f9937428e22664991 (PR)
1. Check liveness of validators at old version: aptos-node-v1.9.5
compatibility::simple-validator-upgrade::liveness-check : committed: 5683 txn/s, latency: 5018 ms, (p50: 4800 ms, p90: 8700 ms, p99: 9600 ms), latency samples: 233040
2. Upgrading first Validator to new version: 19fc1ee7f9a84c7450d0656f9937428e22664991
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 1436 txn/s, latency: 18020 ms, (p50: 19600 ms, p90: 25000 ms, p99: 26400 ms), latency samples: 76160
3. Upgrading rest of first batch to new version: 19fc1ee7f9a84c7450d0656f9937428e22664991
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 671 txn/s, submitted: 705 txn/s, expired: 34 txn/s, latency: 33645 ms, (p50: 38700 ms, p90: 55200 ms, p99: 56400 ms), latency samples: 57764
4. upgrading second batch to new version: 19fc1ee7f9a84c7450d0656f9937428e22664991
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 6371 txn/s, latency: 5223 ms, (p50: 4800 ms, p90: 9300 ms, p99: 10200 ms), latency samples: 229380
5. check swarm health
Compatibility test for aptos-node-v1.9.5 ==> 19fc1ee7f9a84c7450d0656f9937428e22664991 passed
Test Ok

Copy link
Contributor

github-actions bot commented Mar 3, 2024

❌ Forge suite framework_upgrade failure on aptos-node-v1.9.5 ==> 19fc1ee7f9a84c7450d0656f9937428e22664991

Forge test runner terminated:
Trailing Log Lines:
             at ./testsuite/forge/src/runner.rs:598:30
  22: forge::run_forge
             at ./testsuite/forge-cli/src/main.rs:424:11
  23: forge::main
             at ./testsuite/forge-cli/src/main.rs:350:21
  24: core::ops::function::FnOnce::call_once
             at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
{"level":"INFO","source":{"package":"aptos_forge","file":"testsuite/forge/src/backend/k8s/cluster_helper.rs:292"},"thread_name":"main","hostname":"forge-framework-upgrade-pr-12327-1709429375-aptos-node-v1-9-5","timestamp":"2024-03-03T02:05:31.802367Z","message":"Deleting namespace forge-framework-upgrade-pr-12327: Some(NamespaceStatus { conditions: None, phase: Some(\"Terminating\") })"}
{"level":"INFO","source":{"package":"aptos_forge","file":"testsuite/forge/src/backend/k8s/cluster_helper.rs:400"},"thread_name":"main","hostname":"forge-framework-upgrade-pr-12327-1709429375-aptos-node-v1-9-5","timestamp":"2024-03-03T02:05:31.802413Z","message":"aptos-node resources for Forge removed in namespace: forge-framework-upgrade-pr-12327"}
Debugging output:
NAME                                   READY   STATUS      RESTARTS   AGE
aptos-node-0-validator-0               1/1     Running     0          33m
aptos-node-1-validator-0               1/1     Running     0          32m
aptos-node-2-validator-0               1/1     Running     0          31m
aptos-node-3-validator-0               1/1     Running     0          31m
genesis-aptos-genesis-eforge54-2lhzs   0/1     Completed   0          35m

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 CICD:run-framework-upgrade-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant