orchestrator log cleanup (#394)

* remove print statements from orchestrator

Author: JannikSt

crates/orchestrator/src/api/routes/nodes.rs

@@ -73,7 +73,6 @@ async fn get_nodes(query: Query<NodeQuery>, app_state: Data<AppState>) -> HttpRe
 }
 
 async fn restart_node_task(node_id: web::Path<String>, app_state: Data<AppState>) -> HttpResponse {
-    println!("restart_node_task: {}", node_id);
     let node_address = Address::from_str(&node_id).unwrap();
     let node = app_state.store_context.node_store.get_node(&node_address);
     match node {
@@ -208,7 +207,6 @@ async fn get_node_logs(node_id: web::Path<String>, app_state: Data<AppState>) ->
 }
 
 async fn get_node_metrics(node_id: web::Path<String>, app_state: Data<AppState>) -> HttpResponse {
-    println!("get_node_metrics: {}", node_id);
     let node_address = Address::from_str(&node_id).unwrap();
     let metrics = app_state
         .store_context
@@ -360,7 +358,6 @@ mod tests {
 
         let body = test::read_body(resp).await;
         let json: serde_json::Value = serde_json::from_slice(&body).unwrap();
-        println!("json {:?}", json);
         assert_eq!(
             json["success"], true,
             "Expected success to be true but got {:?}",

crates/orchestrator/src/metrics/mod.rs

@@ -32,9 +32,6 @@ impl MetricsContext {
         label: &str,
         value: f64,
     ) {
-        println!("record_compute_task_gauge: {:?}", node_address);
-        let vals = vec![node_address, task_id, label, &self.pool_id];
-        println!("vals: {:?}", vals);
         self.compute_task_gauges
             .with_label_values(&[node_address, task_id, label, &self.pool_id])
             .set(value);

crates/orchestrator/src/status_update/mod.rs

@@ -68,18 +68,12 @@ impl NodeStatusUpdater {
 
     #[cfg(not(test))]
     async fn is_node_in_pool(&self, node: &OrchestratorNode) -> bool {
-        let node_in_pool: bool = match self
+        let node_in_pool: bool = (self
             .contracts
             .compute_pool
             .is_node_in_pool(self.pool_id, node.address)
-            .await
-        {
-            Result::Ok(result) => result,
-            Result::Err(e) => {
-                println!("Error checking if node is in pool: {}", e);
-                false
-            }
-        };
+            .await)
+            .unwrap_or(false);
         node_in_pool
     }
 

crates/orchestrator/src/store/domains/metrics_store.rs

@@ -122,7 +122,6 @@ impl MetricsStore {
             result.insert(clean_key.to_string(), total);
         }
 
-        println!("result {:?}", result);
         result
     }
 
@@ -131,7 +130,6 @@ impl MetricsStore {
         let all_keys: Vec<String> = con
             .keys(format!("{}:*:*", ORCHESTRATOR_METRICS_STORE))
             .unwrap();
-        println!("all_keys {:?}", all_keys);
 
         let tasks = all_keys
             .iter()