feat(deployer): log error in case of failed start_kill

shuttle-hq · Sep 18, 2023 · ecb9740 · ecb9740
1 parent 31ee040
commit ecb9740
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/deployer/src/runtime_manager.rs b/deployer/src/runtime_manager.rs
@@ -13,7 +13,7 @@ use shuttle_proto::{
 };
 use tokio::{io::AsyncBufReadExt, io::BufReader, process, sync::Mutex};
 use tonic::transport::Channel;
-use tracing::{debug, info, trace};
+use tracing::{debug, error, info, trace};
 use uuid::Uuid;
 
 const MANIFEST_DIR: &str = env!("CARGO_MANIFEST_DIR");
@@ -177,7 +177,13 @@ impl RuntimeManager {
 
     pub fn kill_process(&mut self, id: Uuid) {
         if let Some((mut process, _)) = self.runtimes.lock().unwrap().remove(&id) {
-            let _ = process.start_kill();
+            match process.start_kill() {
+                Ok(_) => (),
+                Err(err) => error!(
+                    deployment_id = %id, "failed to start the killing of the runtime: {}",
+                    err
+                ),
+            }
         }
     }