fix: Adding in the missing suggestions that Aiden has poinited out

core/startos/src/manager/manager_seed.rs

@@ -29,8 +29,15 @@ impl ManagerSeed {
         )
         .await
         {
-            Err(e) if e.kind == ErrorKind::NotFound => (), // Already stopped
-            Err(e) if e.kind == ErrorKind::Timeout => (),  // Already stopped In theory
+            Err(e) if e.kind == ErrorKind::NotFound => {
+                tracing::info!(
+                    "Command for package {command_id} should already be stopped",
+                    command_id = &self.manifest.id
+                );
+            } // Already stopped
+            Err(e) if e.kind == ErrorKind::Timeout => {
+                tracing::warn!("Command for package {command_id} had to be timed out, but we have dropped which means it should be killed", command_id = &self.manifest.id);
+            } // Already stopped In theory
             a => a?,
         }
         Ok(())

core/startos/src/util/docker.rs

@@ -113,11 +113,10 @@ pub async fn stop_container(
     signal: Option<Signal>,
 ) -> Result<(), Error> {
     let mut cmd = Command::new(CONTAINER_TOOL);
+    let mut cmd = cmd.timeout(timeout);
     cmd.arg("stop");
     if let Some(dur) = timeout {
-        cmd.arg("-t")
-            .arg(dur.as_secs().to_string())
-            .timeout(Some(dur));
+        cmd.arg("-t").arg(dur.as_secs().to_string());
     }
     if let Some(sig) = signal {
         cmd.arg("-s").arg(sig.to_string());