fix(metrics): version wasn't being properly reported (#1321)

* bet

* bet

* bet

* bet

mod/execution/pkg/client/client.go

@@ -343,8 +343,9 @@ func (s *EngineClient[ExecutionPayloadT]) buildJWTHeader() (http.Header, error)
 	return header, nil
 }
 
+// Name returns the name of the engine client.
 func (s *EngineClient[ExecutionPayloadT]) Name() string {
-	return "EngineClient"
+	return "engine-client"
 }
 
 // ================================ Info ================================

mod/node-builder/pkg/services/version/metrics.go

@@ -43,7 +43,12 @@ func newVersionMetrics(
 
 // reportVersion increments the versionReported counter.
 func (vm *versionMetrics) reportVersion(version string) {
-	vm.logger.Info("this node is running", "version", version)
+	vm.logger.Info(
+		"this node is running beacon-kit software",
+		"version",
+		version,
+	)
 	vm.sink.IncrementCounter(
-		"beacon_kit.runtime.version.reported", "version", version)
+		"beacon_kit.runtime.version.reported", "version", version,
+	)
 }

mod/node-builder/pkg/services/version/version.go

@@ -38,8 +38,8 @@ type ReportingService struct {
 	logger log.Logger[any]
 	// version represents the current version of the running chain.
 	version string
-	// ticker is used to trigger periodic logging at specified intervals.
-	ticker *time.Ticker
+	// reportingInterval is the interval at which the version is reported.
+	reportingInterval time.Duration
 	// metrics contains the metrics for the version service.
 	metrics *versionMetrics
 }
@@ -51,30 +51,31 @@ func NewReportingService(
 	version string,
 ) *ReportingService {
 	return &ReportingService{
-		logger:  logger,
-		version: version,
-		ticker: time.NewTicker(
-			defaultReportingInterval,
-		),
-		metrics: newVersionMetrics(logger, telemetrySink),
+		logger:            logger,
+		version:           version,
+		reportingInterval: defaultReportingInterval,
+		metrics:           newVersionMetrics(logger, telemetrySink),
 	}
 }
 
 // Name returns the name of the service.
 func (*ReportingService) Name() string {
-	return "ReportingService"
+	return "reporting"
 }
 
 // Start begins the periodic logging of the chain version.
 func (v *ReportingService) Start(ctx context.Context) error {
+	ticker := time.NewTicker(v.reportingInterval)
+	v.metrics.reportVersion(v.version)
 	go func() {
 		for {
 			select {
+			case <-ticker.C:
+				v.metrics.reportVersion(v.version)
+				continue
 			case <-ctx.Done():
-				v.ticker.Stop()
+				ticker.Stop()
 				return
-			case <-v.ticker.C:
-				v.metrics.reportVersion(v.version)
 			}
 		}
 	}()

mod/storage/pkg/manager/manager.go

@@ -73,7 +73,7 @@ func NewDBManager[
 func (m *DBManager[
 	BeaconBlockT, BlockEventT, SubscriptionT,
 ]) Name() string {
-	return "DBManager"
+	return "db-manager"
 }
 
 // TODO: fr implementation