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

Add minimum deployment gas points setting #3211

Merged
merged 6 commits into from
Dec 20, 2024
Merged

Add minimum deployment gas points setting #3211

merged 6 commits into from
Dec 20, 2024

Conversation

miloszm
Copy link
Contributor

@miloszm miloszm commented Dec 17, 2024

Adds minimum deployment gas points setting,
enforces a minimum cost for contract deployment even if the contract size is small.
Protects network from spamming with a large number of small contracts.

Implements a top priority #3210

@miloszm miloszm requested a review from herr-seppia December 17, 2024 14:25
@miloszm miloszm force-pushed the min_deploy_points branch 4 times, most recently from c535a75 to a090479 Compare December 17, 2024 15:10
@herr-seppia
Copy link
Member

Can this new value be configurable?

So that we can rollout on testnet without producing forks

@miloszm
Copy link
Contributor Author

miloszm commented Dec 18, 2024

Can this new value be configurable?

So that we can rollout on testnet without producing forks

Done

@miloszm
Copy link
Contributor Author

miloszm commented Dec 19, 2024

This PR has passed the test in a cloud node being part of a testnet cluster.

@miloszm
Copy link
Contributor Author

miloszm commented Dec 19, 2024

Rebased against execution-core to dusk-core renamed master.

@miloszm miloszm merged commit 49f9d63 into master Dec 20, 2024
16 checks passed
@miloszm miloszm deleted the min_deploy_points branch December 20, 2024 09:25
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.

2 participants