Instrument more i/o functions (cowprotocol#3522)

# Description

Looking at [a sample
trace](https://g-0263500beb.grafana-workspace.eu-central-1.amazonaws.com/explore?schemaVersion=1&panes=%7B%22is4%22%3A%7B%22datasource%22%3A%22aess0ehg3ludcb%22%2C%22queries%22%3A%5B%7B%22query%22%3A%22a5ff616eb36199c01b78cacfe18da481%22%2C%22queryType%22%3A%22traceql%22%2C%22refId%22%3A%22A%22%2C%22datasource%22%3A%7B%22type%22%3A%22tempo%22%2C%22uid%22%3A%22aess0ehg3ludcb%22%7D%2C%22limit%22%3A20%2C%22tableType%22%3A%22traces%22%7D%5D%2C%22range%22%3A%7B%22from%22%3A%221753364224905%22%2C%22to%22%3A%221753367824905%22%7D%7D%7D&orgId=1)
I found that "estimator" is taking a long time, but most of the time is
not accounted for. After going through the code I am led to believe it's
the get_trade() function.

# Changes

- [x] Added more instrumentation

Author: fafk

crates/shared/src/price_estimation/buffered.rs

@@ -21,6 +21,7 @@ use {
         time::Duration,
     },
     tokio::{sync::broadcast, task::JoinHandle},
+    tracing::instrument,
 };
 
 /// Buffered configuration.
@@ -86,6 +87,7 @@ where
     Inner: NativePriceBatchFetching + NativePriceEstimating + 'static,
 {
     /// Request to get estimate prices in a batch
+    #[instrument(skip_all)]
     fn estimate_native_price(
         &self,
         token: H160,

crates/shared/src/price_estimation/native_price_cache.rs

@@ -17,7 +17,7 @@ use {
         time::{Duration, Instant},
     },
     tokio::time,
-    tracing::Instrument,
+    tracing::{Instrument, instrument},
 };
 
 #[derive(prometheus_metric_storage::MetricStorage)]
@@ -433,6 +433,7 @@ impl CachingNativePriceEstimator {
 }
 
 impl NativePriceEstimating for CachingNativePriceEstimator {
+    #[instrument(skip_all)]
     fn estimate_native_price(
         &self,
         token: H160,

crates/shared/src/price_estimation/sanitized.rs

@@ -14,6 +14,7 @@ use {
     model::order::BUY_ETH_ADDRESS,
     primitive_types::H160,
     std::sync::Arc,
+    tracing::instrument,
 };
 
 /// Verifies that buy and sell tokens are supported and handles
@@ -53,6 +54,7 @@ impl SanitizedPriceEstimator {
 }
 
 impl PriceEstimating for SanitizedPriceEstimator {
+    #[instrument(skip_all)]
     fn estimate(
         &self,
         query: Arc<Query>,

crates/shared/src/price_estimation/trade_finder.rs

@@ -63,6 +63,7 @@ impl TradeEstimator {
 }
 
 impl Inner {
+    #[instrument(skip_all)]
     async fn estimate(
         self: Arc<Self>,
         query: Arc<Query>,

crates/shared/src/trade_finding/external.rs

@@ -226,6 +226,7 @@ impl TradeFinding for ExternalTradeFinder {
         })
     }
 
+    #[instrument(skip_all)]
     async fn get_trade(&self, query: &Query) -> Result<TradeKind, TradeError> {
         self.shared_query(query).await
     }