remove deleteContainer, add restartContainer

confluentinc · Oct 30, 2024 · d247571 · d247571
1 parent f0d4d01
commit d247571
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 19 deletions.
diff --git a/src/docker/containers.ts b/src/docker/containers.ts
@@ -134,21 +134,7 @@ export async function stopContainer(id: string) {
   }
 }
 
-export async function deleteContainer(id: string) {
-  const client = new ContainerApi();
-  const init: RequestInit = defaultRequestInit();
-
-  try {
-    await client.containerDelete({ id }, init);
-  } catch (error) {
-    if (error instanceof ResponseError) {
-      logger.error("Error response deleting container:", {
-        status: error.response.status,
-        statusText: error.response.statusText,
-        body: await error.response.clone().json(),
-      });
-    } else {
-      logger.error("Error removing container:", error);
-    }
-  }
+export async function restartContainer(id: string) {
+  await stopContainer(id);
+  await startContainer(id);
 }
diff --git a/src/docker/workflows/index.ts b/src/docker/workflows/index.ts
@@ -2,7 +2,7 @@ import net from "net";
 import { CancellationToken, commands, Progress, window } from "vscode";
 import { ContainerSummary } from "../../clients/docker";
 import { Logger } from "../../logging";
-import { startContainer } from "../containers";
+import { restartContainer, startContainer } from "../containers";
 import { imageExists, pullImage } from "../images";
 
 /** Basic container information for a local resource. */
@@ -122,7 +122,7 @@ export abstract class LocalResourceWorkflow {
               return;
             }
             if (anyRunning) {
-              // TODO: implement stop+start in downstream branch
+              promises.push(restartContainer(container.Id));
             } else {
               promises.push(startContainer(container.Id));
             }