fix: update ROUND_TIMEOUT_MS to be reported in histogram

Author: 0xIcarus

consensus/src/counters.rs

@@ -547,11 +547,13 @@ pub static TIMEOUT_COUNT: Lazy<IntCounter> = Lazy::new(|| {
 });
 
 /// Round timeout in milliseconds
-pub static ROUND_TIMEOUT_MS: Lazy<IntCounterVec> = Lazy::new(|| {
-    register_int_counter_vec!(
+pub static ROUND_TIMEOUT_MS: Lazy<Histogram> = Lazy::new(|| {
+    register_histogram!(
         "aptos_consensus_round_timeout_ms",
         "Round timeout in milliseconds",
-        &["round"]
+        vec![
+            100.0, 200.0, 500.0, 1000.0, 2000.0, 5000.0, 10000.0, 20000.0, 50000.0, 100000.0
+        ]
     )
     .unwrap()
 });

consensus/src/round_manager.rs

@@ -333,7 +333,7 @@ impl RoundManager {
         new_round_event: NewRoundEvent,
     ) -> anyhow::Result<()> {
         counters::CURRENT_ROUND.set(new_round_event.round as i64);
-        counters::ROUND_TIMEOUT_MS.with_label_values(&[&new_round_event.round.to_string()]).inc_by(new_round_event.timeout.as_millis() as u64);
+        counters::ROUND_TIMEOUT_MS.observe(new_round_event.timeout.as_millis() as f64);
         match new_round_event.reason {
             NewRoundReason::QCReady => {
                 counters::QC_ROUNDS_COUNT.inc();

crates/aptos-telemetry/src/core_metrics.rs

@@ -140,10 +140,7 @@ fn collect_consensus_metrics(core_metrics: &mut BTreeMap<String, String>) {
     // Get the round timeout in milliseconds
     core_metrics.insert(
         CONSENSUS_ROUND_TIMEOUT_MS.into(),
-        aptos_consensus::counters::ROUND_TIMEOUT_MS
-            .with_label_values(&[&aptos_consensus::counters::CURRENT_ROUND.get().to_string()])
-            .get()
-            .to_string(),
+        get_histogram_values(&aptos_consensus::counters::ROUND_TIMEOUT_MS),
     );
 
     // Get sync info messages count