elixir-omg icon indicating copy to clipboard operation
elixir-omg copied to clipboard
verified publisher icon omgnetwork

EXIT_PROCESSOR_SLA_SECONDS instead of EXIT_PROCESSOR_SLA_MARGIN

Open unnawut opened this issue 5 years ago • 4 comments

Overview

Currently EXIT_PROCESSOR_SLA_MARGIN is in number of blocks while EXIT_PERIOD_SECONDS is in seconds. This is prone to misunderstanding.

Ino 22 hours ago can we unify EXIT_PROCESSOR_SLA_MARGIN into seconds?

Ino 22 hours ago I’m afraid this is prone to a mistake

piotr 22 hours ago move to seconds could be a good move (consistent with block time & exit period settings, as opposed to other margins), however might require some legwork

Suggestion is to replace EXIT_PROCESSOR_SLA_MARGIN with EXIT_PROCESSOR_SLA_SECONDS

unnawut avatar Mar 17 '20 08:03 unnawut

What values do we do for EXIT_PROCESSOR_SLA_SECONDS for docker-compose

souradeep-das avatar Apr 23 '20 11:04 souradeep-das

So as discussed earlier we should be able to derive from EXIT_PROCESSOR_SLA_MARGIN and ETHEREUM_BLOCK_TIME_SECONDS.

Also might be worth using EXIT_PROCESSOR_SLA_MS to have the same time unit as most other configs.

unnawut avatar Apr 23 '20 12:04 unnawut

cc'ing @kevsul if I missed any other conversations on this one!

unnawut avatar Apr 23 '20 12:04 unnawut

But isn't MIN_EXIT_PERIOD_SECONDS in seconds ?

souradeep-das avatar Apr 23 '20 12:04 souradeep-das