tp-qemu icon indicating copy to clipboard operation
tp-qemu copied to clipboard

qemu_guest_agent: Add new case for installing qga via installer

Open 6-dehan opened this issue 2 years ago • 5 comments

check qemu-ga.service function after installing via virtio-win-guest-installer.

ID: 2101738 Signed-off-by: demeng [email protected]

6-dehan avatar Jun 29 '22 07:06 6-dehan

@vivianQizhu could you please help to review? (1/1) Host_RHEL.m9.u1.qcow2.virtio_scsi.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.qemu_guest_agent.virtio_serial.gagent_driver_update_via_installer_tool.q35: STARTED (1/1) Host_RHEL.m9.u1.qcow2.virtio_scsi.up.virtio_net.Guest.Win2019.x86_64.io-github-autotest-qemu.qemu_guest_agent.virtio_serial.gagent_driver_update_via_installer_tool.q35: PASS (306.02 s)

6-dehan avatar Jun 29 '22 08:06 6-dehan

@leidwang Could you please help to review this patch as well? appreciate it.

6-dehan avatar Jul 11 '22 04:07 6-dehan

@6-dehan Hi, Please move from 'Draft' to 'Ready for review' if it is ready. And I actually found several different scripts has exactly same implementation on the first few steps. We have discussed with @leidwang and @menli820 that we would like to have a unique module for adding extra function test to those existed cases, sorry that not let you know earlier, I assumed that qga might be different but seems I was wrong about that, checking the existed ones obviously qga also share the same logic. Let's wait for @menli820's PR or we can discuss if you would like to take it. Thanks.

vivianQizhu avatar Jul 15 '22 03:07 vivianQizhu

@6-dehan Is this ready for review?

vivianQizhu avatar Aug 30 '22 07:08 vivianQizhu

@6-dehan Is this ready for review?

nope, I don't add the content that we talked on our last meeting. I'll work on it asap.

6-dehan avatar Aug 30 '22 12:08 6-dehan

Hi @vivianQizhu, could you please continue to help to review this? (1/1) Host_RHEL.m9.u2.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win2016.x86_64.io-github-autotest-qemu.qemu_guest_agent.virtio_serial.gagent_driver_update_via_installer_tool.q35: PASS (226.28 s)

6-dehan avatar Jan 12 '23 11:01 6-dehan