destroy always tear down the container (#4443)

vicb · web-flow · commit ec0ef7cd92e2 · 2025-07-02T12:45:40.000Z
fixes #4440
diff --git a/src/workerd/server/container-client.c++ b/src/workerd/server/container-client.c++
@@ -413,10 +413,7 @@ kj::Promise<void> ContainerClient::monitor(MonitorContext context) {
 }
 
 kj::Promise<void> ContainerClient::destroy(DestroyContext context) {
-  const auto [running, _ports] = co_await inspectContainer();
-  if (running) {
-    co_await destroyContainer();
-  }
+  co_await destroyContainer();
 }
 
 kj::Promise<void> ContainerClient::signal(SignalContext context) {

Original file line number	Diff line number	Diff line change
`@@ -413,10 +413,7 @@ kj::Promise<void> ContainerClient::monitor(MonitorContext context) {`
`413`	`413`	`}`
`414`	`414`
`415`	`415`	`kj::Promise<void> ContainerClient::destroy(DestroyContext context) {`
`416`		`- const auto [running, _ports] = co_await inspectContainer();`
`417`		`- if (running) {`
`418`		`- co_await destroyContainer();`
`419`		`- }`
	`416`	`+ co_await destroyContainer();`
`420`	`417`	`}`
`421`	`418`
`422`	`419`	`kj::Promise<void> ContainerClient::signal(SignalContext context) {`