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

adopt AIP-61 terminology for consistency #12123

Merged
merged 5 commits into from
Feb 21, 2024
Merged

adopt AIP-61 terminology for consistency #12123

merged 5 commits into from
Feb 21, 2024

Conversation

alinush
Copy link
Contributor

@alinush alinush commented Feb 20, 2024

Description

Copy link

trunk-io bot commented Feb 20, 2024

⏱️ 21h 52m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-move-unit-coverage 4h 2m 🟩🟩 (+5 more)
windows-build 3h 28m 🟩🟩🟩🟩🟩 (+6 more)
rust-unit-tests 2h 47m 🟥🟩 (+4 more)
rust-move-tests 2h 33m 🟩🟩🟩 (+5 more)
execution-performance / single-node-performance 1h 34m 🟩🟩🟩🟥🟩
rust-smoke-tests 1h 28m 🟩🟩
run-tests-main-branch 1h 3m 🟥🟥🟥🟥🟥 (+5 more)
rust-images / rust-all 1h 🟩🟩🟩🟩
rust-lints 51m 🟩🟩🟩🟩 (+4 more)
forge-e2e-test / forge 36m 🟩🟩
forge-compat-test / forge 35m 🟩🟩
check 34m 🟩🟩🟩🟩 (+5 more)
cli-e2e-tests / run-cli-tests 26m 🟥🟥
check-dynamic-deps 24m 🟩🟩🟩🟩🟩 (+6 more)
general-lints 17m 🟩🟩🟩🟩 (+4 more)
semgrep/ci 5m 🟩🟩🟩🟩🟩 (+6 more)
node-api-compatibility-tests / node-api-compatibility-tests 4m 🟩🟩🟩🟩
file_change_determinator 2m 🟩🟩🟩🟩🟩 (+5 more)
file_change_determinator 2m 🟩🟩🟩🟩 (+5 more)
execution-performance / file_change_determinator 53s 🟩🟩🟩🟩🟩
permission-check 52s 🟩🟩🟩🟩🟩 (+6 more)
file_change_determinator 40s 🟩🟩🟩🟩
permission-check 34s 🟩🟩🟩🟩🟩 (+6 more)
permission-check 31s 🟩🟩🟩🟩🟩 (+6 more)
permission-check 29s 🟩🟩🟩🟩🟩 (+6 more)
permission-check 9s 🟩🟩🟩🟩
determine-docker-build-metadata 7s 🟩🟩🟩🟩

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

Job Duration vs 7d avg Delta
rust-images / rust-all 16m 13m +27%

settingsfeedbackdocs ⋅ learn more about trunk.io

Copy link
Contributor

@heliuchuan heliuchuan left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

codecov bot commented Feb 20, 2024

Codecov Report

Attention: 53 lines in your changes are missing coverage. Please review.

Comparison is base (d771cec) 71.4% compared to head (853648b) 71.4%.
Report is 6 commits behind head on main.

❗ Current head 853648b differs from pull request most recent head eaf0cbf. Consider uploading reports for the commit eaf0cbf to get more accurate results

Files Patch % Lines
aptos-move/aptos-vm/src/oidb_validation.rs 9.0% 20 Missing ⚠️
types/src/oidb/test_utils.rs 0.0% 6 Missing ⚠️
types/src/transaction/authenticator.rs 0.0% 6 Missing ⚠️
types/src/transaction/mod.rs 0.0% 5 Missing ⚠️
types/src/oidb/mod.rs 33.3% 4 Missing ⚠️
types/src/on_chain_config/aptos_features.rs 0.0% 4 Missing ⚠️
types/src/oidb/bn254_circom.rs 0.0% 3 Missing ⚠️
types/src/oidb/circuit_testcases.rs 0.0% 3 Missing ⚠️
types/src/oidb/configuration.rs 66.6% 1 Missing ⚠️
types/src/oidb/openid_sig.rs 0.0% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #12123   +/-   ##
=======================================
  Coverage    71.4%    71.4%           
=======================================
  Files         810      810           
  Lines      184821   184821           
=======================================
+ Hits       132005   132052   +47     
+ Misses      52816    52769   -47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alinush alinush force-pushed the alin/oidb branch 2 times, most recently from e8702ac to 5d6999a Compare February 21, 2024 16:10
@alinush alinush marked this pull request as ready for review February 21, 2024 17:25
@alinush alinush enabled auto-merge (squash) February 21, 2024 17:25

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.

