Correct inclusionDistance in failed attestation log

ChainSafe · Apr 8, 2022 · c475345 · c475345
1 parent b302eca
commit c475345
Showing 1 changed file with 12 additions and 11 deletions.
diff --git a/packages/lodestar/src/metrics/validatorMonitor.ts b/packages/lodestar/src/metrics/validatorMonitor.ts
@@ -226,17 +226,6 @@ export function createValidatorMonitor(
           metrics.validatorMonitor.prevEpochOnChainTargetAttesterMiss.inc({index});
         }
 
-        if (failedAttestation) {
-          logger.verbose("Failed attestation in previous epoch", {
-            validatorIndex: index,
-            prevEpoch: currentEpoch - 1,
-            isPrevSourceAttester: summary.isPrevSourceAttester,
-            isPrevHeadAttester: summary.isPrevHeadAttester,
-            isPrevTargetAttester: summary.isPrevTargetAttester,
-            inclusionDistance: summary.inclusionDistance,
-          });
-        }
-
         const prevEpochSummary = monitoredValidator.summaries.get(previousEpoch);
         const attestationCorrectHead = prevEpochSummary?.attestationCorrectHead;
         if (attestationCorrectHead !== null && attestationCorrectHead !== undefined) {
@@ -264,6 +253,18 @@ export function createValidatorMonitor(
         if (balance !== undefined) {
           metrics.validatorMonitor.prevEpochOnChainBalance.set({index}, balance);
         }
+
+        if (failedAttestation) {
+          logger.verbose("Failed attestation in previous epoch", {
+            validatorIndex: index,
+            prevEpoch: currentEpoch - 1,
+            isPrevSourceAttester: summary.isPrevSourceAttester,
+            isPrevHeadAttester: summary.isPrevHeadAttester,
+            isPrevTargetAttester: summary.isPrevTargetAttester,
+            // inclusionDistance is not available in summary since altair
+            inclusionDistance,
+          });
+        }
       }
     },