remove unnecessary Result<()> return value from terminate function

Author: rolyatmax

plane/src/drone/backend_manager.rs

@@ -259,11 +259,7 @@ impl BackendManager {
         }
     }
 
-    pub async fn terminate(
-        self: &Arc<Self>,
-        kind: TerminationKind,
-        reason: TerminationReason,
-    ) -> Result<()> {
+    pub async fn terminate(self: &Arc<Self>, kind: TerminationKind, reason: TerminationReason) {
         let state = self
             .state
             .lock()
@@ -276,8 +272,6 @@ impl BackendManager {
             TerminationKind::Hard => state.to_hard_terminating(reason),
         };
         self.set_state(new_state);
-
-        Ok(())
     }
 
     pub fn mark_terminated(self: &Arc<Self>, exit_code: Option<i32>) -> Result<()> {

plane/src/drone/executor.rs

@@ -232,7 +232,7 @@ impl Executor {
                     manager.clone()
                 };
 
-                manager.terminate(*kind, *reason).await?;
+                manager.terminate(*kind, *reason).await;
             }
         }