From c51ba731995172e8ae5388931291340b572c3cd0 Mon Sep 17 00:00:00 2001 From: Fupan Li Date: Fri, 19 Jul 2024 09:45:29 +0800 Subject: [PATCH] container: fix the issue of send signal to process It's better to check the container's status before try to send signal to it. Since there's no need to send signal to it when the container's stopped. Signed-off-by: Fupan Li --- .../virt_container/src/container_manager/container_inner.rs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/runtime-rs/crates/runtimes/virt_container/src/container_manager/container_inner.rs b/src/runtime-rs/crates/runtimes/virt_container/src/container_manager/container_inner.rs index eaa68fc064..78804e8a03 100644 --- a/src/runtime-rs/crates/runtimes/virt_container/src/container_manager/container_inner.rs +++ b/src/runtime-rs/crates/runtimes/virt_container/src/container_manager/container_inner.rs @@ -275,6 +275,10 @@ impl ContainerInner { signal: u32, all: bool, ) -> Result<()> { + if self.check_state(vec![ProcessStatus::Stopped]).await.is_ok() { + return Ok(()); + } + let mut process_id: agent::ContainerProcessID = process.clone().into(); if all { // force signal init process