clay icon indicating copy to clipboard operation
clay copied to clipboard

Set ETH price limit, the same as we do for GNT

Open craigmayhew opened this issue 5 years ago • 1 comments

Rationale

Prevents financial loss due to gas price. I spent orders of magnitude more today on eth than on glm. Setting an eth price would have prevented this, and would prevent temporary gas spikes from clearing out ones eth wallet.

Description

When running golemcli tasks create task.json adhere to a gas price limit set in task.json or possibly set via command line argument on golemcli. e.g. golemcli tasks create task.json --gasprice=5000000

User Interface

I'm only using wasm tasks on golemcli. I'm unsure if this is already in the gui for other types of tasks (at this time wasm is only available via golemcli and not gui.

craigmayhew avatar Nov 04 '19 18:11 craigmayhew

Maximum gas price is currently hardcoded in SCI: https://github.com/golemfactory/golem-smart-contracts-interface/blob/8c66d492c90178ac4ea0d45d79a35ad878c96344/golem_sci/implementation.py#L72-L74 So it won't exceed 20 gwei even in case of price spikes. But it's not customizable.

jiivan avatar Jan 10 '20 15:01 jiivan