Skip to content

[V131-specific] Asset-specific Dynamic Outbound Fee Multipliers #check-lint-warning

[V131-specific]

Intended to close #1879 (closed)
'Gas costs are Asset-specific, and so Assets could have distinct Dynamic Outbound Fee Multipliers'.

Regression test .yaml updates illustrate a proposed endpoint format.

#check-lint-warning is intended to solely be for AddGasAsset refactoring to take an Asset argument,
acted on only in the updated-version gas manager.

As the all-Assets outbound multiplier has been at 15000 for a while,
I suggest that before implementation Min/MaxOutboundFeeMultiplierBasisPoints be tightened to or to just around the current multiplier,
then gradually loosened after implementation to allow individual multipliers to change slowly rather than suddenly.

Related:
(regarding !3421 (comment 1766642786) )
!3421 (merged) 'Only spend UTXO unused gas for Migrate outbounds'
which aims to avoid THORChain deducting and reimbursing the full MaxGas for UTXO chains.

As always, feedback is welcome,
for instance if my use of range indices is found to approach unreadability.


2024-02-14 update:

The smoke test update appears to require a change of the expected gas units for an Ethereum Mocknet transaction
( !3423 (2d824ff7) )
; insight as to why would be welcome.

Edited by Multipartite

Merge request reports