refactor(blockgas): remove block gas

[tac0turtle]

look into removing blockgas from context and gas service.

It is used in upgrade but i dont see us consuming gas there.

[yihuang]

[feature request] along with removing block gas meter, add BlockGasUsed() to Context, which is filled with the sum of gasUsed of the txs after they are executed, the ethermint feemarket module needs access to that, currently it use ctx.BlockGasMeter().Consumed().

[akaladarshi]

Hey @tac0turtle, Just for clarification when you say removing the BlockGasMeter from context and gas service do you mean replace it with gasMeter or remove it altogether?

[tac0turtle]

the ethermint feemarket module needs access to that

when does this run?

[tac0turtle]

Hey @tac0turtle, Just for clarification when you say removing the BlockGasMeter from context and gas service do you mean replace it with gasMeter or remove it altogether?

gasmeter stays

[yihuang]

the ethermint feemarket module needs access to that

when does this run?

End blocker

[ValarDragon]

Happy to remove the block gas meter!

[yihuang]

another thing to be careful is, it's only safe to remove block gas meter if we check the sum of gas wanted in the process proposal handler, which is only the case when we use the SDK mempool, AFAIK