chore: Remove autometrics

These were not used AFAIK.
get10101 · Dec 14, 2023 · 82255a7 · 82255a7
1 parent dbaefa6
commit 82255a7
Show file tree

Hide file tree

Showing 13 changed files with 7 additions and 233 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/coordinator/Cargo.toml b/coordinator/Cargo.toml
@@ -44,10 +44,6 @@ version = "0.9.4"
 version = "1"
 features = ["backtrace"]
 
-[dependencies.autometrics]
-version = "0.5"
-features = ["prometheus-exporter"]
-
 [dependencies.axum]
 version = "0.6.20"
 features = ["ws", "query", "multipart"]

diff --git a/coordinator/src/admin.rs b/coordinator/src/admin.rs
@@ -4,7 +4,6 @@ use crate::parse_channel_id;
 use crate::routes::AppState;
 use crate::AppError;
 use anyhow::Context;
-use autometrics::autometrics;
 use axum::extract::Path;
 use axum::extract::Query;
 use axum::extract::State;
@@ -36,7 +35,6 @@ pub struct Balance {
     pub onchain: u64,
 }
 
-#[autometrics]
 pub async fn get_balance(State(state): State<Arc<AppState>>) -> Result<Json<Balance>, AppError> {
     spawn_blocking(move || {
         let offchain = state.node.inner.get_ldk_balance();
@@ -127,7 +125,6 @@ impl From<(SubChannel, Option<Contract>, String, Option<OffsetDateTime>)> for Dl
     }
 }
 
-#[autometrics]
 #[instrument(skip_all, err(Debug))]
 pub async fn list_dlc_channels(
     State(state): State<Arc<AppState>>,

diff --git a/coordinator/src/bin/coordinator.rs b/coordinator/src/bin/coordinator.rs
@@ -311,9 +311,6 @@ async fn main() -> Result<()> {
         }
     });
 
-    // Start the metrics exporter
-    autometrics::prometheus_exporter::init();
-
     tracing::debug!("Listening on http://{}", http_address);
 
     match axum::Server::bind(&http_address)

diff --git a/coordinator/src/node.rs b/coordinator/src/node.rs
@@ -16,7 +16,6 @@ use anyhow::bail;
 use anyhow::ensure;
 use anyhow::Context;
 use anyhow::Result;
-use autometrics::autometrics;
 use bitcoin::hashes::hex::ToHex;
 use bitcoin::secp256k1::PublicKey;
 use commons::order_matching_fee_taker;
@@ -260,7 +259,6 @@ impl Node {
         Ok(())
     }
 
-    #[autometrics]
     async fn open_position(
         &self,
         conn: &mut PgConnection,
@@ -406,7 +404,6 @@ impl Node {
         Ok(())
     }
 
-    #[autometrics]
     pub async fn start_closing_position(
         &self,
         conn: &mut PgConnection,
@@ -567,7 +564,6 @@ impl Node {
         Ok(channel_details)
     }
 
-    #[autometrics]
     pub fn process_incoming_dlc_messages(&self) {
         if !self
             .inner

diff --git a/coordinator/src/orderbook/trading.rs b/coordinator/src/orderbook/trading.rs
@@ -6,7 +6,6 @@ use anyhow::anyhow;
 use anyhow::bail;
 use anyhow::Context;
 use anyhow::Result;
-use autometrics::autometrics;
 use bitcoin::secp256k1::PublicKey;
 use bitcoin::Network;
 use bitcoin::XOnlyPublicKey;
@@ -276,7 +275,7 @@ pub async fn process_new_order(
 /// The caller is expected to provide a list of `opposite_direction_orders` of [`OrderType::Limit`]
 /// and opposite [`Direction`] to the `market_order`. We nevertheless ensure that this is the case
 /// to be on the safe side.
-#[autometrics]
+
 fn match_order(
     market_order: &Order,
     opposite_direction_orders: Vec<Order>,