fix(cmon): Use the correct 'layer' label

Node connection error events were being emitted with a lable named 'chainId', which was not defined on the metrics spec. This was causing nodeConnectionFailures to be treated as unhandled errors.
haouvw · Apr 17, 2023 · a26c484 · a26c484
1 parent fdaa95b
commit a26c484
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/old-poets-camp.md b/.changeset/old-poets-camp.md
@@ -0,0 +1,5 @@
+---
+'@eth-optimism/chain-mon': patch
+---
+
+Fixes a bug in the wd-mon service where a node connection failure event was not handled correctly
diff --git a/packages/chain-mon/src/wd-mon/service.ts b/packages/chain-mon/src/wd-mon/service.ts
@@ -142,7 +142,7 @@ export class WithdrawalMonitor extends BaseServiceV2<Options, Metrics, State> {
         section: 'getBlockNumber',
       })
       this.metrics.nodeConnectionFailures.inc({
-        chainId: this.state.messenger.l1ChainId,
+        layer: 'l1',
         section: 'getBlockNumber',
       })
       await sleep(this.options.sleepTimeMs)