chore: clean up from electra review (#7102)

ChainSafe · Sep 23, 2024 · 0dceb83 · 0dceb83
1 parent cd98c23
commit 0dceb83
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 7 deletions.
diff --git a/packages/beacon-node/src/api/impl/validator/index.ts b/packages/beacon-node/src/api/impl/validator/index.ts
@@ -1087,7 +1087,7 @@ export function getValidatorApi(
 
       await waitForSlot(slot); // Must never request for a future slot > currentSlot
 
-      const dataRootHex = toHex(attestationDataRoot);
+      const dataRootHex = toRootHex(attestationDataRoot);
       const aggregate = chain.attestationPool.getAggregate(slot, null, dataRootHex);
       const fork = chain.config.getForkName(slot);
 

diff --git a/packages/beacon-node/src/metrics/metrics/beacon.ts b/packages/beacon-node/src/metrics/metrics/beacon.ts
@@ -122,7 +122,7 @@ export function createBeaconMetrics(register: RegistryMetricCreator) {
 
     headState: {
       unfinalizedPubkeyCacheSize: register.gauge({
-        name: "head_state_unfinalized_pubkey_cache_size",
+        name: "beacon_head_state_unfinalized_pubkey_cache_size",
         help: "Current size of the unfinalizedPubkey2Index cache in the head state",
       }),
     },

diff --git a/packages/beacon-node/src/metrics/metrics/lodestar.ts b/packages/beacon-node/src/metrics/metrics/lodestar.ts
@@ -381,11 +381,11 @@ export function createLodestarMetrics(
 
     epochCache: {
       finalizedPubkeyDuplicateInsert: register.gauge({
-        name: "lodestar_epoch_cache_finalized_pubkey_duplicate_insert",
+        name: "lodestar_epoch_cache_finalized_pubkey_duplicate_insert_total",
         help: "Total count of duplicate insert of finalized pubkeys",
       }),
       newUnFinalizedPubkey: register.gauge({
-        name: "lodestar_epoch_cache_new_unfinalized_pubkey",
+        name: "lodestar_epoch_cache_new_unfinalized_pubkey_total",
         help: "Total count of unfinalized pubkeys added",
       }),
     },

diff --git a/packages/beacon-node/test/unit/util/sszBytes.test.ts b/packages/beacon-node/test/unit/util/sszBytes.test.ts
@@ -6,7 +6,7 @@ import {ForkName, MAX_COMMITTEES_PER_SLOT} from "@lodestar/params";
 import {
   getAttDataFromAttestationSerialized,
   getAttDataFromSignedAggregateAndProofPhase0,
-  getAggregationBitsFromAttestationSerialized as getAggregationBitsFromAttestationSerialized,
+  getAggregationBitsFromAttestationSerialized,
   getBlockRootFromAttestationSerialized,
   getBlockRootFromSignedAggregateAndProofSerialized,
   getSlotFromAttestationSerialized,

diff --git a/packages/params/src/presets/mainnet.ts b/packages/params/src/presets/mainnet.ts
@@ -127,7 +127,6 @@ export const mainnetPreset: BeaconPreset = {
   MAX_PENDING_PARTIALS_PER_WITHDRAWALS_SWEEP: 8,
   // 2**11 * 10**9 (= 2,048,000,000,000) Gwei
   MAX_EFFECTIVE_BALANCE_ELECTRA: 2048000000000,
-  // 2**16 (= 65536)
   MIN_SLASHING_PENALTY_QUOTIENT_ELECTRA: 4096,
   MIN_ACTIVATION_BALANCE: 32000000000,
   PENDING_BALANCE_DEPOSITS_LIMIT: 134217728,

diff --git a/packages/params/src/presets/minimal.ts b/packages/params/src/presets/minimal.ts
@@ -128,7 +128,6 @@ export const minimalPreset: BeaconPreset = {
   MAX_PENDING_PARTIALS_PER_WITHDRAWALS_SWEEP: 1,
   // 2**11 * 10**9 (= 2,048,000,000,000) Gwei
   MAX_EFFECTIVE_BALANCE_ELECTRA: 2048000000000,
-  // 2**16 (= 65536)
   MIN_SLASHING_PENALTY_QUOTIENT_ELECTRA: 4096,
   MIN_ACTIVATION_BALANCE: 32000000000,
   PENDING_BALANCE_DEPOSITS_LIMIT: 134217728,