Copy link
Contributor

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

Compatibility test results for aptos-node-v1.9.5 ==> eaf0cbfab21e21f1bcc0227b6895d74f53b73013 (PR)
1. Check liveness of validators at old version: aptos-node-v1.9.5
compatibility::simple-validator-upgrade::liveness-check : committed: 6150 txn/s, latency: 5234 ms, (p50: 4800 ms, p90: 10800 ms, p99: 11600 ms), latency samples: 227580
2. Upgrading first Validator to new version: eaf0cbfab21e21f1bcc0227b6895d74f53b73013
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 634 txn/s, submitted: 669 txn/s, expired: 34 txn/s, latency: 36849 ms, (p50: 43000 ms, p90: 57400 ms, p99: 58800 ms), latency samples: 53339
3. Upgrading rest of first batch to new version: eaf0cbfab21e21f1bcc0227b6895d74f53b73013
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 566 txn/s, submitted: 605 txn/s, expired: 39 txn/s, latency: 37216 ms, (p50: 42900 ms, p90: 53700 ms, p99: 57600 ms), latency samples: 49850
4. upgrading second batch to new version: eaf0cbfab21e21f1bcc0227b6895d74f53b73013
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 2314 txn/s, latency: 12944 ms, (p50: 15500 ms, p90: 18800 ms, p99: 19500 ms), latency samples: 106460
5. check swarm health
Compatibility test for aptos-node-v1.9.5 ==> eaf0cbfab21e21f1bcc0227b6895d74f53b73013 passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on eaf0cbfab21e21f1bcc0227b6895d74f53b73013

two traffics test: inner traffic : committed: 7068 txn/s, latency: 5547 ms, (p50: 5400 ms, p90: 6900 ms, p99: 10200 ms), latency samples: 3053700
two traffics test : committed: 100 txn/s, latency: 2195 ms, (p50: 2100 ms, p90: 2500 ms, p99: 4500 ms), latency samples: 1820
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.239, avg: 0.202", "QsPosToProposal: max: 0.441, avg: 0.388", "ConsensusProposalToOrdered: max: 0.601, avg: 0.566", "ConsensusOrderedToCommit: max: 0.531, avg: 0.501", "ConsensusProposalToCommit: max: 1.101, avg: 1.068"]
Max round gap was 1 [limit 4] at version 1312583. Max no progress secs was 3.894912 [limit 15] at version 1312583.
Test Ok

