From 2d62db6a9889d4bc17e4086488499e3a698d150d Mon Sep 17 00:00:00 2001
From: dawn <90008@klbr.net>
Date: Sat, 27 Jun 2026 22:16:24 +0000
Subject: [PATCH] [ingest] downcast generic reports to exclude commit and signing key errors from retries

---
 src/ingest/indexer/shard.rs | 21 +++++++++++++++------
 1 file(s) changed, 15 insertion(s)(+), 6 deletion(s)(-)

diff --git a/src/ingest/indexer/shard.rs b/src/ingest/indexer/shard.rs
--- a/src/ingest/indexer/shard.rs
+++ b/src/ingest/indexer/shard.rs
@@ -301,12 +301,21 @@
     // don't retry commit or sync on key fetch errors
     // since we'll just try again later if we get commit or sync again
     fn check_if_retriable_failure(e: &IngestError) -> bool {
-        matches!(
-            e,
-            IngestError::Generic(_)
-                | IngestError::Resolver(ResolverError::Ratelimited)
-                | IngestError::Resolver(ResolverError::Transport(_))
-        )
+        match e {
+            IngestError::Commit(_) | IngestError::NoSigningKey(_) => false,
+            IngestError::Generic(report) => {
+                if report.downcast_ref::<jacquard_repo::error::CommitError>().is_some()
+                    || report.downcast_ref::<crate::resolver::NoSigningKeyError>().is_some()
+                {
+                    false
+                } else {
+                    true
+                }
+            }
+            IngestError::Resolver(ResolverError::Ratelimited) => true,
+            IngestError::Resolver(ResolverError::Transport(_)) => true,
+            _ => false,
+        }
     }
 
     fn handle_commit<'s, 'c>(

--
tangled.sh