kata-containers/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2026-08-05 02:19:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

tests: Remove explicit sleeps from "liveness" checks

Browse Source 
The vhost-user tests uses SSH and checking the RAM to test for the
liveness of the VM - replace the explicit sleep before them with
`wait_until()` allowing them to potentially finish earlier.

Signed-off-by: Rob Bradford <rbradford@meta.com>
This commit is contained in:
Rob Bradford
2026-04-15 09:09:33 +01:00
parent a1cfbd6f5c
commit f55a90c170
1 changed files with 12 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
cloud-hypervisor/tests/common/tests_wrappers.rs
View File

@@ -657,12 +657,13 @@ pub(crate) fn test_vhost_user_net(
let desired_ram = 1024 << 20;
resize_command(&api_socket, None, Some(desired_ram), None, None);
thread::sleep(std::time::Duration::new(10, 0));
// Here by simply checking the size (through ssh), we validate
// the connection is still working, which means vhost-user-net
// keeps working after the resize.
assert!(guest.get_total_memory().unwrap_or_default() > 960_000);
assert!(wait_until(Duration::from_secs(10), || guest
.get_total_memory()
.unwrap_or_default()
> 960_000));
}
});
@@ -790,9 +791,10 @@ pub(crate) fn test_vhost_user_blk(
let desired_ram = 1024 << 20;
resize_command(&api_socket, None, Some(desired_ram), None, None);
thread::sleep(std::time::Duration::new(10, 0));
assert!(guest.get_total_memory().unwrap_or_default() > 960_000);
assert!(wait_until(Duration::from_secs(10), || guest
.get_total_memory()
.unwrap_or_default()
> 960_000));
// Check again the content of the block device after the resize
// has been performed.
@@ -1025,8 +1027,10 @@ pub(crate) fn _test_virtio_fs(
let desired_ram = 1024 << 20;
resize_command(&api_socket, None, Some(desired_ram), None, None);
thread::sleep(std::time::Duration::new(30, 0));
assert!(guest.get_total_memory().unwrap_or_default() > 960_000);
assert!(wait_until(Duration::from_secs(30), || guest
.get_total_memory()
.unwrap_or_default()
> 960_000));
// After the resize, check again that file1 exists and its
// content is "foo".