@alinush alinush merged commit 3bef23f into main Feb 21, 2024
42 of 43 checks passed
@alinush alinush deleted the alin/oidb branch February 21, 2024 19:50
alinush added a commit that referenced this pull request Feb 21, 2024
adopt AIP-61 terminology for consistency
alinush added a commit that referenced this pull request Feb 21, 2024
adopt AIP-61 terminology for consistency
sherry-x pushed a commit that referenced this pull request Feb 21, 2024
adopt AIP-61 terminology for consistency
zjma added a commit that referenced this pull request Feb 22, 2024
* Fix `iss`-related bug in Groth16 path & refactor (#12017)

Co-authored-by: Oliver <[email protected]>

* [aptosvm] Simplify VM flows (#11888)

* Duplicated logic for creating the gas meter for view functions has been removed.
* Duplicated logic for calculating gas used for view functions has been removed.
* There was unreachable code in failure transaction cleanup, where the discarded
status has been returned immediately, but then re-checked again. The first check
is shifted inside.
* No more default transaction metadata.
* Scripts are now validated consistently.
* Simplifies transaction execution function signature to avoid `Option<String>`.
* Removes duplicated features from `AptosVM` and keeps them in `MoveVMExt`.
* Fixes a bug when script hash was not computed for `RunOnAbort`.

Related tests are moved  to `move-e2e-tests`.

* [Compiler V2] Critical edge elimination (#11894)

Implement a pass to eliminate critical edges by splitting them with empty blocks

* [consensus configs] reduce sending block size from 2500 to 1900 (#12091)

### Description

The block output limit is no longer hit with p2p txns.

### Test Plan

Forge `realistic_env_max_load` TPS improves.

* [Indexer-grpc] Add profiling support. (#12034)

* Minor aggregator cleanup (#12013)

* Minor aggregator cleanup

* Addressing PR comments

* [move] rotate_authentication_key_call should not modify OriginatingAddress (#12108)

Co-authored-by: Alin Tomescu <[email protected]>

* [Data Streaming Service] Add dynamic prefetching support

* [Data Streaming Service] Add dynamic prefetching unit tests.

* [Data Streaming Service] Update existing integration tests.

* [State Sync] Add backpressure to fast sync receiver.

* Update perf baseline for gas charging coverage improvements (reducing throughput) (#12124)

* Reduce latency of cloning network sender using Arc pointers (#12103)

* Avoid cloning network sender using Arc pointers

* Removing a clone

* 100 node sweep test

* Removing a few clone operations

* reset forge test

* Removing some clones

* Removing clones

* adopt AIP-61 terminology for consistency (#12123)

adopt AIP-61 terminology for consistency

* [Consensus] Remove non-decoupled execution and refactor for cleaner interfaces (#12104)

* fix jwk key logging (#12090)

* remove spurious error lines (#12137)

* randomness #1: types update from randomnet (#12106)

* types update from randomnet

* update

* lint

* lint

* All validators broadcast commit vote messages (#12059)

* All validators broadcast commit messages

* Forge testing

* Increase timeout for forge

* test forge realistic_env_workload_sweep_test

* run realistic_env_workload_sweep_test

* run realistic_env_workload_sweep_test

* run sweep test

* increase forge runner duration

* forge testing

* Letting the proposer also broadcast commit decision for backward compatibility

* removing forge changes

* Added a TODO

* [vm] Resource access control: runtime engine (#10544)

* [vm] Resource access control: runtime engine

Implements the runtime engine for resource access control:

- a representation of access control specifiers in `loaded_data::runtime_access_specifiers`.
- a loader for access specifiers in `runtime::loader::access_specifier_loader`.
- a new stateful object representing the access control logic in `runtime::access_control`.
- finally the use of the `AccessControlState` in `runtime::interpreter`.

* Addressing reviewer comments.

* Addressing reviewer comments.

* typo: PTLA -> PTAL

* Rebasing: adjusting to upstream changes

* Rebasing

* ObjectCodeDeployment API cleanup update (#12133)

* ObjectCodeDeployment API cleanup update (#12141)

* [Compiler-v2] porting more V1 unit tests to V2 (#12085)

* update tests

* fix bug

* fix-12116

* fix missing space

* add expected got

* remove live-var tests

* fix had_erros

* fix

* Enable the max object nesting check (#12129)

* Resolved the warning for unused variable (#12157)

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* Squashed commit of the following:

commit a50ffec
Author: Zhoujun Ma <[email protected]>
Date:   Thu Feb 22 21:10:12 2024 +0000

    lint

commit 388350f
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 13:04:28 2024 -0800

    update

commit 76f7eca
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:56:04 2024 -0800

    update

commit a663542
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:54:18 2024 -0800

    update

commit b439449
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:34:14 2024 -0800

    update

commit 3378ceb
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:17:06 2024 -0800

    update

commit 6cd6685
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:15:05 2024 -0800

    update

commit 6d89f37
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:13:51 2024 -0800

    update

commit 980f257
Author: zhoujun.ma <[email protected]>
Date:   Thu Feb 22 12:12:04 2024 -0800

    update

commit 16e9349
Author: Zhoujun Ma <[email protected]>
Date:   Thu Feb 22 18:25:08 2024 +0000

    lint

---------

Co-authored-by: Alin Tomescu <[email protected]>
Co-authored-by: Oliver <[email protected]>
Co-authored-by: George Mitenkov <[email protected]>
Co-authored-by: Zekun Wang <[email protected]>
Co-authored-by: Brian (Sunghoon) Cho <[email protected]>
Co-authored-by: Guoteng Rao <[email protected]>
Co-authored-by: Satya Vusirikala <[email protected]>
Co-authored-by: David Wolinsky <[email protected]>
Co-authored-by: Josh Lind <[email protected]>
Co-authored-by: igor-aptos <[email protected]>
Co-authored-by: Sital Kedia <[email protected]>
Co-authored-by: Wolfgang Grieskamp <[email protected]>
Co-authored-by: Teng Zhang <[email protected]>
Co-authored-by: Junkil Park <[email protected]>
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