Change the metric names to conform to the limiter names

ackintosh · ackintosh · commit 11468d8e0e97 · 2025-04-10T07:12:50.000+09:00
diff --git a/beacon_node/lighthouse_network/src/metrics.rs b/beacon_node/lighthouse_network/src/metrics.rs
@@ -206,16 +206,16 @@ pub static REPORT_PEER_MSGS: LazyLock<Result<IntCounterVec>> = LazyLock::new(||
     )
 });
 
-pub static SELF_LIMITER_REQUEST_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(|| {
+pub static OUTBOUND_REQUEST_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(|| {
     try_create_histogram(
-        "self_limiter_request_idling_seconds",
+        "outbound_request_idling_seconds",
         "The time our own request remained idle in the self-limiter",
     )
 });
 
-pub static RESPONSE_LIMITER_RESPONSE_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(|| {
+pub static RESPONSE_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(|| {
     try_create_histogram(
-        "response_limiter_idling_seconds",
+        "response_idling_seconds",
         "The time our response remained idle in the response limiter",
     )
 });
diff --git a/beacon_node/lighthouse_network/src/rpc/response_limiter.rs b/beacon_node/lighthouse_network/src/rpc/response_limiter.rs
@@ -147,7 +147,7 @@ impl<E: EthSpec> ResponseLimiter<E> {
                     ) {
                         Ok(()) => {
                             metrics::observe_duration(
-                                &crate::metrics::RESPONSE_LIMITER_RESPONSE_IDLING,
+                                &crate::metrics::RESPONSE_IDLING,
                                 timestamp_now().saturating_sub(response.queued_at),
                             );
                             responses.push(response)
diff --git a/beacon_node/lighthouse_network/src/rpc/self_limiter.rs b/beacon_node/lighthouse_network/src/rpc/self_limiter.rs
@@ -286,7 +286,7 @@ impl<Id: ReqId, E: EthSpec> SelfRateLimiter<Id, E> {
         // Finally return any queued events.
         if let Some((peer_id, event, queued_at)) = self.ready_requests.pop() {
             metrics::observe_duration(
-                &crate::metrics::SELF_LIMITER_REQUEST_IDLING,
+                &crate::metrics::OUTBOUND_REQUEST_IDLING,
                 timestamp_now().saturating_sub(queued_at),
             );
             return Poll::Ready(BehaviourAction::NotifyHandler {

Original file line number	Diff line number	Diff line change
`@@ -206,16 +206,16 @@ pub static REPORT_PEER_MSGS: LazyLock<Result<IntCounterVec>> = LazyLock::new(\|\|`
`206`	`206`	`)`
`207`	`207`	`});`
`208`	`208`
`209`		`-pub static SELF_LIMITER_REQUEST_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(\|\| {`
	`209`	`+pub static OUTBOUND_REQUEST_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(\|\| {`
`210`	`210`	`try_create_histogram(`
`211`		`- "self_limiter_request_idling_seconds",`
	`211`	`+ "outbound_request_idling_seconds",`
`212`	`212`	`"The time our own request remained idle in the self-limiter",`
`213`	`213`	`)`
`214`	`214`	`});`
`215`	`215`
`216`		`-pub static RESPONSE_LIMITER_RESPONSE_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(\|\| {`
	`216`	`+pub static RESPONSE_IDLING: LazyLock<Result<Histogram>> = LazyLock::new(\|\| {`
`217`	`217`	`try_create_histogram(`
`218`		`- "response_limiter_idling_seconds",`
	`218`	`+ "response_idling_seconds",`
`219`	`219`	`"The time our response remained idle in the response limiter",`
`220`	`220`	`)`
`221`	`221`	`});`