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 base-gas.md #9317

Merged
merged 1 commit into from
Jul 31, 2023
Merged

Update base-gas.md #9317

merged 1 commit into from
Jul 31, 2023

Conversation

cryptomolot
Copy link
Contributor

@cryptomolot cryptomolot commented Jul 26, 2023

replaced the broken links. I hope I had the right logic (moving files to a different gas_schedule folder).

I am also not sure about numbers of some parametres (they are different compared to new transaction.rs ; if you want I can update this with new numbers too.

@cryptomolot cryptomolot requested a review from saharct as a code owner July 26, 2023 11:00
@cryptomolot cryptomolot mentioned this pull request Jul 28, 2023
Copy link
Contributor

@saharct saharct left a comment

Choose a reason for hiding this comment

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

Thanks!

@saharct saharct enabled auto-merge (squash) July 31, 2023 22:01
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

✅ Forge suite compat success on aptos-node-v1.5.1 ==> 8d46bdd00f9a474e5c539932a3701e8effced19c

Compatibility test results for aptos-node-v1.5.1 ==> 8d46bdd00f9a474e5c539932a3701e8effced19c (PR)
1. Check liveness of validators at old version: aptos-node-v1.5.1
compatibility::simple-validator-upgrade::liveness-check : committed: 4615 txn/s, latency: 7080 ms, (p50: 7400 ms, p90: 10200 ms, p99: 10500 ms), latency samples: 170760
2. Upgrading first Validator to new version: 8d46bdd00f9a474e5c539932a3701e8effced19c
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 1596 txn/s, latency: 15627 ms, (p50: 18700 ms, p90: 22000 ms, p99: 22500 ms), latency samples: 92580
3. Upgrading rest of first batch to new version: 8d46bdd00f9a474e5c539932a3701e8effced19c
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 1781 txn/s, latency: 15960 ms, (p50: 19200 ms, p90: 22100 ms, p99: 22600 ms), latency samples: 92620
4. upgrading second batch to new version: 8d46bdd00f9a474e5c539932a3701e8effced19c
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 3203 txn/s, latency: 9244 ms, (p50: 10200 ms, p90: 12600 ms, p99: 13800 ms), latency samples: 137760
5. check swarm health
Compatibility test for aptos-node-v1.5.1 ==> 8d46bdd00f9a474e5c539932a3701e8effced19c passed
Test Ok

@github-actions
Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 8d46bdd00f9a474e5c539932a3701e8effced19c

two traffics test: inner traffic : committed: 6521 txn/s, latency: 6005 ms, (p50: 5700 ms, p90: 7800 ms, p99: 11100 ms), latency samples: 2823600
two traffics test : committed: 100 txn/s, latency: 3330 ms, (p50: 3100 ms, p90: 4300 ms, p99: 6900 ms), latency samples: 1720
Max round gap was 1 [limit 4] at version 1391120. Max no progress secs was 3.876757 [limit 10] at version 1391120.
Test Ok

@github-actions
Copy link
Contributor

✅ Forge suite framework_upgrade success on aptos-node-v1.5.1 ==> 8d46bdd00f9a474e5c539932a3701e8effced19c

Compatibility test results for aptos-node-v1.5.1 ==> 8d46bdd00f9a474e5c539932a3701e8effced19c (PR)
Upgrade the nodes to version: 8d46bdd00f9a474e5c539932a3701e8effced19c
framework_upgrade::framework-upgrade::full-framework-upgrade : committed: 4841 txn/s, latency: 6748 ms, (p50: 6300 ms, p90: 9900 ms, p99: 13200 ms), latency samples: 179120
5. check swarm health
Compatibility test for aptos-node-v1.5.1 ==> 8d46bdd00f9a474e5c539932a3701e8effced19c passed
Test Ok

@saharct saharct merged commit f385cdd into aptos-labs:main Jul 31, 2023
gedigi pushed a commit that referenced this pull request Aug 2, 2023
xbtmatt pushed a commit that referenced this pull request Aug 13, 2023
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