On 13/01/2026 00.01, Yodel Eldar wrote:
Introduce a functional test of vhost-user-bridge and enter it into
MAINTAINERS under the vhost section.
The test runs vhost-user-bridge as a subprocess, then launches a guest
with four backends: a unix domain socket for vhost-user, a UDP socket, a
user-mode net, and a hubport to hub the UDP and user backends; only the
vhost-user backend is exposed, the rest are deviceless. This
configuration mimics the testing setup described in the initial commit
of vhost-user-bridge in 3595e2eb0a23.
A file containing a hardcoded UUID is created by the test in a scratch
file and exposed to the guest via the tftp parameter of the user netdev.
The guest requests the file by invoking tftp, and the test verifies its
sha256 hashsum.
Similarly, a file containing another hardcoded UUID is created in the
guest. A call to check_http_download() serves the file via http to the
host and verifies integrity with its hashsum.
Suggested-by: Cédric Le Goater <[email protected]>
Suggested-by: Marc-André Lureau <[email protected]>
Suggested-by: Michael S. Tsirkin <[email protected]>
Suggested-by: Thomas Huth <[email protected]>
Signed-off-by: Yodel Eldar <[email protected]>
---
...
+ def test_vhost_user_bridge(self):
+ prompt = "~ # "
+ host_uuid_filename = "vubr-test-uuid.txt"
+ guest_uuid_path = "/tmp/uuid.txt"
+ kernel_path = self.ASSET_KERNEL_INITRAMFS.fetch()
+
+ vubr_path = self.build_file("contrib", "vhost-user-bridge",
+ "vhost-user-bridge")
+ if is_readable_executable_file(vubr_path) is None:
+ self.skipTest("Could not find a readable and executable "
+ "vhost-user-bridge")
+
+ vubr_log_path = self.log_file("vhost-user-bridge.log")
+ self.log.info("For the vhost-user-bridge application log,"
+ f" see: {vubr_log_path}")
+
+ sock_dir = self.socket_dir()
+ ud_socket_path = os.path.join(sock_dir.name, "vubr-test.sock")
+
+ tftpdir = self.scratch_file("tftp")
+ shutil.rmtree(tftpdir, ignore_errors=True)
+ os.mkdir(tftpdir)
+ host_uuid_path = self.scratch_file("tftp", host_uuid_filename)
+ with open(host_uuid_path, "w") as host_uuid_file:
FYI, I just got another test merged that checks the files with pylint now -
you might need to add an encoding="ascii" or encoding="utf-8" to that open()
call now to avoid that pylint complains.
+ host_uuid_file.write(self.HOST_UUID)
+
+ with Ports() as ports:
+ lport, rport, hostfwd_port = ports.find_free_ports(3)
+
+ self.configure_vm(ud_socket_path, lport, rport, hostfwd_port,
+ tftpdir)
+
+ vubr_args = self.assemble_vubr_args(vubr_path, ud_socket_path,
+ lport, rport)
+
+ with open(vubr_log_path, "w+") as vubr_log, \
dito.
+ subprocess.Popen(vubr_args, stdin=subprocess.DEVNULL,
+ stdout=vubr_log,
+ stderr=subprocess.STDOUT) as vubr_proc:
+ self.launch_kernel(kernel_path, wait_for=prompt)
+
+ exec_command_and_wait_for_pattern(self,
+ f"tftp -g -r {host_uuid_filename} 10.0.2.2 ; "
+ f"sha256sum {host_uuid_filename}", self.HOST_UUID_HSUM)
+ wait_for_console_pattern(self, prompt)
+
+ exec_command_and_wait_for_pattern(self,
+ f"echo -n '{self.GUEST_UUID}' > {guest_uuid_path}", prompt)
+ self.check_http_download(guest_uuid_path, self.GUEST_UUID_HSUM)
+ wait_for_console_pattern(self, prompt)
+
+ self.vm.shutdown()
+ vubr_proc.terminate()
+ vubr_proc.wait()
+
+if __name__ == '__main__':
+ LinuxKernelTest.main()
With the encoding=... added:
Reviewed-by: Thomas Huth <[email protected]>
