Add in protocol min gas price #12269
Comments
|
We just need to bring over https://github.com/cosmos/gaia/tree/yaru/rho-glob-fee/x/globalfee or the TGrade implementation. I don't see a need to re-invent the wheel. |
|
Also, this seems to be a duplicate of the original global min fee issue I wrote, which @marbar3778 closed in favor of your issue #8917, which I personally think is too broad in scope and should be closed with more concrete proposals. |
|
Yes #4527. I think we should just adopt what Gaia has IMO as a versioned module ;) |
|
OK, I can handle that. IMO #4527 is different than this task. |
|
im not sure if this should be a module, I was thinking it could be a base app param, that if an app wants to use they can set to > 0. |
|
That was the original idea: to use the param, but I think we want to go to module owned logic. |
|
@alexanderbez it looks that @yaruwangway used the TGrade solution in Gaia. The provenance solution is more complex. They have a mechanism to assess additional fees for every possible message type and smart contract. TGrade does what we need. Question is if we want to handle it through new module or bundle it in an existing module. |
|
Sorry! I never meant to mention Provenance -- I meant TGrade the entire time (I got the two mixed up) :) Yes, Gaia already uses the TGrade approach (a simple module). I think they modified it slightly? Is that true @yaruwangway? In any case, I would say get that module into the SDK and we can call it a day. It's extremely simple and addresses users needs. I don't think this needs to exist in |
|
I'm rebuilding that module in Umee to:
|
|
@alexanderbez , sry for the late reply.(was on holiday). gaia will introduce global fee and we plan to propose the global fee by gov proposal later. It is same module as tgrade. except we will still let global fee to bypass some configured msg types. |
|
Got it. I think we should bring this module into the SDK and sub-module version it. cc @marbar3778 I would close this issue and open a new one that states we will integrate or bring in the Tgrade module. |
|
could we also allow bypass fees like https://github.com/cosmos/gaia/pull/1447/files ? for both global fee and the node's min_gas setup in |
|
so these are two different features. I think we should do it separately but in the same module. |
|
Message filtering is app specific -- that wouldn't belong in the SDK. If we want, we can add a no-op/empty list message filter ante handler decorator that apps can use to extend. |
|
I'm closing this issue. The hub already introduced a min-fee mechanism via If we deem it necessary, we can simply just include that module in the SDK. |
and others
Summary
Last year we had many discussions about gas price problems and a need for solid fee market model
Problem Definition
Chains need a more robust way to define minimum gas price to better protect against network spam. We agree that the ultimate solution will be something like eip1559. But nobody is building it so far, and everyone has other priorities until they will see a fire.
Crypto.com takes more responsible approach and contributed with the multi-layer gas fee proposal. Tgrade implemented what we were proposing a short term solution: gov controlled min gas price. It seams Gaia team is looking to do it as well.
Proposal
Implement gov controlled min gas price. We need to decide how this will be handled:
Personally I prefer the first one, unless we are really deprecating params and moving away from them.
This will be implemented as antehandler unless we take other directions on how "middlewares" will evolve next.
We can reuse one of:
Related
For Admin Use
The text was updated successfully, but these errors were encountered: