From d24003b74d232d2ff643a6fc9b5ee10cb5f2b056 Mon Sep 17 00:00:00 2001
From: John Gallagher <john@oxidecomputer.com>
Date: Wed, 21 Aug 2024 09:29:21 -0700
Subject: [PATCH] Tiny cleanup: remove unnecessary `result` check (#6401)

---
 nexus/src/app/background/tasks/blueprint_execution.rs | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/nexus/src/app/background/tasks/blueprint_execution.rs b/nexus/src/app/background/tasks/blueprint_execution.rs
index d13e5428f8..dbbfcb3b14 100644
--- a/nexus/src/app/background/tasks/blueprint_execution.rs
+++ b/nexus/src/app/background/tasks/blueprint_execution.rs
@@ -104,11 +104,9 @@ impl BlueprintExecutor {
             Ok(RealizeBlueprintOutput { needs_saga_recovery }) => {
                 // If executing the blueprint requires activating the saga
                 // recovery background task, do that now.
-                if let Ok(output) = &result {
-                    if output.needs_saga_recovery {
-                        info!(&opctx.log, "activating saga recovery task");
-                        self.saga_recovery.activate();
-                    }
+                if needs_saga_recovery {
+                    info!(&opctx.log, "activating saga recovery task");
+                    self.saga_recovery.activate();
                 }
 
                 json!({