Skip to content

Commit

Permalink
fix(op-node): remove 3s stepCtx for sequencer (#111)
Browse files Browse the repository at this point in the history
  • Loading branch information
bnoieh committed Jan 11, 2024
1 parent ffec6c8 commit 71b17cc
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions op-node/rollup/driver/state.go
Original file line number Diff line number Diff line change
Expand Up @@ -369,13 +369,7 @@ func (s *Driver) eventLoop() {
case <-stepReqCh:
s.metrics.SetDerivationIdle(false)
s.log.Debug("Derivation process step", "onto_origin", s.derivation.Origin(), "attempts", stepAttempts)
stepCtx := context.Background()
if s.driverConfig.SequencerEnabled && !s.driverConfig.SequencerStopped && s.driverConfig.SequencerPriority {
var cancelStep context.CancelFunc
stepCtx, cancelStep = context.WithTimeout(ctx, 3*time.Second)
defer cancelStep()
}
err := s.derivation.Step(stepCtx)
err := s.derivation.Step(context.Background())
stepAttempts += 1 // count as attempt by default. We reset to 0 if we are making healthy progress.
if err == io.EOF {
s.log.Debug("Derivation process went idle", "progress", s.derivation.Origin(), "err", err)
Expand Down

0 comments on commit 71b17cc

Please sign in to comment.