Fix qemu simple_unplug issue

avocado-framework / avocado-vt

Avocado VT Plugin

https://avocado-vt.readthedocs.org/

Other

83 stars 241 forks source link

Fix qemu simple_unplug issue #3961

Closed luckyh closed 3 weeks ago

luckyh commented 1 month ago

qdevices: remove hook_drive_bus attribute

The hook_drive_bus attribute was introduced for handling parentship between the QDevices and theirs parent nodes. However, since that time, it improperly made them seeing each other as the child, which is senseless. This patch removes the attribute and the relevant code accordingly.
qcontainer: fix simple_unplug issue

This ensure that a device would be removed from the DevContainer before its parent(s) been removed.

ID: 2408

luckyh commented 1 month ago

@YongxueHong @qingwangrh @yanan-fu would you mind to help review this PR? thanks in advance!

qingwangrh commented 1 month ago

Pass python ConfigTest.py --testcase=insert_media,multi_disk.virtio_scsi_variants.passthrough.generic.with_cache_writethrough --guestname=RHEL.9.4.0 --driveformat=virtio_blk,virtio_scsi --imageformat=qcow2,raw --firmware=default_bios --customsparams="no prepare\nvm_mem_limit = 12G\n" --clone=no

LGTM