fraud liveness: schedule another liveness event if sequencer is jailed?

[danwt]

Close criteria: make sure to reset liveness timer when sequencer jailed

When sequencer is jailed, the ticker should be removed
When a sequencer becomes active, ticker should start afresh

Should probs be done via onJail hook or similar

---

See also #1028

dymension/x/rollapp/keeper/liveness.go

Lines 62 to 75 in 0a624a3

	if e.IsJail {
	err := k.sequencerKeeper.JailLiveness(ctx, e.RollappId)
	if err != nil {
	return errorsmod.Wrap(err, "jail liveness")
	}
	} else {
	err := k.sequencerKeeper.SlashLiveness(ctx, e.RollappId)
	if err != nil {
	return errorsmod.Wrap(err, "slash liveness")
	}
	}
	ra := k.MustGetRollapp(ctx, e.RollappId)
	k.RescheduleLivenessEvent(ctx, &ra)

[danwt]

"""Within the lifecycle of a rollapp, there are a few situations where liveness events are not needed: in case a proposer is jailed, or in case the rollapp is frozen.
These two cases are not handled, with the rollapp renewing by default liveness events in an infinite schedule.
Recommendation
We recommend updating the liveness scheduling logic to not reschedule events in case the proposer is jailed or when the rollapp is frozen."""

[danwt]

omri: "also re liveness - currently we start to count “liveness” the moment the sequencer registered right? not upon first state update."

we need to make sure it's fair to new sequencer

[danwt]

yea just to make sure I’m clear - we should count once he registered, and not once he sent his first state update. this has big product implications.

[danwt]

superseded by #1328