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

Remove node's local min gas price #3896

Open
1 task
Tracked by #1509
ninabarbakadze opened this issue Sep 23, 2024 · 2 comments
Open
1 task
Tracked by #1509

Remove node's local min gas price #3896

ninabarbakadze opened this issue Sep 23, 2024 · 2 comments
Assignees
Labels
WS: BestTxs Reliable and seamless transaction flow

Comments

@ninabarbakadze
Copy link
Member

Description

Currently, every validator node is able to set their local min gas price in app.toml. User's txs won't be accepted by the node if it doesn't meet that local threshold. On top of that we also have a global network min gas price that all transactions need to adhere to.

Having two different thresholds to meet creates fragmented fee policies and makes it harder for the user to anticipate whether their transactions will be accepted or not by the network.

Acceptance Criteria

  • Get rid of node's local min gas price
@ninabarbakadze ninabarbakadze added the WS: BestTxs Reliable and seamless transaction flow label Sep 23, 2024
@rootulp
Copy link
Collaborator

rootulp commented Sep 23, 2024

IIRC we had discussions about this change and we didn't get full support to remove the node's local min gas price. I think one reason to keep it was that validator's can locally configure their local min gas price faster than governance or a hard-fork can coordinate to change the network min gas price.

So I think this issue has a prerequisite to get internal consensus that it should be removed.

@ninabarbakadze
Copy link
Member Author

We did get consensus on this. What we didn't get consensus on was increasing the mempool size. The meeting was recorded so we can go back and check if you feel unsure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WS: BestTxs Reliable and seamless transaction flow
Projects
None yet
Development

No branches or pull requests

2 participants