fuel-core icon indicating copy to clipboard operation
fuel-core copied to clipboard

chore(gas_price_service): config flag to set static da gas price

Open rymnc opened this issue 6 months ago • 4 comments

with the eigen integration, we don't need to dynamically inc/dec the da gas price, and hence we should use a static one that can be configured via the cli

rymnc avatar Jun 25 '25 14:06 rymnc

I think we have everything to control that=D We can set min and max to point to the same value

Image

xgreenx avatar Jun 25 '25 16:06 xgreenx

Yes. But we will still be writing things to the DB and we don't need to be. If the DA price is static, then there is no reason to run that portion of the gas price service.

MitchTurner avatar Jun 26 '25 14:06 MitchTurner

The alternative would be just to have the gas price determine the gas price as before and not actually set it to be static.

MitchTurner avatar Jun 26 '25 14:06 MitchTurner

We need to take into account the compression regardless. This is a feature that is missing from the current ETH DA as well: we need to use the compressed bytes instead of the uncompressed bytes. The compression ratio can be as high as 8x and that dramatically changes the prediction. With static, we don't have the predicted profit anymore, but we still need to accommodate DoS attacks of incompressible Txs.

MitchTurner avatar Jun 26 '25 14:06 MitchTurner