effect of changing the output from one of the NBD
tests.
Reported-by: Thomas Huth
Signed-off-by: Daniel P. Berrangé
---
NB, wrt qemu-stable this will be for the 9.1 stable branch once
that is created, no earlier releases will need this.
tests/qemu-iotests/233.out | 12 ++--
1 file
On 27/08/2024 09.05, Markus Armbruster wrote:
Daniel P. Berrangé writes:
On Mon, Aug 12, 2024 at 05:38:41PM +0200, Thomas Huth wrote:
On 24/07/2024 11.47, Daniel P. Berrangé wrote:
The current TLS session I/O APIs just return a synthetic errno
value on error, which has been translated from
On 27/08/2024 15.16, Thomas Huth wrote:
On 23/08/2024 09.28, Philippe Mathieu-Daudé wrote:
Hi,
On 21/8/24 10:27, Thomas Huth wrote:
From: Daniel P. Berrangé
Many tests need to access assets stored on remote sites. We don't want
to download these during test execution when run by
On 26/08/2024 17.26, Peter Xu wrote:
On Fri, Aug 23, 2024 at 03:13:12PM +0900, Akihiko Odaki wrote:
A test run may create boot files several times. Delete the previous boot
file before creating a new one.
Signed-off-by: Akihiko Odaki
Reviewed-by: Michael S. Tsirkin
Acked-by: Thomas Huth
I
On 23/08/2024 09.28, Philippe Mathieu-Daudé wrote:
Hi,
On 21/8/24 10:27, Thomas Huth wrote:
From: Daniel P. Berrangé
Many tests need to access assets stored on remote sites. We don't want
to download these during test execution when run by meson, since this
risks hitting test timeouts
On 26/08/2024 19.07, Jared Rossi wrote:
Hi Thomas,
I just wanted to get your thoughts on the status of the netboot loader merge.
I see that the first patch from this series was merged, but not the others. Is
there any issue with the rest of the changes?
Hi Jared,
there's no issue with
Just had to update the asset checksum to use SHA256 instead of SHA1,
but apart from that it is a pretty much straightforward conversion.
Signed-off-by: Thomas Huth
---
MAINTAINERS | 1 +
tests/avocado/boot_linux_console.py | 24 ---
tests/functional
t as an example here).
Thomas Huth (2):
tests/functional: Add the LinuxKernelTest for testing the Linux boot
process
tests/functional: Convert the m68k Q800 Avocado test into a functional
test
MAINTAINERS | 1 +
tests/avocado/boot_linux_consol
Copy the LinuxKernelTest from tests/acceptance/boot_linux_console.py
to be able to convert the related tests to the functional test framework
in the following patches.
Signed-off-by: Thomas Huth
---
tests/functional/qemu_test/__init__.py| 1 +
tests/functional/qemu_test/linuxkernel.py | 41
On 27/08/2024 11.21, Philippe Mathieu-Daudé wrote:
Hi Thomas,
On 27/8/24 07:34, Thomas Huth wrote:
On 27/08/2024 00.10, Philippe Mathieu-Daudé wrote:
Use the 'linux_kernel' namespace to provide common helpers
to functional tests booting a Linux kernel.
Suggested-by: Thomas Huth
On 27/08/2024 11.16, Philippe Mathieu-Daudé wrote:
On 27/8/24 07:27, Thomas Huth wrote:
On 26/08/2024 22.59, Philippe Mathieu-Daudé wrote:
On 26/8/24 11:10, Thomas Huth wrote:
On 24/08/2024 18.08, Philippe Mathieu-Daudé wrote:
Straight forward conversion. Update the SHA1 hashes to
SHA256
On 27/08/2024 00.10, Philippe Mathieu-Daudé wrote:
Use the 'linux_kernel' namespace to provide common helpers
to functional tests booting a Linux kernel.
Suggested-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
---
tests/functional/qemu_test/linux_kern
On 26/08/2024 22.59, Philippe Mathieu-Daudé wrote:
On 26/8/24 11:10, Thomas Huth wrote:
On 24/08/2024 18.08, Philippe Mathieu-Daudé wrote:
Straight forward conversion. Update the SHA1 hashes to
SHA256 hashes since SHA1 should not be used anymore nowadays.
Add extract_from_deb() method in
-aarch64_virt OK 30.44s 3 subtests passed
...
Signed-off-by: Philippe Mathieu-Daudé
---
Based-on: <20240822114146.86838-1-phi...@linaro.org>
---
Reviewed-by: Thomas Huth
t-in-custom-executor
[2] https://about.gitlab.com/releases/2024/05/16/gitlab-17-0-released/
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Message-ID: <20240816213203.18350-1-phi...@linaro.org>
Signed-off-by: Thomas Huth
---
.gitlab-ci.d/cirrus/build.yml | 2 +-
1 file
From: Akihiko Odaki
A test run may create boot files several times. Delete the previous boot
file before creating a new one.
Signed-off-by: Akihiko Odaki
Reviewed-by: Michael S. Tsirkin
Message-ID: <20240823-san-v4-7-a24c6dfa4...@daynix.com>
Signed-off-by: Thomas Huth
---
tests
failure of the qtests on MSYS2, we can revert this patch
to get everything back.
Suggested-by: Philippe Mathieu-Daudé
Message-ID: <20240820170142.55324-1-th...@redhat.com>
Signed-off-by: Thomas Huth
---
.gitlab-ci.d/windows.yml | 26 ++
1 file changed, 2 insert
e previous boot file
Philippe Mathieu-Daudé (1):
gitlab-ci: Replace build_script -> step_script in Cirrus jobs
Thomas Huth (1):
.gitlab-ci.d/windows.yml: Disable the qtests in the MSYS2 job
tests/qtest/migration-test.c | 18 +++---
.gitlab-ci.d/cirrus/build.yml |
new file
in the qemu_tests folder, so that all the related tests can simply inherit
from that class? That way we would also get the KERNEL_COMMON_COMMAND_LINE
handling for free in all the tests that we convert. What do you think?
Thomas
sts passed
Signed-off-by: Philippe Mathieu-Daudé
---
Reviewed-by: Thomas Huth
you simply use our qemu_memalign() function instead?
Thomas
On 23/08/2024 14.54, Philippe Mathieu-Daudé wrote:
On 21/8/24 10:27, Thomas Huth wrote:
Provide a meson.build file for the upcoming python-based functional
tests, and add some wrapper glue targets to the tests/Makefile.include
file. We are going to use two "speed" modes for the functi
_img):
+return qemu_img
+if has_cmd('qemu-img'):
+return 'qemu-img'
+test.cancel('Could not find "qemu-img", which is required to '
+'create temporary images')
Please use self.skipTest() instead ... cancel() was something from the
Avocado framework only.
Apart from that the patch looks fine to me now, thanks!
Thomas
Anyway, you can do the precaching manually by running:
make precache-functional
HTH,
Thomas
4-thorough+thorough /
func-aarch64-aarch64_sbsaref" and timeouts without information which tests
pass, which fail.
Try running "make check-functional V=1" ... it should show you the
individual subtests, so you hopefully can conclude from the tests that
succeeded to the one that was hanging.
Thomas
user-mode emulation.
Signed-off-by: Philippe Mathieu-Daudé
---
tests/functional/qemu_test/__init__.py | 2 +-
tests/functional/qemu_test/testcase.py | 17 +
2 files changed, 18 insertions(+), 1 deletion(-)
Reviewed-by: Thomas Huth
+--
1 file changed, 1 insertion(+), 18 deletions(-)
Reviewed-by: Thomas Huth
(cmd)
+run_cmd([qemu_img, 'create', '-f', 'qcow2', image_path, '8M'])
... so this seems wrong to me?
tests/avocado/avocado_qemu/__init__.py used to have a get_qemu_img()
function, maybe you could port that one over?
Thomas
%)
mode change 100644 => 100755
Thanks for your help with the conversions!
Reviewed-by: Thomas Huth
rcp.py} | 56 ++-
3 files changed, 33 insertions(+), 26 deletions(-)
rename tests/{avocado/machine_arm_integratorcp.py =>
functional/test_arm_integratorcp.py} (63%)
mode change 100644 => 100755
Reviewed-by: Thomas Huth
/load_bflt.py:LoadBFLT.test_stm32: PASS (0.09 s)
RESULTS: PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 |
CANCEL 0
JOB TIME : 0.62 s
Reported-by: Thomas Huth
Signed-off-by: Philippe Mathieu-Daudé
---
tests/avocado/load_bflt.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion
, 54 insertions(+), 55 deletions(-)
delete mode 100644 tests/avocado/machine_mips_malta.py
create mode 100755 tests/functional/test_mipsel_malta.py
Reviewed-by: Thomas Huth
from avocado.utils import ssh
+from avocado.utils import process, ssh
from avocado.utils.path import find_command
Reviewed-by: Thomas Huth
... but is the test itself still working for you? When I run it, I get:
...
(074/281)
tests/avocado/linux_ssh_mips_malta.py:LinuxSSH.test_mips_malta
On 21/08/2024 17.11, Philippe Mathieu-Daudé wrote:
On 19/7/24 11:50, Thomas Huth wrote:
Only some few tests are using the LinuxTest class. Move the related
code into a separate file so that this does not pollute the main
namespace.
Signed-off-by: Thomas Huth
---
tests/avocado/avocado_qemu
ips64el
-:avocado: tags=machine:malta
-:avocado: tags=cpu:I6400
-:avocado: tags=mips:smp
-:avocado: tags=flaky
-"""
-self.do_test_i6400_framebuffer_logo(8)
-
class MaltaMachine(QemuSystemTest):
def do_test_yamon(self):
diff --git a/tests/functional/meson.build b/tests/functional/meson.build
index 8d28313a65..1823f88897 100644
--- a/tests/functional/meson.build
+++ b/tests/functional/meson.build
@@ -56,6 +56,7 @@ tests_mips64el_quick = [
]
tests_mips64el_thorough = [
+ 'mips64el_malta',
'mips64el_loongson3v',
]
Maybe keep the list in alphabetical order?
Anyway:
Reviewed-by: Thomas Huth
~^
TypeError: can only concatenate str (not "NoneType") to str
Since we don't really need precaching in this case, maybe you could move the
asset definition into the function itself, or do this test without the asset
framework?
Thoma
On 21/08/2024 11.43, Philippe Mathieu-Daudé wrote:
On 21/8/24 10:27, Thomas Huth wrote:
A straight forward conversion, we just also have to remove the decorator
@skipUnless(os.getenv('SPEED')) since all non-trivial functional tests
are running in SPEED=thorough mode now. Also make sur
On 21/08/2024 11.31, Philippe Mathieu-Daudé wrote:
On 21/8/24 10:27, Thomas Huth wrote:
Without this change, the new Avocado v103 fails to find the tests
that are based on the LinuxTest class.
Reorder before previous patch?
The funny thing is that Avocado v88 breaks the other way round if
Use the Python standard zipfile module instead of avocado.utils for
extracting the ZIP file that we download here, and use the standard
subprocess module for running the "tail" command.
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: T
ere should not really be a problem).
The wheel file has been obtained with:
pip download --only-binary :all: --dest . --no-cache pycotap
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
python/wheels/pycotap-1.3.1-py3-none-any.whl | Bin 0 -> 5119 bytes
py
The next patch is going to add some entries that need more space between
the command and the help text, so let's increase the indentation here
first.
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Reviewed-by: Alex Bennée
Signed-off-by: Thomas Huth
---
Without this change, the new Avocado v103 fails to find the tests
that are based on the LinuxTest class.
Suggested-by: Cleber Rosa
Signed-off-by: Thomas Huth
---
tests/avocado/avocado_qemu/linuxtest.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/avocado
Document the new functional testing framework. The text is originally
based on the Avocado documentation, but heavily modified to match the
new framework.
Signed-off-by: Thomas Huth
---
docs/devel/testing/functional.rst | 269 ++
docs/devel/testing/index.rst
and there are stale read-only files already around.
Signed-off-by: Thomas Huth
---
MAINTAINERS | 2 +-
tests/functional/meson.build | 2 +
.../test_ppc64_hv.py} | 53 ---
3 files changed, 25 inserti
Now that we converted many tests from the "check-avocado" test suite
to the "check-functional" test suite, we should make sure that these
also get tested in the CI.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
.g
Mostly a straight-forward conversion. Looks like we can simply drop
the avocado datadrainer stuff when not using the avocado framework
anymore.
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS
will be set via meson.build later.
Signed-off-by: Daniel P. Berrangé
Signed-off-by: Thomas Huth
---
MAINTAINERS| 5 +
tests/functional/qemu_test/__init__.py | 13 ++
tests/functional/qemu_test/cmd.py | 178 +
tests/functional/qemu_test
Nothing thrilling in here, it's just a straight forward conversion.
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 1 +
tests/functional/meson.build
The main testing documentation file got very overloaded already.
Thus let's split the Avocado information into a separate file.
Signed-off-by: Thomas Huth
---
docs/devel/testing/avocado.rst | 581 +
docs/devel/testing/index.rst | 1 +
docs/devel/te
The avocado_qemu.Test class has been renamed a while back in commit
2283b627bc ("tests/avocado: Rename avocado_qemu.Test -> QemuSystemTest"),
so we should reflect this now in the documentation, too.
Signed-off-by: Thomas Huth
---
docs/devel/testing/avo
While we're at it, update the SHA1 hashes to SHA256 hashes since
SHA1 should not be used anymore nowadays.
Signed-off-by: Thomas Huth
---
tests/functional/meson.build | 1 +
.../test_linux_initrd.py} | 37 +++
2 files changed, 22 inser
Nothing thrilling in here, just straight forward conversions.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 9 +--
tests/functional/meson.build | 11
.../test_ppc64_powernv.py}| 56
ymore
nowadays.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 4 +-
tests/functional/meson.build | 6 ++
tests/functional/qemu_test/utils.py | 14 +++
.../test_s390x_ccw_virtio.py}
Provide a "gzip_uncompress" function based on the standard "gzip" module
to avoid the usage of avocado.utils here.
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 2
Mathieu-Daudé
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS| 1 +
tests/avocado/riscv_opensbi.py | 63 --
tests/functional/meson.build | 8
tests/functional/test_riscv_opensbi.py | 36
ed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
tests/Makefile.include | 11 ++
tests/functional/meson.build | 66
tests/meson.build| 1 +
3 files changed, 78 insertions(+)
create mode 10064
with this functionality, so let's just add a nice wrapper
function around that.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Tested-by: Daniel P. Berrangé
Signed-off-by: Thomas Huth
---
tests/functional/qemu_test/utils.py | 21 ++
ed-by: Daniel P. Berrangé
Tested-by: Philippe Mathieu-Daudé
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 3 +
tests/functional/meson.build | 6 ++
.../test_info_usernet.py} | 11
em
directly.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 5 ++
tests/functional/meson.build | 5 ++
.../test_cpu_queries.py}
Nothing thrilling in here, it's just a straight forward conversion.
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
tests/functional/meson.build | 1 +
.../test_virtio_gpu.py} | 64 +--
2 files changed, 30 inser
thieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
MAINTAINERS | 1 +
tests/avocado/tesseract_utils.py | 46 ---
tests/functional/meson.build | 4 ++
tests/functional/qemu_test/tessera
Signed-off-by: Thomas Huth
---
MAINTAINERS | 2 +-
tests/avocado/machine_microblaze.py | 61 ---
tests/functional/meson.build | 8 +++
.../functional/test_microblaze_s3adsp1800.py | 40
Building and testing are two separate topics, so let's split the testing
into a separate category and move the related files into a separate folder.
Signed-off-by: Thomas Huth
---
docs/devel/index-build.rst | 14 --
docs/devel/inde
by: Marcin Juszkiewicz
Message-ID: <20240806173119.582857-7-cr...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/avocado/boot_xen.py | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/tests/avocado/boot_xen.py b/tests/avocado/boot_xen.py
index f29bc58b9e..4
Instead of using the "archive" module from avocado.utils, switch
these tests to use the new wrapper function that is based on the
"tarfile" module instead.
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Philippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
Signed
FC
Cleber Rosa (4):
tests/avocado: machine aarch64: standardize location and RO access
tests/avocado/boot_xen.py: fetch kernel during test setUp()
tests/avocado/machine_aarch64_sbsaref.py: allow for rw usage of image
Bump avocado to 103.0
Daniel P. Berrangé (2):
tests/functional: add
ewicz
Message-ID: <20240806173119.582857-9-cr...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/avocado/machine_aarch64_sbsaref.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/avocado/machine_aarch64_sbsaref.py
b/tests/avocado/machine_aarch64_sbsaref.py
index
Create log files for each test separately, one file that contains
the basic logging and one that contains the console output.
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Thomas Huth
---
tests/functional/qemu_test/testcase.py | 27 +-
1 file changed, 26 insertions
://avocado-framework.readthedocs.io/en/103.0/releases/lts/103_0.html
Signed-off-by: Cleber Rosa
Tested-by: Marcin Juszkiewicz
Message-ID: <20240806173119.582857-2-cr...@redhat.com>
Signed-off-by: Thomas Huth
---
pythondeps.toml| 2 +-
tests/Makefile.include | 2 +-
2 files changed, 2 inse
Now that we've got the Asset class with pre-caching, we can convert
some Avocado tests that use fetch_asset() for downloading their
required files.
Signed-off-by: Thomas Huth
---
MAINTAINERS | 12 ++---
tests/avocado/machine_mips_loongson3v.py
hilippe Mathieu-Daudé
Tested-by: Philippe Mathieu-Daudé
[thuth: Drop sha1 support, use hash on file content for naming instead of URL,
add the possibility to specify the cache dir via environment variable]
Signed-off-by: Thomas Huth
---
tests/functional/qemu_test/__init__.py | 1 +
t
ys done by
make check-functional
Signed-off-by: Daniel P. Berrangé
Tested-by: Richard Henderson
[thuth: Remove the duplicated "path = os.path.basename(...)" line]
Signed-off-by: Thomas Huth
---
tests/Makefile.include | 3 ++-
tests/fu
y: Cleber Rosa
Tested-by: Marcin Juszkiewicz
Message-ID: <20240806173119.582857-5-cr...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/avocado/machine_aarch64_sbsaref.py | 2 +-
tests/avocado/machine_aarch64_virt.py| 14 +++---
2 files changed, 8 insertions(+), 8 deletions
py.
Signed-off-by: Thomas Huth
---
tests/avocado/boot_linux_console.py | 20
tests/avocado/machine_s390_ccw_virtio.py | 2 ++
2 files changed, 2 insertions(+), 20 deletions(-)
diff --git a/tests/avocado/boot_linux_console.py
b/tests/avocado/boot_linux_console.py
i
failure of the qtests on MSYS2, we can revert this patch
to get everything back.
Suggested-by: Philippe Mathieu-Daudé
Signed-off-by: Thomas Huth
---
This decreases the runtime of the job from ~ 70 minutes to
~ 45 minutes for me.
.gitlab-ci.d/windows.yml | 26 ++
1
On 20/08/2024 10.03, Thomas Huth wrote:
On 20/08/2024 00.31, Philippe Mathieu-Daudé wrote:
On 16/8/24 09:22, Thomas Huth wrote:
From: Cleber Rosa
Commit 9b45cc993 added many cases of skipUnless for the sake of
organizing flaky tests. But, Python decorators *must* follow what
they decorate
On 20/08/2024 00.31, Philippe Mathieu-Daudé wrote:
On 16/8/24 09:22, Thomas Huth wrote:
From: Cleber Rosa
Commit 9b45cc993 added many cases of skipUnless for the sake of
organizing flaky tests. But, Python decorators *must* follow what
they decorate, so the newlines added should *not* exist
free to reach out if any questions arise.
Signed-off-by: Beraldo Leal
---
MAINTAINERS | 3 ---
1 file changed, 3 deletions(-)
Thank you very, very much for your past reviews, Beraldo!
I'll queue the patch along with the other testing patches that I'm collecting.
Thomas
-Daudé wrote:
Signed-off-by: Philippe Mathieu-Daudé
---
I don't know how to use Powershell do use nproc+1 jobs
to optimize jobs waiting on I/O.
Well, we're calling into bash there, so you could do it as part of the bash
statement?
Thomas
---
.gitlab-ci.d/windows.yml | 3 ++-
1 fi
On 19/08/2024 12.21, Philippe Mathieu-Daudé wrote:
On 19/8/24 07:30, Thomas Huth wrote:
On 16/08/2024 19.18, Philippe Mathieu-Daudé wrote:
On 16/8/24 18:40, Thomas Huth wrote:
On 16/08/2024 18.34, Philippe Mathieu-Daudé wrote:
On 16/8/24 17:37, Thomas Huth wrote:
The qtests are broken since
--hard "$CI_COMMIT_SHA"
- build_script:
+ step_script:
- mkdir build
- cd build
- ../configure --enable-werror $CONFIGURE_ARGS
Reviewed-by: Thomas Huth
On 16/08/2024 19.18, Philippe Mathieu-Daudé wrote:
On 16/8/24 18:40, Thomas Huth wrote:
On 16/08/2024 18.34, Philippe Mathieu-Daudé wrote:
On 16/8/24 17:37, Thomas Huth wrote:
The qtests are broken since a while in the MSYS2 job in the gitlab-CI,
likely due to some changes in the MSYS2
On 16/08/2024 18.34, Philippe Mathieu-Daudé wrote:
On 16/8/24 17:37, Thomas Huth wrote:
The qtests are broken since a while in the MSYS2 job in the gitlab-CI,
likely due to some changes in the MSYS2 environment. So far nobody has
neither a clue what's going wrong here, nor an idea how t
On 16/08/2024 18.21, Alex Bennée wrote:
Thomas Huth writes:
The qtests are broken since a while in the MSYS2 job in the gitlab-CI,
likely due to some changes in the MSYS2 environment. So far nobody has
neither a clue what's going wrong here, nor an idea how to fix this
(in fact most
ilable for properly analyzing this problem), so let's disable the
qtests here again to get at least the test coverage for the compilation
and unit tests back to the CI.
Signed-off-by: Thomas Huth
---
.gitlab-ci.d/windows.yml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/.gitlab-ci.d/window
')
+self.vm.launch()
+query_block_result = self.vm.qmp('query-block')['return']
+self.assertEqual(len(query_block_result), 1)
+self.assertTrue(query_block_result[0]["inserted"]["ro"])
I think such a test would be a better fit for the qemu-iotests, e.g.
tests/qemu-iotests/118 looks pretty similar already, maybe this could be
added there instead?
Thomas
n?
Stefan's patch has:
From: Stefan Weil via
Please re-spin.
Should I just drop Stefan's doc tweaks?
No, IIRC, you can fix the commit by doing
git commit --amend --reset-author="..."
HTH,
Thomas
On 16/08/2024 10.27, Akihiko Odaki wrote:
On 2024/08/16 17:24, Thomas Huth wrote:
On 16/08/2024 10.21, Akihiko Odaki wrote:
On 2024/08/16 17:03, Thomas Huth wrote:
On 16/08/2024 09.30, Akihiko Odaki wrote:
On 2024/08/16 16:27, Thomas Huth wrote:
On 16/08/2024 09.12, Akihiko Odaki wrote:
On
On 16/08/2024 09.45, Philippe Mathieu-Daudé wrote:
On 16/8/24 09:22, Thomas Huth wrote:
From: Pierrick Bouvier
Found on debian stable.
../target/s390x/tcg/translate.c: In function ‘get_mem_index’:
../target/s390x/tcg/translate.c:398:1: error: control reaches end of
non-void function
On 16/08/2024 10.21, Akihiko Odaki wrote:
On 2024/08/16 17:03, Thomas Huth wrote:
On 16/08/2024 09.30, Akihiko Odaki wrote:
On 2024/08/16 16:27, Thomas Huth wrote:
On 16/08/2024 09.12, Akihiko Odaki wrote:
On 2024/08/16 16:03, Thomas Huth wrote:
On 16/08/2024 08.22, Akihiko Odaki wrote
On 16/08/2024 09.30, Akihiko Odaki wrote:
On 2024/08/16 16:27, Thomas Huth wrote:
On 16/08/2024 09.12, Akihiko Odaki wrote:
On 2024/08/16 16:03, Thomas Huth wrote:
On 16/08/2024 08.22, Akihiko Odaki wrote:
Commit 23ef50ae2d0c (".gitlab-ci.d/buildtest.yml: Use
-fno-sanitize=function i
On 16/08/2024 09.12, Akihiko Odaki wrote:
On 2024/08/16 16:03, Thomas Huth wrote:
On 16/08/2024 08.22, Akihiko Odaki wrote:
Commit 23ef50ae2d0c (".gitlab-ci.d/buildtest.yml: Use
-fno-sanitize=function in the clang-system job") adds
-fno-sanitize=function for the CI but doesn't
ge-ID: <20240815122719.727639-1-th...@redhat.com>
Tested-by: Stefan Weil
Signed-off-by: Thomas Huth
---
meson.build | 5 +
1 file changed, 5 insertions(+)
diff --git a/meson.build b/meson.build
index 81ecd4bae7..fbda17c987 100644
--- a/meson.build
+++ b/meson.build
@@ -315,6 +315,11 @@ e
: Thomas Huth
Tested-by: Marcin Juszkiewicz
Message-ID: <20240806173119.582857-3-cr...@redhat.com>
Signed-off-by: Thomas Huth
---
tests/avocado/boot_linux_console.py | 1 -
tests/avocado/intel_iommu.py| 1 -
tests/avocado/linux_initrd.py | 1 -
tests/avocado/machine_aspeed.py
From: Daniel P. Berrangé
Refresh with the newly added gtk-vnc package
Signed-off-by: Daniel P. Berrangé
Message-ID: <20240718094159.902024-3-berra...@redhat.com>
[thuth: fixed conflicts in .gitlab-ci.d/cirrus/*.vars]
Signed-off-by: Thomas Huth
---
.gitlab-ci.d/cirrus/freebsd-1
Message-ID: <20240814224132.897098-4-pierrick.bouv...@linaro.org>
Signed-off-by: Thomas Huth
---
target/s390x/tcg/translate.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/target/s390x/tcg/translate.c b/target/s390x/tcg/translate.c
index c81e035dea..bcfff40b25 100644
--- a/target/s39
: refresh package lists with lcitool
Pierrick Bouvier (1):
target/s390x: fix build warning (gcc-12 -fsanitize=thread)
Thomas Huth (1):
meson.build: Check for the availability of __attribute__((gcc_struct)) on
MSYS2
meson.build | 5
From: Daniel P. Berrangé
The gtk-vnc package is used by the vnc-display-test qtest
program. Technically only gvnc is needed, but since we
already pull in the gtk3 dep, it is harmless to depend
on gtk-vnc.
Signed-off-by: Daniel P. Berrangé
Reviewed-by: Thomas Huth
Message-ID
ed from
.gitlab-ci.d/buildtest.yml because -fno-sanitize=function in meson.build
does not affect --extra-cflags due to argument ordering.
Signed-off-by: Akihiko Odaki
---
Changes in v3:
- I was not properly dropping the change of .gitlab-ci.d/buildtest.yml
but only updated the message. v3 fixe
s well,
and use --enable-sanitizers. I think it should be ok to leave
-fno-sanitize-recover=undefined in extra.
That could be a solution, too, but I think the change in meson.build should
then also be moved to the spot where -fsanitize=undefined is added.
Thomas
401 - 500 of 3018 matches
Mail list logo