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

Update Docker images #12026

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Update Docker images #12026

merged 1 commit into from
Feb 15, 2024

Conversation

gedigi
Copy link
Contributor

@gedigi gedigi commented Feb 15, 2024

Update Docker images

@gedigi gedigi added the CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR label Feb 15, 2024
@gedigi gedigi requested a review from a team as a code owner February 15, 2024 00:34
@gedigi gedigi requested a review from a team February 15, 2024 00:34
Copy link

trunk-io bot commented Feb 15, 2024

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite compat success on aptos-node-v1.8.3 ==> d38d119cf30a15f9ab8f44f691b9461910082bce

Compatibility test results for aptos-node-v1.8.3 ==> d38d119cf30a15f9ab8f44f691b9461910082bce (PR)
1. Check liveness of validators at old version: aptos-node-v1.8.3
compatibility::simple-validator-upgrade::liveness-check : committed: 4748 txn/s, latency: 6736 ms, (p50: 6600 ms, p90: 9900 ms, p99: 14900 ms), latency samples: 180460
2. Upgrading first Validator to new version: d38d119cf30a15f9ab8f44f691b9461910082bce
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 1846 txn/s, latency: 15714 ms, (p50: 19400 ms, p90: 22000 ms, p99: 22300 ms), latency samples: 92340
3. Upgrading rest of first batch to new version: d38d119cf30a15f9ab8f44f691b9461910082bce
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 1859 txn/s, latency: 15779 ms, (p50: 18700 ms, p90: 22000 ms, p99: 22300 ms), latency samples: 92980
4. upgrading second batch to new version: d38d119cf30a15f9ab8f44f691b9461910082bce
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 2958 txn/s, latency: 9976 ms, (p50: 9900 ms, p90: 17300 ms, p99: 20600 ms), latency samples: 127200
5. check swarm health
Compatibility test for aptos-node-v1.8.3 ==> d38d119cf30a15f9ab8f44f691b9461910082bce passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on d38d119cf30a15f9ab8f44f691b9461910082bce

two traffics test: inner traffic : committed: 7848 txn/s, latency: 4870 ms, (p50: 4500 ms, p90: 6000 ms, p99: 12200 ms), latency samples: 3390740
two traffics test : committed: 100 txn/s, latency: 2204 ms, (p50: 2100 ms, p90: 2400 ms, p99: 9600 ms), latency samples: 1740
Latency breakdown for phase 0: ["QsBatchToPos: max: 0.226, avg: 0.205", "QsPosToProposal: max: 0.168, avg: 0.135", "ConsensusProposalToOrdered: max: 0.568, avg: 0.538", "ConsensusOrderedToCommit: max: 0.474, avg: 0.454", "ConsensusProposalToCommit: max: 1.019, avg: 0.992"]
Max round gap was 1 [limit 4] at version 1594501. Max no progress secs was 7.687736 [limit 15] at version 1594501.
Test Ok

Copy link

codecov bot commented Feb 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f86d996) 70.2% compared to head (d38d119) 69.9%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##             main   #12026     +/-   ##
=========================================
- Coverage    70.2%    69.9%   -0.4%     
=========================================
  Files        2220     2220             
  Lines      418896   418896             
=========================================
- Hits       294344   292965   -1379     
- Misses     124552   125931   +1379     

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

