From 87dbd9b7b459d565edfe468e4400d46be6aa6c5d Mon Sep 17 00:00:00 2001
From: twoeths <10568965+twoeths@users.noreply.github.com>
Date: Fri, 5 Sep 2025 17:30:37 +0700
Subject: [PATCH] fix: correct leveldb metrics (#8335)

**Motivation**

- misuse `dbRead*` vs `dbWrite*` which leads to incorrect metric

**Description**

- this PR should ideally be with `unstable` target branch, however
leaving here so that we can investigate #8334 easier
- we will merge this to unstable later anyway


part of #8334

Co-authored-by: Tuyen Nguyen <twoeths@users.noreply.github.com>
---
 packages/db/src/controller/level.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/db/src/controller/level.ts b/packages/db/src/controller/level.ts
index 14c6c2190d03..a28658f90907 100644
--- a/packages/db/src/controller/level.ts
+++ b/packages/db/src/controller/level.ts
@@ -189,9 +189,9 @@ export class LevelDbController implements DatabaseController<Uint8Array, Uint8Ar
 
   /** Capture metrics for db.iterator, db.keys, db.values .all() calls */
   private async metricsAll<T>(promise: Promise<T[]>, bucket: string): Promise<T[]> {
-    this.metrics?.dbWriteReq.inc({bucket}, 1);
+    this.metrics?.dbReadReq.inc({bucket}, 1);
     const items = await promise;
-    this.metrics?.dbWriteItems.inc({bucket}, items.length);
+    this.metrics?.dbReadItems.inc({bucket}, items.length);
     return items;
   }
 
@@ -201,7 +201,7 @@ export class LevelDbController implements DatabaseController<Uint8Array, Uint8Ar
     getValue: (item: T) => K,
     bucket: string
   ): AsyncIterable<K> {
-    this.metrics?.dbWriteReq.inc({bucket}, 1);
+    this.metrics?.dbReadReq.inc({bucket}, 1);
 
     let itemsRead = 0;
 
@@ -212,7 +212,7 @@ export class LevelDbController implements DatabaseController<Uint8Array, Uint8Ar
       yield getValue(item);
     }
 
-    this.metrics?.dbWriteItems.inc({bucket}, itemsRead);
+    this.metrics?.dbReadItems.inc({bucket}, itemsRead);
   }
 
   /** Start interval to capture metric for db size */