feat(query): add bucket name to storage http request metrics (#17431)

* feat(query): add bucket name to storage http request metrics

* z

Author: everpcpc

src/common/metrics/src/metrics/storage.rs

@@ -35,6 +35,7 @@ static OMIT_FILTER_ROWS: LazyLock<Counter> = LazyLock::new(|| register_counter("
 struct StorageHttpLabels {
     host: String,
     method: String,
+    bucket: String,
 }
 
 static STORAGE_HTTP_REQUESTS_COUNT: LazyLock<FamilyCounter<StorageHttpLabels>> =
@@ -320,9 +321,13 @@ pub fn metrics_inc_omit_filter_rows(c: u64) {
 }
 
 /// Storage Http metrics.
-pub fn metrics_inc_storage_http_requests_count(host: String, method: String) {
+pub fn metrics_inc_storage_http_requests_count(host: String, method: String, bucket: String) {
     STORAGE_HTTP_REQUESTS_COUNT
-        .get_or_create(&StorageHttpLabels { host, method })
+        .get_or_create(&StorageHttpLabels {
+            host,
+            method,
+            bucket,
+        })
         .inc();
 }
 

src/common/storage/src/http_client.rs

@@ -60,7 +60,16 @@ impl HttpFetch for StorageHttpClient {
             }
             m => m.as_str(),
         };
-        metrics_inc_storage_http_requests_count(host.to_string(), method.to_string());
+        // get first component in path as bucket name
+        let bucket = match url.path_segments() {
+            Some(mut segments) => segments.next().unwrap_or("-"),
+            None => "-",
+        };
+        metrics_inc_storage_http_requests_count(
+            host.to_string(),
+            method.to_string(),
+            bucket.to_string(),
+        );
 
         let (parts, body) = req.into_parts();