mirrors/crosvm
1
0
Fork 0
mirror of https://chromium.googlesource.com/crosvm/crosvm synced 2024-11-24 12:34:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

e2e_tests: Clarify prebuilt download logs

Browse source 
This helps identify a potential deadlock by the NamedLock.

BUG=None
TEST=CQ

Change-Id: I0ef9606b6b5e36a1564ff3fee7720ae013365cc7
Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/4228215
Reviewed-by: Vikram Auradkar <auradkar@google.com>
Commit-Queue: Dennis Kempin <denniskempin@google.com>
This commit is contained in:
Dennis Kempin 2023-02-07 10:58:07 -08:00 committed by crosvm LUCI
parent 887e2a1859
commit c2ec26d7e8
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
e2e_tests/tests/fixture/vm.rs
View file

@ -170,7 +170,6 @@ impl TestVm {
let kernel_path = kernel_path();
if env::var("CROSVM_CARGO_TEST_KERNEL_BINARY").is_err() {
if !kernel_path.exists() {
println!("Downloading kernel prebuilt:");
download_file(&kernel_prebuilt_url(), &kernel_path).unwrap();
}
}
@ -179,7 +178,6 @@ impl TestVm {
let rootfs_path = rootfs_path();
if env::var("CROSVM_CARGO_TEST_ROOTFS_IMAGE").is_err() {
if !rootfs_path.exists() {
println!("Downloading rootfs prebuilt:");
download_file(&rootfs_prebuilt_url(), &rootfs_path).unwrap();
}
}

2
prebuilts/src/lib.rs
View file

@ -80,9 +80,11 @@ pub fn download_file(url: &str, destination: &Path) -> Result<()> {
// Another process may have already downloaded this since we last checked.
if destination.exists() {
println!("Prebuilt {destination:?} has already been downloaded by another process.");
return Ok(());
}
println!("Downloading prebuilt {url} to {destination:?}");
let mut cmd = sys::download_command(url, destination);
match cmd.status() {
Ok(exit_code) => {