From f57966ce409d47a8a1ef6857e63c26c786a31d27 Mon Sep 17 00:00:00 2001
From: danielle-tfh <danielle.nagar@toolsforhumanity.com>
Date: Wed, 15 Jan 2025 16:05:51 +0100
Subject: [PATCH] Add metrics for batch mismatch and skipped request

---
 iris-mpc-gpu/src/server/actor.rs | 1 +
 iris-mpc/src/bin/server.rs       | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/iris-mpc-gpu/src/server/actor.rs b/iris-mpc-gpu/src/server/actor.rs
index 6ee983c5c..47c17fa57 100644
--- a/iris-mpc-gpu/src/server/actor.rs
+++ b/iris-mpc-gpu/src/server/actor.rs
@@ -1514,6 +1514,7 @@ impl ServerActor {
                     "Batch mismatch with node {}. Queues seem to be out of sync.",
                     i
                 );
+                metrics::counter!("batch.mismatch").increment(1);
                 return Err(eyre!(
                     "Batch mismatch with node {}. Queues seem to be out of sync.",
                     i
diff --git a/iris-mpc/src/bin/server.rs b/iris-mpc/src/bin/server.rs
index 2ad947414..20cc9ad03 100644
--- a/iris-mpc/src/bin/server.rs
+++ b/iris-mpc/src/bin/server.rs
@@ -266,6 +266,11 @@ async fn receive_batch(
                             // Some party (maybe us) already meant to delete this request, so we
                             // skip it. Ignore this message when calculating the batch size.
                             msg_counter -= 1;
+                            metrics::counter!("skip.request.deleted.sqs.request").increment(1);
+                            tracing::warn!(
+                                "Skipping request due to it being from synced deleted ids: {}",
+                                smpc_request.signup_id
+                            );
                             continue;
                         }