oxidecomputer
diff --git a/‎server/src/manager.rs‎
Lines changed: 7 additions & 1 deletion b/‎server/src/manager.rs‎
Lines changed: 7 additions & 1 deletion
@@ -598,7 +598,13 @@ impl JobManager {
     }
 
     pub async fn job_stop(&self, _authn: &Identity, job_id: &JobId) -> Result<(), JobError> {
-        self.monitor(MonitorRequest::Stop(job_id.to_owned())).await
+        if let Some(status) = self.active_jobs.lock().await.get(job_id)
+            && status.is_active()
+        {
+            self.monitor(MonitorRequest::Stop(job_id.to_owned()))
+                .await?;
+        }
+        Ok(())
     }
 
     pub async fn job_output(
Original file line number	Diff line number	Diff line change
`@@ -598,7 +598,13 @@ impl JobManager {`
`598`	`598`	`}`
`599`	`599`
`600`	`600`	`pub async fn job_stop(&self, _authn: &Identity, job_id: &JobId) -> Result<(), JobError> {`
`601`		`- self.monitor(MonitorRequest::Stop(job_id.to_owned())).await`
	`601`	`+ if let Some(status) = self.active_jobs.lock().await.get(job_id)`
	`602`	`+ && status.is_active()`
	`603`	`+ {`
	`604`	`+ self.monitor(MonitorRequest::Stop(job_id.to_owned()))`
	`605`	`+ .await?;`
	`606`	`+ }`
	`607`	`+ Ok(())`
`602`	`608`	`}`
`603`	`609`
`604`	`610`	`pub async fn job_output(`