-
Notifications
You must be signed in to change notification settings - Fork 1k
Gas & Fee #1327
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
Merged
Merged
Gas & Fee #1327
Changes from 53 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
130d304
Reworks `gas` module. Introduces `GasMetering` trait and gas sub-units
grarco b8dd0a1
Adds gas and fee protocol parameters
grarco 0345e79
Adds `split_borrow` method to `WriteLogAndStorage` trait
grarco 01d88a5
Adds a precommit write log to `WriteLog`. Adjusts gas accounting with…
grarco d030d33
Adds core methods for fee unshielding. Brings back `GasLimit` to `u64`
grarco 18a1c82
Adds new CLI arguments for gas & fee
grarco f3a91be
Always expects a block proposer. Moves `WrapperTx` handling inside `d…
grarco 7efbb23
Reviews `ErrorCodes`. Adds fee validation function
grarco 309d13e
Updates unit tests for mandatory block proposer
grarco b0d4500
Renames `BlockAllocator` and makes it track gas too
grarco 310b493
`prepare_proposal` validates gas & fee. Extracts validation to a sepa…
grarco c741770
`process_proposal` validates gas & fees
grarco 72e1c62
Adds benchmarks crate for gas
grarco 2ca6fbd
Whitelisted gas in wasm
grarco 32ad862
Exposes wasm functions for consuming gas
grarco 6b36b78
Renames gas function and adds exposed function for signatures' verifi…
grarco 55e0c4c
Gas metering for sig verification
grarco b35fda0
Updates wasm vps with gas and host sig verification
grarco 67ede24
Fixes `clean-wasm` step of wasm Makefile
grarco f2e3662
Removes masp fees + gas for the vp
grarco 2401b2d
Updates gas computation with multipliers
grarco 99d3ebf
Adds spare gas for each tx in the storage queue
grarco 3c2fdec
`ShellParams` carries a generic wl_storage. Reworks `dispatch_tx` to …
grarco b40c6e7
Helper storage functions to construct fee unshielding tx in protocol
grarco 9124ae4
Updates gas meter objects initializations
grarco 5aa3a89
Dry-run wrapper tx
grarco 300d670
Reworks wasm compilation step. Accepts `floats` wasm feature
grarco 6942795
Renames `gas_payer` to `fee_payer`
grarco 36074e2
Handles wrapper's signer separately
grarco c47b19d
Sdk functions to return the optional `Epoch` for fee unshielding
grarco 1c89ca1
Adds disposable wrapper signer. Adjusts tx construction for fees
grarco 435c95e
Misc adjustments for testing
grarco 5b69afa
Updates e2e tests
grarco d21f801
Updates gas meters in tests
grarco 2ce9523
Adds disposable signer e2e test. Updates other tests
grarco 5c6a387
Adds fee unshielding integration test. Updates other tests
grarco a53eda6
Updates visibility of functions
grarco 0c49f6a
Adds gas and fees protocol parameters to query response
grarco ffd6b60
Reduces the scale of the gas sub-units
grarco c19617c
Fixes `fee_amount` parsing
grarco a50c0c9
Refactors `get_tx_fee` to avoid using `Uint`
grarco 7abb349
Renames fee payers cli arguments
grarco 8910e0e
Fixes gas in tests
grarco 3a941b0
Renames fees cli arguments
grarco 604427c
Improves fee-related logs in the client
grarco 047469b
Moves disposable signing key generation to wallet module
grarco 1531ea6
Saves optional disposable keypair to wallet
grarco 30d6bfa
Fixes broken fee optimization logic in cli. Fixes fee unshielding log…
grarco 33418c7
Adds check on masp gas payer
grarco 48a05a3
Changes `dev` feature to `testing`
grarco c4b28b8
Keep write log changes before fee unshield failure in `charge_fee`
grarco ecf2962
[ci] wasm checksums update
grarco af95690
changelog: #1327
grarco 5b36a37
Removes unused or renamed files
grarco File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| - Implemented the runtime gas and fee system. | ||
| ([\#1327](https://github.com/anoma/namada/pull/1327)) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,6 +3,7 @@ resolver = "2" | |
|
|
||
| members = [ | ||
| "apps", | ||
| "benches", | ||
| "core", | ||
| "ethereum_bridge", | ||
| "proof_of_stake", | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.