autotest / tp-libvirt

Test Provider for Libvirt and related virtualization backends
Other
33 stars 169 forks source link

libvirt_rng: update rng to use external snapshot #5872

Closed yanqzhan closed 1 month ago

yanqzhan commented 2 months ago
Only external snapshot is officially supported, and it supports
revert now. Update rng cases to use this. Since every step is
external snapshot now, no need to do the 3rd snapshot test which
was called snapshot_with_rng.
yanqzhan commented 2 months ago

JOB LOG : /var/log/avocado/job-results/job-2024-09-03T06.29-b3de8ec/job.log (1/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_rdm.snapshot_shutoff.device_assign: PASS (26.56 s) (2/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_tcp_connect.snapshot_shutoff.device_assign: PASS (27.57 s) (3/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_tcp_bind.snapshot_shutoff.device_assign: PASS (28.38 s) (4/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_udp.snapshot_shutoff.device_assign: PASS (26.48 s) RESULTS : PASS 4 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

JOB LOG : /var/log/avocado/job-results/job-2024-09-03T06.31-c3bd82c/job.log (1/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_rdm.snapshot_running.device_assign: PASS (50.74 s) (2/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_tcp_connect.snapshot_running.device_assign: PASS (77.23 s) (3/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_tcp_bind.snapshot_running.device_assign: PASS (83.06 s) (4/4) type_specific.io-github-autotest-libvirt.libvirt_rng.rng_snapshot.back_udp.snapshot_running.device_assign: PASS (50.52 s) RESULTS : PASS 4 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

dzhengfy commented 2 months ago

@nanli1 Could you help review?

yanqzhan commented 2 months ago

depends on https://github.com/avocado-framework/avocado-vt/pull/3990

yanqzhan commented 2 months ago

Any other related cases result?

Done. Actually all rng snapshot cases were skipped due to internal snapshot error. I was thinking only to modify the rdm running case first. But modifying them all seems not a big difficulty now. HAHA.

yanqzhan commented 2 months ago

@nanli1 Codes updated. Could you help review again? also with the depending pr https://github.com/avocado-framework/avocado-vt/pull/3990. Thank you!