action #111602
Updated by okurz over 2 years ago
## Observation okurz recently changed 18-qemu-options.t so the second run in the `qemu_append_option` subtest appends `-M ? -version`. Previously it appended `-M ?`. The test assumes that qemu will always print only the version information (not supported machines) when run this way, but that doesn't seem safe. In Fedora's build environment, we get both the version information and the supported machines: 2: # [2022-05-25T19:04:44.856378Z] [debug] starting: /usr/bin/qemu-system-i386 -chardev ringbuf,id=serial0,logfile=serial0,logappend=on -serial chardev:serial0 -audiodev none,id=snd0 -device intel-hda -device hda-output,audiodev=snd0 -global isa-fdc.fdtypeA=none -m 1024 -netdev user,id=qanet0 -device virtio-net,netdev=qanet0,mac=52:54:00:12:34:56 -object rng-random,filename=/dev/urandom,id=rng0 -device virtio-rng-pci,rng=rng0 -boot once=d -device qemu-xhci -device usb-tablet -smp 1 -no-shutdown -vnc :90,share=force-shared -device virtio-serial -chardev pipe,id=virtio_console,path=virtio_console,logfile=virtio_console.log,logappend=on -device virtconsole,chardev=virtio_console,name=org.openqa.console.virtio_console -chardev pipe,id=virtio_console1,path=virtio_console1,logfile=virtio_console1.log,logappend=on -device virtconsole,chardev=virtio_console1,name=org.openqa.console.virtio_console1 -chardev socket,path=qmp_socket,server=on,wait=off,id=qmp_socket,logfile=qmp_socket.log,logappend=on -qmp chardev:qmp_socket -S -M ? -version -device virtio-scsi-pci,id=scsi0 -blockdev driver=file,node-name=hd0-file,filename=/tmp/18-qemu-options.t-dNVG/pool/raid/hd0,cache.no-flush=on -blockdev driver=qcow2,node-name=hd0,file=hd0-file,cache.no-flush=on -device virtio-blk,id=hd0-device,drive=hd0,serial=hd0 2: # [2022-05-25T19:04:44.862180Z] [debug] Waiting for 0 attempts 2: # perl: warning: Setting locale failed. 2: # perl: warning: Please check that your locale settings: 2: # LANGUAGE = (unset), 2: # LC_ALL = "en_US.UTF-8", 2: # LANG = "en_US.UTF-8" 2: # are supported and installed on your system. 2: # perl: warning: Falling back to the standard locale ("C"). 2: # [2022-05-25T19:04:45.032982Z] [debug] Waiting for 1 attempts 2: # [2022-05-25T19:04:45.033289Z] [info] ::: backend::baseclass::die_handler: Backend process died, backend errors are reported below in the following lines: 2: # QEMU terminated before QMP connection could be established. Check for errors below 2: # [2022-05-25T19:04:45.033642Z] [info] ::: OpenQA::Qemu::Proc::save_state: Saving QEMU state to qemu_state.json 2: # [2022-05-25T19:04:45.035028Z] [debug] Passing remaining frames to the video encoder 2: # [2022-05-25T19:04:45.087372Z] [debug] Waiting for video encoder to finalize the video 2: # [2022-05-25T19:04:45.087556Z] [debug] The built-in video encoder (pid 960824) terminated 2: # [2022-05-25T19:04:45.088978Z] [debug] QEMU: QEMU emulator version 6.2.0 (qemu-6.2.0-10.fc36) 2: # [2022-05-25T19:04:45.089061Z] [debug] QEMU: Copyright (c) 2003-2021 Fabrice Bellard and the QEMU Project developers 2: # [2022-05-25T19:04:45.089115Z] [debug] QEMU: Supported machines are: 2: # [2022-05-25T19:04:45.089170Z] [debug] QEMU: microvm microvm (i386) … 2: # [2022-05-25T19:04:45.089227Z] [debug] QEMU: xenfv-4.2 Xen Fully-virtualized PC 2: # [2022-05-25T19:04:45.089284Z] [debug] QEMU: xenfv Xen Fully-virtualized PC (alias of xenfv-3.1) 2: # [2022-05-25T19:04:45.089339Z] [debug] QEMU: xenfv-3.1 Xen Fully-virtualized PC 2: # [2022-05-25T19:04:45.089395Z] [debug] QEMU: pc Standard PC (i440FX + PIIX, 1996) (alias of pc-i440fx-6.2) 2: # [2022-05-25T19:04:45.089450Z] [debug] QEMU: pc-i440fx-6.2 Standard PC (i440FX + PIIX, 1996) (default) 2: # [2022-05-25T19:04:45.089503Z] [debug] QEMU: pc-i440fx-6.1 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089557Z] [debug] QEMU: pc-i440fx-6.0 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089610Z] [debug] QEMU: pc-i440fx-5.2 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089706Z] [debug] QEMU: pc-i440fx-5.1 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089761Z] [debug] QEMU: pc-i440fx-5.0 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089813Z] [debug] QEMU: pc-i440fx-4.2 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089866Z] [debug] QEMU: pc-i440fx-4.1 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089919Z] [debug] QEMU: pc-i440fx-4.0 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.089971Z] [debug] QEMU: pc-i440fx-3.1 Standard PC (i440FX + PIIX, 1996) 2: # [2022-05-25T19:04:45.090022Z] [debug] QEMU: pc-i440fx-3.0 2: # [2022-05-25T19:04:45.091199Z] [debug] sending magic and exit 2: # [2022-05-25T19:04:45.091579Z] [debug] received magic close 2: # [2022-05-25T19:04:45.097114Z] [debug] backend process exited: 0 2: # failed to start VM at /builddir/build/BUILD/os-autoinst-548335fc1544beeb51e86fe83e025a0f0aa4901e/t/../backend/driver.pm line 107. 2: # [2022-05-25T19:04:45.197976Z] [debug] stopping command server 960811 because test execution ended through exception 2: # [2022-05-25T19:04:45.398914Z] [debug] done with command server 2: # [2022-05-25T19:04:45.399034Z] [debug] stopping autotest process 960814 2: # [2022-05-25T19:04:45.599884Z] [debug] done with autotest process 2: # 960807: EXIT 1 2: # ' 2: # matches '(?^u:Supported machines are\:)' 2: # Looks like you failed 1 test of 14. On my local system, I get just the supported machines, no version information: [adamw@xps13k os-autoinst (rawhide *%)]$ qemu-system-i386 -M ? -version Supported machines are: microvm microvm (i386) xenfv-4.2 Xen Fully-virtualized PC xenfv Xen Fully-virtualized PC (alias of xenfv-3.1) xenfv-3.1 Xen Fully-virtualized PC pc Standard PC (i440FX + PIIX, 1996) (alias of pc-i440fx-7.0) pc-i440fx-7.0 Standard PC (i440FX + PIIX, 1996) (default) … pc-i440fx-6.2 Standard PC (i440FX + PIIX, 1996) pc-i440fx-6.1 Standard PC (i440FX + PIIX, 1996) pc-i440fx-6.0 Standard PC (i440FX + PIIX, 1996) pc-i440fx-5.2 Standard PC (i440FX + PIIX, 1996) pc-i440fx-5.1 Standard PC (i440FX + PIIX, 1996) pc-i440fx-5.0 Standard PC (i440FX + PIIX, 1996) pc-i440fx-4.2 Standard PC (i440FX + PIIX, 1996) pc-i440fx-4.1 Standard PC (i440FX + PIIX, 1996) pc-i440fx-4.0 Standard PC (i440FX + PIIX, 1996) pc-i440fx-3.1 Standard PC (i440FX + PIIX, 1996) pc-i440fx-3.0 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.9 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.8 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.7 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.6 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.5 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.4 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.3 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.2 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.12 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.11 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.10 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.1 Standard PC (i440FX + PIIX, 1996) pc-i440fx-2.0 Standard PC (i440FX + PIIX, 1996) pc-i440fx-1.7 Standard PC (i440FX + PIIX, 1996) (deprecated) pc-i440fx-1.6 Standard PC (i440FX + PIIX, 1996) (deprecated) pc-i440fx-1.5 Standard PC (i440FX + PIIX, 1996) (deprecated) pc-i440fx-1.4 Standard PC (i440FX + PIIX, 1996) (deprecated) q35 Standard PC (Q35 + ICH9, 2009) (alias of pc-q35-7.0) pc-q35-7.0 Standard PC (Q35 + ICH9, 2009) pc-q35-6.2 Standard PC (Q35 + ICH9, 2009) pc-q35-6.1 Standard PC (Q35 + ICH9, 2009) pc-q35-6.0 Standard PC (Q35 + ICH9, 2009) pc-q35-5.2 Standard PC (Q35 + ICH9, 2009) pc-q35-5.1 Standard PC (Q35 + ICH9, 2009) pc-q35-5.0 Standard PC (Q35 + ICH9, 2009) pc-q35-4.2 Standard PC (Q35 + ICH9, 2009) pc-q35-4.1 Standard PC (Q35 + ICH9, 2009) pc-q35-4.0.1 Standard PC (Q35 + ICH9, 2009) pc-q35-4.0 Standard PC (Q35 + ICH9, 2009) pc-q35-3.1 Standard PC (Q35 + ICH9, 2009) pc-q35-3.0 Standard PC (Q35 + ICH9, 2009) pc-q35-2.9 Standard PC (Q35 + ICH9, 2009) pc-q35-2.8 Standard PC (Q35 + ICH9, 2009) pc-q35-2.7 Standard PC (Q35 + ICH9, 2009) pc-q35-2.6 Standard PC (Q35 + ICH9, 2009) pc-q35-2.5 Standard PC (Q35 + ICH9, 2009) pc-q35-2.4 Standard PC (Q35 + ICH9, 2009) pc-q35-2.12 Standard PC (Q35 + ICH9, 2009) pc-q35-2.11 Standard PC (Q35 + ICH9, 2009) pc-q35-2.10 Standard PC (Q35 + ICH9, 2009) isapc ISA-only PC none empty machine x-remote Experimental remote machine xenpv Xen Para-virtualized PC [adamw@xps13k os-autoinst (rawhide *%)]$ I'm not really sure what the point of this test is, anyway. Why are we testing qemu behaviour here? Shouldn't we only care whether the parameters got appended properly? ## Acceptance criteria * **AC1:** A conscious decision was made if we want to do such test at all or not ## Suggestions * Read the question from AdamWilliamson why we test this at all, understand why the according test was included, make sure that we only do test what makes sense feature and coverage wise