Merge pull request #17216 from vbotbuildovich/backport-pr-16842-v23.3…

….x-285 [v23.3.x] raft/recovery: better metric for recovery bandwidth consumption
redpanda-data · Mar 26, 2024 · face89c · face89c
2 parents 02bd8b5 + 7e403a0
commit face89c
Showing 1 changed file with 16 additions and 5 deletions.
diff --git a/src/v/raft/coordinated_recovery_throttle.cc b/src/v/raft/coordinated_recovery_throttle.cc
@@ -91,11 +91,22 @@ void coordinated_recovery_throttle::setup_metrics() {
         namespace sm = ss::metrics;
         _public_metrics.add_group(
           prometheus_sanitize::metrics_name("raft:recovery"),
-          {sm::make_gauge(
-            "partition_movement_available_bandwidth",
-            [this] { return _throttler.available(); },
-            sm::description(
-              "Bandwidth available for partition movement. bytes/sec"))});
+          {// note: deprecate partition_movement_available_bandwidth
+           // in favor of partition_movement_consumed_bandwidth when
+           // possible.
+           sm::make_gauge(
+             "partition_movement_available_bandwidth",
+             [this] { return _throttler.available(); },
+             sm::description(
+               "Bandwidth available for partition movement. bytes/sec")),
+           sm::make_gauge(
+             "partition_movement_consumed_bandwidth",
+             [this] {
+                 return _throttler.last_reset_capacity()
+                        - _throttler.available();
+             },
+             sm::description(
+               "Bandwidth consumed for partition movement. bytes/sec"))});
     }
 }