Skip to content

fix: tps zero metrics#20656

Merged
deffrian merged 7 commits intomerge-train/spartanfrom
nikita/n-tps-zero-metrics
Feb 26, 2026
Merged

fix: tps zero metrics#20656
deffrian merged 7 commits intomerge-train/spartanfrom
nikita/n-tps-zero-metrics

Conversation

@deffrian
Copy link
Collaborator

@deffrian deffrian commented Feb 18, 2026

Ref: A-493
Updated dashboard:
https://aztecprotocol.github.io/benchmark-page-data/bench/?branch=pr-20656

  • Split peerCount metric into total connections and healthy connections
  • Fixed memory leak in peer manager: peerConnectedAt map entries are now deleted after recording connection duration
  • Fixed mined-delay tracking for TxPoolV2
  • Added attestation pool instrumentation
  • Renamed metrics/groups
  • Other zero metric fixes

@deffrian deffrian changed the base branch from next to merge-train/spartan February 18, 2026 17:17
@deffrian deffrian force-pushed the nikita/n-tps-zero-metrics branch from d4264ee to a04ef40 Compare February 18, 2026 17:22
@deffrian deffrian added the ci-full Run all master checks. label Feb 18, 2026
@deffrian deffrian changed the title fix: fn tps zero metrics [WIP]fix: fn tps zero metrics Feb 18, 2026
@deffrian deffrian marked this pull request as ready for review February 18, 2026 17:28
@deffrian deffrian changed the title [WIP]fix: fn tps zero metrics fix: tps zero metrics Feb 18, 2026
@deffrian deffrian marked this pull request as draft February 18, 2026 17:31
@deffrian deffrian added the ci-draft Run CI on draft PRs. label Feb 18, 2026
@deffrian deffrian force-pushed the nikita/n-tps-zero-metrics branch 2 times, most recently from ac433fa to b9903c5 Compare February 19, 2026 21:38
@deffrian deffrian added the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 19, 2026
@deffrian deffrian marked this pull request as ready for review February 19, 2026 21:46
@deffrian deffrian removed ci-full Run all master checks. ci-draft Run CI on draft PRs. labels Feb 19, 2026
@AztecBot AztecBot removed the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 19, 2026
@deffrian deffrian added the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 20, 2026
@AztecBot AztecBot removed the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 20, 2026
@deffrian deffrian force-pushed the nikita/n-tps-zero-metrics branch from 29b8e0d to cdbe63b Compare February 21, 2026 00:28
@deffrian deffrian added the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 21, 2026
@AztecBot AztecBot removed the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 21, 2026
@deffrian deffrian added the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 21, 2026
@AztecBot AztecBot removed the ci-network-bench Run Spartan network benchmarks (TPS + proving) on this PR label Feb 21, 2026
Copy link
Contributor

@mralj mralj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pre-approving, I have only small nitpicks about naming

}
}

public objectAdded(key: bigint | string): void {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't like naming of this - it's to vague

this.txAddedTimestamp.set(key, Date.now());
}

public objectRemoved(key: bigint | string): void {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar to above, naming is too vague

@deffrian deffrian force-pushed the nikita/n-tps-zero-metrics branch from ba1a67b to 7aaf82a Compare February 26, 2026 18:27
@deffrian deffrian merged commit 437f090 into merge-train/spartan Feb 26, 2026
10 checks passed
@deffrian deffrian deleted the nikita/n-tps-zero-metrics branch February 26, 2026 18:58
github-merge-queue bot pushed a commit that referenced this pull request Mar 3, 2026
BEGIN_COMMIT_OVERRIDE
fix: track last seen nonce in case of stale fallback L1 RPC node
(#20855)
feat: Validate num txs in block proposals (#20850)
fix(archiver): enforce checkpoint boundary on rollbackTo (#20908)
fix: tps zero metrics (#20656)
fix: handle scientific notation in bigintConfigHelper (#20929)
feat(aztec): node enters standby mode on genesis root mismatch (#20938)
fix: logging of class instances (#20807)
feat(slasher): make slash grace period relative to rollup upgrade time
(#20942)
chore: add script to find PRs to backport (#20956)
chore: remove unused prover-node dep (#20955)
fix: increase minFeePadding in e2e_bot bridge resume tests and harden
GasFees.mul() (#20962)
feat(sequencer): (A-526) rotate publishers when send fails (#20888)
chore: (A-554) bump reth version 1.6.0 -> 1.11.1 for eth devnet (#20889)
chore: metric on how many epochs validator has been on committee
(#20967)
fix: set wallet minFeePadding in BotFactory constructor (#20992)
chore: deflake epoch invalidate block test (#21001)
chore(sequencer): e2e tests for invalid signature recovery in checkpoint
attestations (#20971)
chore: deflake duplicate proposals and attestations (#20990)
chore: deflake epochs mbps test (#21003)
feat: reenable function selectors in txPublicSetupAllowList (#20909)
fix: limit offenses when voting in tally slashing mode by
slashMaxPayloadSize (#20683)
fix(spartan): wire SEQ_L1_PUBLISHING_TIME_ALLOWANCE_IN_SLOT env var
(#21017)
END_COMMIT_OVERRIDE
johnathan79717 pushed a commit that referenced this pull request Mar 4, 2026
Ref: A-493
- Split `peerCount` metric into total connections and healthy
connections
- Fixed memory leak in peer manager: `peerConnectedAt` map entries are
now deleted after recording connection duration
  - Fixed mined-delay tracking for `TxPoolV2`
  - Added attestation pool instrumentation
  - Renamed metrics/groups
  - Other zero metric fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants