On Fri, Jan 8, 2021 at 3:59 PM Thomas Huth <th...@redhat.com> wrote: > > There was a race condition in the first test where there was already the > "crw" output in the dmesg, but the "0.0.4711" entry has not been created > in the /sys fs yet. Fix it by waiting until it is there. > > The second test has even more problems on gitlab-CI. Even after adding some > more synchronization points (that wait for some messages in the "dmesg" > output to make sure that the modules got loaded correctly), there are still > occasionally some hangs in this test when it is running in the gitlab-CI. > So far I was unable to reproduce these hangs locally on my computer, so > this issue might take a while to debug. Thus disable the 2nd test in the > gitlab-CI until the problems are better understood and fixed. > > Signed-off-by: Thomas Huth <th...@redhat.com> > --- > tests/acceptance/machine_s390_ccw_virtio.py | 14 ++++++++++++-- > 1 file changed, 12 insertions(+), 2 deletions(-) >
Reviewed-by: Willian Rampazzo <willi...@redhat.com> Tested-by: Willian Rampazzo <willi...@redhat.com> JOB ID : 6b2b3c1f6f6b0c4c2e9fd694b475bd12c193adbd JOB LOG : /home/linux1/src/qemu.dev/build/tests/results/job-2021-01-08T16.24-6b2b3c1/job.log (1/2) tests/acceptance/machine_s390_ccw_virtio.py:S390CCWVirtioMachine.test_s390x_devices: PASS (8.78 s) (2/2) tests/acceptance/machine_s390_ccw_virtio.py:S390CCWVirtioMachine.test_s390x_fedora: PASS (23.86 s) RESULTS : PASS 2 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0 JOB TIME : 33.02 s