@gedigi gedigi merged commit 4f4cba3 into main Feb 15, 2024
63 of 88 checks passed
@gedigi gedigi deleted the docker-update-images branch February 15, 2024 17:10
danielxiangzl added a commit that referenced this pull request Feb 16, 2024
* clean error log lines (#12019)

* [table info][2/4] add utils for table info backup and restore and redesign the db read (#11793)

* separate indexer async v2 db from aptosdb

* address comments

* add utils for table info backup and restore and redesign the db read

* address comments to spawn block sync file ops

* address comments

* tests for events and improve event v1 handling (#12012)

* [move-vm] Cache verified modules (#12002)

* [move-vm] Cache verified modules

* fixup! [move-vm] Cache verified modules

* [passkey] Add MAX_BYTES limit for signatures (#11697)

* [passkey] Add MAX_BYTES limit for signatures

* [passkey] Add tracing for AssertionSignature type and fix README

* [passkey] Rebased on latest main, rerun authenticator_regenerate.sh

* Object Code Deployment module with CLI commands generated (#11748)

* [simple] rename RG split in VmChangeSet flag (#12027)

* rename RG split in VmChangeSet flag

old name was stale, when charging was different

* [fuzzing] fixes oss-fuzz FP and fuzz.sh (#12030)

* [fuzzing] fixes oss-fuzz FP and fuzz.sh

* Update Docker images (#12026)

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

* Update release.yaml (#12020)

* Update release.yaml

* enable REFUNDABLE_BYTES

* enable FairnessShuffler

* enable WEBAUTHN_SIGNATURE

* AIP-54 Object Code Deployment release addition

* enable vtxn and jwk consensus

* Update release.yaml

adding aggregators v2 flags, and updating execution onchain config

* add feature flag for zkID (ZK-only mode)

* fix jwk/zkid entries in release yaml 1.10 (#12024)

* update

* update

* Update release.yaml

fix flag name

* Update release.yaml

rename feature

---------

Co-authored-by: aldenhu <[email protected]>
Co-authored-by: hariria <[email protected]>
Co-authored-by: John Chang <[email protected]>
Co-authored-by: danielxiangzl <[email protected]>
Co-authored-by: igor-aptos <[email protected]>
Co-authored-by: Alin Tomescu <[email protected]>
Co-authored-by: zhoujunma <[email protected]>

* Cherry-pick VM changes (#12021)

* [gas] add gas charges for type creation

* [gas-calibration] Add calibration sample

* [move-vm] Implement a per-frame cache for paranoid mode

* fixup! [move-vm] Implement a per-frame cache for paranoid mode

* fixup! fixup! [move-vm] Implement a per-frame cache for paranoid mode

* fixup! fixup! fixup! [move-vm] Implement a per-frame cache for paranoid mode

* fixup! fixup! fixup! fixup! [move-vm] Implement a per-frame cache for paranoid mode

* [gas] add gas charges for dependencies

---------

Co-authored-by: Runtian Zhou <[email protected]>

* trivial doc fix

* [GHA] Upgrade actions/checkout to v4

* jwk ob counters (#12048)

* Revert "[GHA] Upgrade actions/checkout to v4"

This reverts commit 04d078f.

* [CI][indexer] fix the e2e localnet. (#12047)

* fix the e2e localnet.

* fix the e2e localnet.

* bump latest gas feature version to 14

Also be conservative and leave legacy parameters in >14 versions for
now. Need to clean up after REFUNDABLE_BYTES feature is actually enabled
on all networks.

* compat test to be against the testnet tag

* [GHA] Upgrade lint-test.yaml and the dependent actions to checkout@v4

actions/checkout@v4 doesn't behave well if both a workflow and an
invoked action checkout the source code on top of each other.

* [GHA] Update pin for tj-actions/changed-files

* start jwk consensus for google (#12053)

* [consensus] check rpc epoch in epoch_manager (#12018)

* [consensus] check rpc epoch in epoch_manager

* fix gas version (13 is deprecated/cannot be used) (#12064)

* FatalVMError shouldn't create "Delayed materialization code invariant" (#12044)

* Move all visibility checking into AST-level function_checker, simplify that code a bit, and improve diagnostics. (#11948)

* rust changes to move all visibility checking to AST and clean it up a bit
* change `Known attribute ... position` warning to a neater `Attribute .. position` warning
* add FunctionData id_loc to allow pointing at function name in declaration for more concise error messages. abstract messages a bit in function_checker
* add 'inlined from' labels to diagnostics with labels, fix bug in function_checker to enable post-inlining visibility checking

* lint

* fix for small stakes

* assert

---------

Co-authored-by: igor-aptos <[email protected]>
Co-authored-by: jill <[email protected]>
Co-authored-by: George Mitenkov <[email protected]>
Co-authored-by: runtianz <[email protected]>
Co-authored-by: Andrew Hariri <[email protected]>
Co-authored-by: John Chang <[email protected]>
Co-authored-by: Gerardo Di Giacomo <[email protected]>
Co-authored-by: sionescu <[email protected]>
Co-authored-by: Junkil Park <[email protected]>
Co-authored-by: aldenhu <[email protected]>
Co-authored-by: danielxiangzl <[email protected]>
Co-authored-by: Alin Tomescu <[email protected]>
Co-authored-by: Victor Gao <[email protected]>
Co-authored-by: Stelian Ionescu <[email protected]>
Co-authored-by: Stelian Ionescu <[email protected]>
Co-authored-by: larry-aptos <[email protected]>
Co-authored-by: Balaji Arun <[email protected]>
Co-authored-by: Brian R. Murphy <[email protected]>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants