Merge pull request #5545 from ethereum-optimism/felipe/event-dispatcher
feat(proxyd): consensus event dispatcher
This commit is contained in:
commit
34884e451d
@ -17,11 +17,14 @@ const (
|
|||||||
PollerInterval = 1 * time.Second
|
PollerInterval = 1 * time.Second
|
||||||
)
|
)
|
||||||
|
|
||||||
|
type OnConsensusBroken func()
|
||||||
|
|
||||||
// ConsensusPoller checks the consensus state for each member of a BackendGroup
|
// ConsensusPoller checks the consensus state for each member of a BackendGroup
|
||||||
// resolves the highest common block for multiple nodes, and reconciles the consensus
|
// resolves the highest common block for multiple nodes, and reconciles the consensus
|
||||||
// in case of block hash divergence to minimize re-orgs
|
// in case of block hash divergence to minimize re-orgs
|
||||||
type ConsensusPoller struct {
|
type ConsensusPoller struct {
|
||||||
cancelFunc context.CancelFunc
|
cancelFunc context.CancelFunc
|
||||||
|
listeners []OnConsensusBroken
|
||||||
|
|
||||||
backendGroup *BackendGroup
|
backendGroup *BackendGroup
|
||||||
backendState map[*Backend]*backendState
|
backendState map[*Backend]*backendState
|
||||||
@ -150,6 +153,16 @@ func WithAsyncHandler(asyncHandler ConsensusAsyncHandler) ConsensusOpt {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func WithListener(listener OnConsensusBroken) ConsensusOpt {
|
||||||
|
return func(cp *ConsensusPoller) {
|
||||||
|
cp.AddListener(listener)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (cp *ConsensusPoller) AddListener(listener OnConsensusBroken) {
|
||||||
|
cp.listeners = append(cp.listeners, listener)
|
||||||
|
}
|
||||||
|
|
||||||
func WithBanPeriod(banPeriod time.Duration) ConsensusOpt {
|
func WithBanPeriod(banPeriod time.Duration) ConsensusOpt {
|
||||||
return func(cp *ConsensusPoller) {
|
return func(cp *ConsensusPoller) {
|
||||||
cp.banPeriod = banPeriod
|
cp.banPeriod = banPeriod
|
||||||
@ -354,6 +367,7 @@ func (cp *ConsensusPoller) UpdateBackendGroupConsensus(ctx context.Context) {
|
|||||||
- with minimum peer count
|
- with minimum peer count
|
||||||
- updated recently
|
- updated recently
|
||||||
- not lagging
|
- not lagging
|
||||||
|
- in sync
|
||||||
*/
|
*/
|
||||||
|
|
||||||
peerCount, inSync, latestBlockNumber, _, lastUpdate, bannedUntil := cp.getBackendState(be)
|
peerCount, inSync, latestBlockNumber, _, lastUpdate, bannedUntil := cp.getBackendState(be)
|
||||||
@ -397,7 +411,9 @@ func (cp *ConsensusPoller) UpdateBackendGroupConsensus(ctx context.Context) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if broken {
|
if broken {
|
||||||
// propagate event to other interested parts, such as cache invalidator
|
for _, l := range cp.listeners {
|
||||||
|
l()
|
||||||
|
}
|
||||||
log.Info("consensus broken", "currentConsensusBlockNumber", currentConsensusBlockNumber, "proposedBlock", proposedBlock, "proposedBlockHash", proposedBlockHash)
|
log.Info("consensus broken", "currentConsensusBlockNumber", currentConsensusBlockNumber, "proposedBlock", proposedBlock, "proposedBlockHash", proposedBlockHash)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -289,6 +289,11 @@ func TestConsensus(t *testing.T) {
|
|||||||
h2.ResetOverrides()
|
h2.ResetOverrides()
|
||||||
bg.Consensus.Unban()
|
bg.Consensus.Unban()
|
||||||
|
|
||||||
|
listenerCalled := false
|
||||||
|
bg.Consensus.AddListener(func() {
|
||||||
|
listenerCalled = true
|
||||||
|
})
|
||||||
|
|
||||||
for _, be := range bg.Backends {
|
for _, be := range bg.Backends {
|
||||||
bg.Consensus.UpdateBackend(ctx, be)
|
bg.Consensus.UpdateBackend(ctx, be)
|
||||||
}
|
}
|
||||||
@ -334,7 +339,7 @@ func TestConsensus(t *testing.T) {
|
|||||||
// should resolve to 0x1, since 0x2 is out of consensus at the moment
|
// should resolve to 0x1, since 0x2 is out of consensus at the moment
|
||||||
require.Equal(t, "0x1", bg.Consensus.GetConsensusBlockNumber().String())
|
require.Equal(t, "0x1", bg.Consensus.GetConsensusBlockNumber().String())
|
||||||
|
|
||||||
// later, when impl events, listen to broken consensus event
|
require.True(t, listenerCalled)
|
||||||
})
|
})
|
||||||
|
|
||||||
t.Run("broken consensus with depth 2", func(t *testing.T) {
|
t.Run("broken consensus with depth 2", func(t *testing.T) {
|
||||||
|
Loading…
Reference in New Issue
Block a user