Use errors.Is in conductor side stopping sequencer

celo-org · Aug 19, 2024 · 5a330b4 · 5a330b4
1 parent 4af385f
commit 5a330b4
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/op-conductor/conductor/service.go b/op-conductor/conductor/service.go
@@ -683,12 +683,17 @@ func (oc *OpConductor) transferLeader() error {
 }
 
 func (oc *OpConductor) stopSequencer() error {
-	oc.log.Info("stopping sequencer", "server", oc.cons.ServerID(), "leader", oc.leader.Load(), "healthy", oc.healthy.Load(), "active", oc.seqActive.Load())
+	oc.log.Info(
+		"stopping sequencer",
+		"server", oc.cons.ServerID(),
+		"leader", oc.leader.Load(),
+		"healthy", oc.healthy.Load(),
+		"active", oc.seqActive.Load())
 
 	_, err := oc.ctrl.StopSequencer(context.Background())
 	if err != nil {
-		if strings.Contains(err.Error(), driver.ErrSequencerAlreadyStopped.Error()) {
-			oc.log.Warn("sequencer already stopped.", "err", err)
+		if errors.Is(err, driver.ErrSequencerAlreadyStopped) {
+			oc.log.Warn("sequencer already stopped", "err", err)
 		} else {
 			return errors.Wrap(err, "failed to stop sequencer")
 		}