Skip to content

Commit

Permalink
refactor(stylus-verifier): add trace logging on image build; rename s…
Browse files Browse the repository at this point in the history
…tart_container to run_container
  • Loading branch information
rimrakhimov committed Oct 3, 2024
1 parent 130e82c commit c5ce76d
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions stylus-verifier/stylus-verifier-logic/src/docker.rs
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ pub async fn run_reproducible(
.await
.context("copying directory to container")?;

let output = start_container(docker, &container_id)
let output = run_container(docker, &container_id)
.await
.context("running container")?;

Expand Down Expand Up @@ -140,6 +140,7 @@ RUN rustup component add rust-src --toolchain {toolchain}-x86_64-unknown-linux-g
match result {
Ok(info) => {
if let Some(value) = info.stream {
tracing::trace!(image_name = image_name, value = value, "building an image");
output.push(value)
}
}
Expand Down Expand Up @@ -222,7 +223,7 @@ async fn copy_directory_to_container(
Ok(())
}

async fn start_container(docker: &Docker, container_id: &str) -> Result<String, anyhow::Error> {
async fn run_container(docker: &Docker, container_id: &str) -> Result<String, anyhow::Error> {
docker.start_container::<String>(container_id, None).await?;
let mut attach_results = docker
.attach_container::<String>(
Expand Down

0 comments on commit c5ce76d

Please sign in to comment.