merge: #2610

2610: fix(si): Ensure that we can recreate a new veritech container r=fnichol a=stack72 Fixes: #2609 When the credentials are incorrect, we want to be able to stop and then restart the container to get the new credentials passed to the container Co-authored-by: stack72 <[email protected]>
systeminit · Aug 16, 2023 · d2a8fb7 · d2a8fb7
2 parents f79ab66 + 841013c
commit d2a8fb7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/labeler.yml b/.github/labeler.yml
@@ -73,6 +73,7 @@ A-si-discord-bot:
   - .prettierrc.js
 A-si:
   - bin/si/**/*
+  - lib/si-cli/**/*
 
 #
 # System Initiative libs (i.e. crates/packages/libraries)

diff --git a/lib/si-cli/src/cmd/start.rs b/lib/si-cli/src/cmd/start.rs
@@ -254,10 +254,9 @@ async fn invoke(app: &AppState, docker: &DockerClient, is_preview: bool) -> CliR
                     continue;
                 }
 
-                println!("Starting existing {0}", container_name.clone());
+                println!("Deleting existing container {0}", container_name.clone());
                 let non_running_container = docker.containers().get(existing.id.as_ref().unwrap());
-                non_running_container.start().await?;
-                continue;
+                non_running_container.delete().await?;
             }
 
             if is_preview {