Skip to content

Commit

Permalink
merge: #2698
Browse files Browse the repository at this point in the history
2698: fix(si): Ensure we are passing the correct name of the container to delete r=stack72 a=stack72



Co-authored-by: stack72 <[email protected]>
  • Loading branch information
si-bors-ng[bot] and stack72 authored Aug 24, 2023
2 parents d33fda2 + 40941af commit f09dfe1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/si-cli/src/cmd/update.rs
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ async fn invoke(
app.stop().await?;

for container in &update.containers {
let container_name = format!("local-{0}-1", container.repository);
let container_name = format!("local-{0}-1", container.repository.clone());
let container_summary = app
.container_engine()
.get_existing_container(container_name.clone())
Expand All @@ -231,7 +231,7 @@ async fn invoke(
)
.await?;
app.container_engine()
.cleanup_image(container_name.to_string())
.cleanup_image(container.repository.clone())
.await?;
}
}
Expand Down

0 comments on commit f09dfe1

Please sign in to comment.