[op-conductor] make safe check interval optional (ethereum-optimism#1…

…0321)
siddharth0a · Apr 29, 2024 · a7ff230 · a7ff230
1 parent f72faa9
commit a7ff230
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/op-conductor/flags/flags.go b/op-conductor/flags/flags.go
@@ -74,6 +74,7 @@ var (
 		Name:    "healthcheck.safe-interval",
 		Usage:   "Interval between safe head progression measured in seconds",
 		EnvVars: opservice.PrefixEnvVar(EnvVarPrefix, "HEALTHCHECK_SAFE_INTERVAL"),
+		Value:   1200,
 	}
 	HealthCheckMinPeerCount = &cli.Uint64Flag{
 		Name:    "healthcheck.min-peer-count",
@@ -103,7 +104,6 @@ var requiredFlags = []cli.Flag{
 	ExecutionRPC,
 	HealthCheckInterval,
 	HealthCheckUnsafeInterval,
-	HealthCheckSafeInterval,
 	HealthCheckMinPeerCount,
 }
 
@@ -112,6 +112,7 @@ var optionalFlags = []cli.Flag{
 	RPCEnableProxy,
 	RaftBootstrap,
 	HealthCheckSafeEnabled,
+	HealthCheckSafeInterval,
 }
 
 func init() {

diff --git a/op-conductor/health/monitor.go b/op-conductor/health/monitor.go
@@ -192,6 +192,7 @@ func (hm *SequencerHealthMonitor) healthCheck() error {
 		return ErrSequencerNotHealthy
 	}
 
+	hm.log.Info("sequencer is healthy")
 	return nil
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -192,6 +192,7 @@ func (hm *SequencerHealthMonitor) healthCheck() error { @@
     		return ErrSequencerNotHealthy
     	}
+    	hm.log.Info("sequencer is healthy")
     	return nil
     }
@@ Expand Down @@