Mario Casquero

Hello @nickzhq, hotplug_mem_negative.invalid_device_addr test case failed with current patch ``` (1/1) Host_RHEL.m10.u0.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.hotplug_mem_negative.invalid_device_addr.q35: STARTED (1/1) Host_RHEL.m10.u0.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.hotplug_mem_negative.invalid_device_addr.q35: FAIL: No valid keywords were found in the qemu prompt message (117.08 s) RESULTS :...

Using the latest code of the patch, tests cases passed **RHEL 9.6 VM** ``` (1/4) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.hotplug_virtio_mem.q35: STARTED (1/4) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.hotplug_virtio_mem.q35: PASS (240.76 s) (2/4) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.hotplug_memory.after.guest_reboot.hotplug.backend_file.policy_default.two.q35: STARTED (2/4) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.hotplug_memory.after.guest_reboot.hotplug.backend_file.policy_default.two.q35: PASS (203.58 s)...

Hello @misanjumn are you sending a new patch for fixing the generated issues?

To discuss here which of the added `# noqa` could be avoided

@happz one question, in the [related issue ](https://github.com/teemtee/tmt/issues/2877)with the patch it's used `tmt -a provision -h local` which IIUC it's the local host already executing tmt, in this case the...

> What about this approach: The `pid` directory is created only if it does not exist with the correct permissions. If the directory is present we would suppose that it...

> This one fails because of incorrect pidfile root in `tmt-reboot` and `tmt-reboot-core`. Pidfile root was not exposed, so they were using the old one. Can we apply the following...

> It does not look applied, I'm looking at diff in this PR and I see no changes I proposed. My fault :confused: :disappointed_relieved: included now

> > ``` > # stderr (1 lines) > # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > Error: crun: executable file `/bin/bash` not found in $PATH: No such file or directory: OCI runtime attempted to...

> This one's easy: as an unprivileged user, you cannot create `/var/tmp/tmt/pid` directory: Correct me if I'm wrong, but adding sudo to the tmt commands would be fine in this...