Fix metric name

ChainSafe · Oct 28, 2024 · 13faa80 · 13faa80
1 parent b0c538c
commit 13faa80
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/packages/beacon-node/src/api/impl/validator/index.ts b/packages/beacon-node/src/api/impl/validator/index.ts
@@ -719,7 +719,7 @@ export function getValidatorApi(
         ...getBlockValueLogInfo(engine.value),
       });
 
-      metrics?.blockProductionSelection.inc({
+      metrics?.blockProductionSelections.inc({
         source: ProducedBlockSource.engine,
         reason: EngineBlockSelectionReason.BuilderCensorship,
       });
@@ -734,7 +734,7 @@ export function getValidatorApi(
         ...getBlockValueLogInfo(builder.value),
       });
 
-      metrics?.blockProductionSelection.inc({
+      metrics?.blockProductionSelections.inc({
         source: ProducedBlockSource.builder,
         reason:
           isEngineEnabled === false
@@ -754,7 +754,7 @@ export function getValidatorApi(
         ...getBlockValueLogInfo(engine.value),
       });
 
-      metrics?.blockProductionSelection.inc({
+      metrics?.blockProductionSelections.inc({
         source: ProducedBlockSource.engine,
         reason:
           isBuilderEnabled === false

diff --git a/packages/beacon-node/src/api/impl/validator/utils.ts b/packages/beacon-node/src/api/impl/validator/utils.ts
@@ -63,7 +63,7 @@ export function selectBlockProductionSource(
   switch (builderSelection) {
     case routes.validator.BuilderSelection.ExecutionAlways:
     case routes.validator.BuilderSelection.ExecutionOnly: {
-      metrics?.blockProductionSelection.inc({
+      metrics?.blockProductionSelections.inc({
         source: ProducedBlockSource.engine,
         reason: EngineBlockSelectionReason.EnginePreferred,
       });
@@ -73,30 +73,30 @@ export function selectBlockProductionSource(
     case routes.validator.BuilderSelection.Default:
     case routes.validator.BuilderSelection.MaxProfit: {
       if (builderBoostFactor === MAX_BUILDER_BOOST_FACTOR) {
-        metrics?.blockProductionSelection.inc({
+        metrics?.blockProductionSelections.inc({
           source: ProducedBlockSource.builder,
           reason: BuilderBlockSelectionReason.BuilderPreferred,
         });
         return ProducedBlockSource.builder;
       }
 
       if (builderBoostFactor === BigInt(0)) {
-        metrics?.blockProductionSelection.inc({
+        metrics?.blockProductionSelections.inc({
           source: ProducedBlockSource.engine,
           reason: EngineBlockSelectionReason.EnginePreferred,
         });
         return ProducedBlockSource.engine;
       }
 
       if (engineBlockValue >= (builderBlockValue * builderBoostFactor) / BigInt(100)) {
-        metrics?.blockProductionSelection.inc({
+        metrics?.blockProductionSelections.inc({
           source: ProducedBlockSource.engine,
           reason: EngineBlockSelectionReason.BlockValue,
         });
         return ProducedBlockSource.engine;
       }
 
-      metrics?.blockProductionSelection.inc({
+      metrics?.blockProductionSelections.inc({
         source: ProducedBlockSource.builder,
         reason: BuilderBlockSelectionReason.BlockValue,
       });
@@ -105,7 +105,7 @@ export function selectBlockProductionSource(
 
     case routes.validator.BuilderSelection.BuilderAlways:
     case routes.validator.BuilderSelection.BuilderOnly: {
-      metrics?.blockProductionSelection.inc({
+      metrics?.blockProductionSelections.inc({
         source: ProducedBlockSource.builder,
         reason: BuilderBlockSelectionReason.BuilderPreferred,
       });

diff --git a/packages/beacon-node/src/metrics/metrics/beacon.ts b/packages/beacon-node/src/metrics/metrics/beacon.ts
@@ -161,11 +161,11 @@ export function createBeaconMetrics(register: RegistryMetricCreator) {
       help: "Count of blocks successfully produced",
       labelNames: ["source"],
     }),
-    blockProductionSelection: register.gauge<
+    blockProductionSelections: register.gauge<
       | {source: ProducedBlockSource.engine; reason: EngineBlockSelectionReason}
       | {source: ProducedBlockSource.builder; reason: BuilderBlockSelectionReason}
     >({
-      name: "beacon_block_production_selection_total",
+      name: "beacon_block_production_selections_total",
       help: "Count of all block production selections by source and reason",
       labelNames: ["source", "reason"],
     }),