Alex Bennée writes:
> Cleber Rosa writes:
>
>> The tests under machine_aarch64_virt.py do not need read-write access
>> to the ISOs. The ones under machine_aarch64_sbsaref.py, on the other
>> hand, will need read-write access, so let's give each test an unique
Marcin Juszkiewicz writes:
> W dniu 8.12.2023 o 20:09, Cleber Rosa pisze:
>> The tests under machine_aarch64_virt.py do not need read-write access
>> to the ISOs. The ones under machine_aarch64_sbsaref.py, on the other
>> hand, will need read-write access, so let's giv
Alex Bennée writes:
> Cleber Rosa writes:
>
>> Based on many runs, the average run time for these 4 tests is around
>> 250 seconds, with 320 seconds being the ceiling. In any way, the
>> default 120 seconds timeout is inappropriate in my experience.
>
> I would
Akihiko Odaki writes:
> On 2023/12/09 4:09, Cleber Rosa wrote:
>> Avocado's fetchasset plugin runs before the actual Avocado job (and
>> any test). It analyses the test's code looking for occurrences of
>> "self.fetch_asset()" in the either the actual test or s
a huge development window), the actual
version bump will be posted, along with more profound changes to the
tests to leverage the new features.
Cleber Rosa (10):
tests/avocado: mips: fallback to HTTP given certificate expiration
tests/avocado: mips: add hint for fetchasset plugin
tests/avocado
tten, make the fetchasset plugin blind to
the assets. This adds redundant code, true, but one that doesn't hurt
the test and aids the fetchasset plugin to download or verify the
existence of these assets in advance.
Signed-off-by: Cleber Rosa
---
tests/avocado/boot_linux_console.py | 15 +
, avoiding causing cache misses (because the assets get
modified and thus need to be dowloaded again) while avoid copying the
root filesystem backing file whenever possible.
This also allow these tests to be run in parallel with newer Avocado
versions.
Signed-off-by: Cleber Rosa
---
tests/avocado
that the one in the cache is
preserved and the cache sees a hit on re-runs.
Signed-off-by: Cleber Rosa
---
tests/avocado/boot_linux_console.py | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/avocado/boot_linux_console.py
b/tests/avocado/boot_linux_console.py
index
Because all tests share the same tags, it's possible to have all of
them at the class level.
Signed-off-by: Cleber Rosa
---
tests/avocado/boot_xen.py | 26 +-
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/tests/avocado/boot_xen.py b/tests/avocado
this project has no control over that certificate and host,
this falls back to plain HTTP instead. The integrity of the
downloaded files can be guaranteed by the existing hashes for those
files (which are not modified here).
Signed-off-by: Cleber Rosa
---
tests/avocado/boot_linux_console.py | 6
Rather than defining a single use variable, let's just use the class
attribute directly.
Signed-off-by: Cleber Rosa
---
tests/avocado/boot_xen.py | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tests/avocado/boot_xen.py b/tests/avocado/boot_xen.py
index f4b63c1ef2
Based on many runs, the average run time for these 4 tests is around
250 seconds, with 320 seconds being the ceiling. In any way, the
default 120 seconds timeout is inappropriate in my experience.
Let's increase the timeout so these tests get a chance to completion.
Signed-off-by: Cleber Rosa
While it's a good practice to have reusable base classes, in this
specific case there's no other user of the BootXenBase class.
By unifying the class used in this test, we can improve readability
and have the opportunity to add some future improvements in a clearer
fashion.
Signed-off-by: Cleber
-off-by: Cleber Rosa
---
tests/avocado/machine_aarch64_sbsaref.py | 9 +++--
tests/avocado/machine_aarch64_virt.py| 14 +++---
2 files changed, 14 insertions(+), 9 deletions(-)
diff --git a/tests/avocado/machine_aarch64_sbsaref.py
b/tests/avocado/machine_aarch64_sbsaref.py
index
hes, which makes future Avocado runs to attempt to
redownload the assets with the same name, but from the different
locations they actually are from. This causes cache misses, extra
downloads, and possibly canceled tests.
Signed-off-by: Cleber Rosa
---
tests/avocado/kvm_xen_guest.py | 3 ++-
tes
On Wed, Jul 26, 2023 at 5:07 AM Thomas Huth wrote:
>
> On 26/07/2023 09.33, Philippe Mathieu-Daudé wrote:
> > On 25/7/23 19:13, Peter Maydell wrote:
> >> Currently this CI job is failing:
> >>
> >> https://gitlab.com/qemu-project/qemu/-/jobs/4737819946
> >>
> >> because:
> >>
> >> (05/59)
> >>
On Wed, Jul 26, 2023 at 3:34 AM Philippe Mathieu-Daudé
wrote:
>
> On 25/7/23 19:13, Peter Maydell wrote:
> > Currently this CI job is failing:
> >
> > https://gitlab.com/qemu-project/qemu/-/jobs/4737819946
> >
> > because:
> >
> > (05/59)
> >
On Thu, Jul 27, 2023 at 11:50 AM Peter Maydell wrote:
>
> Ah, so the problem is that we are trying to download the asset
> file, and the remote server is stalling so it doesn't actually
> download the file in 90s, and Avocado doesn't distinguish
> "hit the timeout while trying to download assets"
On Tue, Jul 25, 2023 at 1:13 PM Peter Maydell wrote:
>
> Currently this CI job is failing:
>
> https://gitlab.com/qemu-project/qemu/-/jobs/4737819946
>
> because:
>
> (05/59)
> tests/avocado/boot_linux_console.py:BootLinuxConsole.test_arm_exynos4210_initrd:
> INTERRUPTED: Missing asset
>
On 4/18/23 03:10, Thomas Huth wrote:
On 17/04/2023 15.43, Alex Bennée wrote:
From: Kautuk Consul
Avocado version 101.0 has a fix to re-compute the checksum
of an asset file if the algorithm used in the *-CHECKSUM
file isn't the same as the one being passed to it by the
avocado user (i.e.
On 4/20/23 12:54, John Snow wrote:
On Thu, Apr 20, 2023, 5:19 AM Philippe Mathieu-Daudé
wrote:
On 17/4/23 19:44, Alex Bennée wrote:
>
> John Snow writes:
>
>> On Mon, Apr 17, 2023 at 9:43 AM Alex Bennée
wrote:
>>>
>>> From: Kautuk Consul
>>>
>>>
On 4/20/23 05:18, Philippe Mathieu-Daudé wrote:
On 17/4/23 19:44, Alex Bennée wrote:
John Snow writes:
On Mon, Apr 17, 2023 at 9:43 AM Alex Bennée
wrote:
From: Kautuk Consul
Avocado version 101.0 has a fix to re-compute the checksum
of an asset file if the algorithm used in the
On 4/17/23 12:50, John Snow wrote:
On Mon, Apr 17, 2023 at 9:43 AM Alex Bennée wrote:
From: Kautuk Consul
Avocado version 101.0 has a fix to re-compute the checksum
of an asset file if the algorithm used in the *-CHECKSUM
file isn't the same as the one being passed to it by the
avocado
On 4/17/23 09:43, Alex Bennée wrote:
From: Kautuk Consul
Avocado version 101.0 has a fix to re-compute the checksum
of an asset file if the algorithm used in the *-CHECKSUM
file isn't the same as the one being passed to it by the
avocado user (i.e. the avocado_qemu python module).
In the
On 3/27/23 07:50, Kautuk Consul wrote:
Avocado version 101.0 has a fix to re-compute the checksum
of an asset file if the algorithm used in the *-CHECKSUM
file isn't the same as the one being passed to it by the
avocado user (i.e. the avocado_qemu python module).
In the earlier avocado
ion from the current authors of this
> tool to loosen the license. At present, those people are:
>
> - John Snow (me!), 411/609
> - Luiz Capitulino, Author, 97/609
> - Daniel Berrangé, 81/609
> - Eduardo Habkost, 10/609
> - Marc-André Lureau, 6/609
> - Fam Zheng, 3/609
> -
t, then we save this information so we can
> later configure the guest.
>
> Suggested-by: Daniel P. Berrangé
> Signed-off-by: Beraldo Leal
> ---
> tests/avocado/avocado_qemu/__init__.py | 13 +++--
> 1 file changed, 7 insertions(+), 6 deletions(-)
Reviewed-by: Cleber Rosa
Tested-by: Cleber Rosa
Beraldo Leal writes:
> On Fri, Mar 11, 2022 at 09:28:24AM -0500, Cleber Rosa wrote:
>>
>> Beraldo Leal writes:
>>
>> > Race conditions can happen with the current code, because the port that
>> > was available might not be anymore by the time the
Beraldo Leal writes:
> Race conditions can happen with the current code, because the port that
> was available might not be anymore by the time the server is started.
>
> By setting the port to 0, PhoneServer it will use the OS default
> behavior to get a free port, then we save this
>
> tests/avocado/avocado_qemu/__init__.py | 2 ++
> 1 file changed, 2 insertions(+)
>
LGTM, despite the root issue is being addressed in Avocado.
Reviewed-by: Cleber Rosa
Philippe Mathieu-Daudé writes:
> On 25/2/22 22:01, Cleber Rosa wrote:
>> Being explicit about the accelerator used on these tests is a good
>> thing in itself, but it will also be used in the filtering rules
>> applied on "make check-avocado".
>> Signed-o
possible, if deemed valuable, to further evolve this
classification into one with a clear separation between mechanism and
content.
Signed-off-by: Cleber Rosa
---
tests/avocado/boot_linux.py | 4 ++
tests/avocado/boot_linux_console.py | 54 +++
tests/
ts or tags to be
used on a "make check-avocado" by setting AVOCADO_TESTS or
AVOCADO_TAGS.
Signed-off-by: Cleber Rosa
---
tests/Makefile.include | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tests/Makefile.include b/tests/Makefile.include
index 676aa0d94
t timeouts or similar
reasons when run under systems with limited or shared resources. To
avoid breakages, especially on CI, let's keep the serial execution
until proven that it won't impact the CI jobs.
Signed-off-by: Cleber Rosa
---
tests/Makefile.include | 1 +
tests/requirements.txt | 2 +-
2 fil
minutes.
[1] https://lists.gnu.org/archive/html/qemu-devel/2021-07/msg07271.html
[2] https://gitlab.com/cleber.gnu/qemu/-/pipelines/479720240
[3] https://gitlab.com/qemu-project/qemu/-/pipelines/478580581
Cleber Rosa (9):
Avocado GitLab CI jobs: don't reset TARGETS and simplify commands
Avocad
If the test is skipped because of their conditionals, the shared_dir
attribute may not exist.
Check for its existence in the tearDown() method to avoid and
AttributeError.
Signed-off-by: Cleber Rosa
---
tests/avocado/virtiofs_submounts.py | 7 ---
1 file changed, 4 insertions(+), 3
t/releases/94_0.html#utility-apis
Signed-off-by: Cleber Rosa
---
tests/avocado/avocado_qemu/__init__.py | 5 +++--
tests/avocado/migration.py | 4 ++--
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/tests/avocado/avocado_qemu/__init__.py
b/tests/avocado/avocado_qemu/_
It's possible to filter based on a combination of criteria. This adds
examples to the documentation.
Signed-off-by: Cleber Rosa
---
docs/devel/testing.rst | 22 ++
1 file changed, 22 insertions(+)
diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index
Being explicit about the accelerator used on these tests is a good
thing in itself, but it will also be used in the filtering rules
applied on "make check-avocado".
Signed-off-by: Cleber Rosa
---
tests/avocado/linux_ssh_mips_malta.py | 3 +++
1 file changed, 3 insertions(+)
diff --g
quot; as a namespace prefix
for everything that Avocado should collect/preserve, and other
prefixes for logged content that should be handled differently.
[1] -
https://avocado-framework.readthedocs.io/en/94.0/releases/92_0.html#users-test-writers
Signed-off-by: Cleber Rosa
---
tests/avocado/a
to be run on each job, and less noise on the test
results.
Signed-off-by: Cleber Rosa
---
.gitlab-ci.d/buildtest-template.yml | 3 +++
.gitlab-ci.d/buildtest.yml | 9 -
2 files changed, 3 insertions(+), 9 deletions(-)
diff --git a/.gitlab-ci.d/buildtest-template.yml
b/.gitlab
On Tue, Feb 1, 2022 at 1:06 PM Alex Bennée wrote:
>
>
> Cleber Rosa writes:
>
> > On Tue, Feb 1, 2022 at 11:20 AM Daniel P. Berrangé
> > wrote:
> >>
> >> On Tue, Feb 01, 2022 at 11:01:43AM -0500, Cleber Rosa wrote:
> >> > On
On Tue, Feb 1, 2022 at 12:01 PM Daniel P. Berrangé wrote:
>
> On Tue, Feb 01, 2022 at 12:29:56AM -0500, Cleber Rosa wrote:
> >
> > Assuming this is about "Testing that QEMU can boot a full distro", I
> > wouldn't
> > try to solve the problem by making the
On Tue, Feb 1, 2022 at 11:20 AM Daniel P. Berrangé wrote:
>
> On Tue, Feb 01, 2022 at 11:01:43AM -0500, Cleber Rosa wrote:
> > On Tue, Feb 1, 2022 at 6:25 AM Alex Bennée wrote:
> > >
> > > We have up to now tried really hard as a project to avoid building and
On Tue, Feb 1, 2022 at 6:07 AM Kashyap Chamarthy wrote:
>
> On Tue, Jan 25, 2022 at 10:20:11AM +0100, Gerd Hoffmann wrote:
> > Hi,
> >
> > > IMHO the ideal scenario would be for us to have a kernel, initrd
> > > containing just busybox tools for the key arch targets we care
> > > about. Those
On Tue, Feb 1, 2022 at 6:25 AM Alex Bennée wrote:
>
> We have up to now tried really hard as a project to avoid building and
> hosting our own binaries to avoid theoretical* GPL compliance issues.
> This is why we've ended up relying so much on distros to build and host
> binaries we can use.
On Fri, Jan 21, 2022 at 10:22 AM Daniel P. Berrangé wrote:
>
> On Fri, Jan 21, 2022 at 12:23:23PM +, Alex Bennée wrote:
> >
> > Peter Maydell writes:
> >
> > > On Fri, 21 Jan 2022 at 10:50, Markus Armbruster wrote:
> > >> No objection, but it's no replacement for looking into why these
python/Makefile | 2 ++
> python/setup.cfg | 1 +
> 2 files changed, 3 insertions(+)
>
Reviewed-by: Cleber Rosa
Tested-by: Cleber Rosa
Hi,
Fosshost.org was kind enough to supply the QEMU project with a public
VM hosted by them. The original use case we anticipated was to set up
a GitLab CI runner, because we assumed the VM was KVM capable, but
that turned out not to be the case.
So, at this point, adding it as a GitLab CI
t with the given configuration (point II) under the
given OS build environment (point I) should be subjected to.
IV) Job definition: GitLab CI jobs that will dispatch the build/test
jobs (see points #II and #III) to the machine specifically
configured according to #I.
Signed-off-by:
c8s-stream-rhel branch) from CentOS repo at
https://git.centos.org/rpms/qemu-kvm/.
* Further separated distribution version and architecture specific
files into separate sub directories.
* Added a gitlab CI rule and variable to allow other repos/users who
have a CentOS Stream 8 x86_64 ru
The NetBSD-7.1.2-prep.iso is no longer available on the CDN, but it's
still available in the archive.
Let's update its location so that users without the file on cache can
still fetch it and run the test.
Signed-off-by: Cleber Rosa
---
tests/acceptance/ppc_prep_40p.py | 4 ++--
1 file changed
The arch and machine tags apply to all tests, so let's define them
only once.
Signed-off-by: Cleber Rosa
---
tests/acceptance/ppc_prep_40p.py | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/tests/acceptance/ppc_prep_40p.py b/tests/acceptance/ppc_prep_40p.py
Rather than defining a single use variable, let's just use the class
attribute directly.
Signed-off-by: Cleber Rosa
---
tests/acceptance/boot_xen.py | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tests/acceptance/boot_xen.py b/tests/acceptance/boot_xen.py
index 66621fd14d
ts, and do not have known issues) can be obtained
with:
make list-acceptance
As a future improvement, a tool can be added to sync the "issue" tags
with issues that end up being resolved closed.
Signed-off-by: Cleber Rosa
---
docs/devel/testing.rst | 28 +
Just a clean up for an unused import.
Signed-off-by: Cleber Rosa
---
tests/acceptance/boot_xen.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/acceptance/boot_xen.py b/tests/acceptance/boot_xen.py
index cd916ddba4..66621fd14d 100644
--- a/tests/acceptance/boot_xen.py
+++ b/tests
The "check-acceptance" make rule won't necessarily run *all* available
tests, because it employs a filter based on the currently configured
targets. This change in the description of the rule makes that
behavior extra clear.
Signed-off-by: Cleber Rosa
---
tests/Makefile.include | 2
Just a removal of an unused imported symbol.
Signed-off-by: Cleber Rosa
---
tests/acceptance/ppc_prep_40p.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/acceptance/ppc_prep_40p.py b/tests/acceptance/ppc_prep_40p.py
index 6b28a69ea5..5e61e686bd 100644
--- a/tests/acceptance
The kernel is a common blob used in all tests. By moving it to the
setUp() method, the "fetch asset" plugin will recognize the kernel and
attempt to fetch it and cache it before the tests are started.
Signed-off-by: Cleber Rosa
---
tests/acceptance/boot_xen.py | 13 ++-
Because all tests share the same tags, it's possible to have all of
them at the class level.
Signed-off-by: Cleber Rosa
---
tests/acceptance/boot_xen.py | 26 +-
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/tests/acceptance/boot_xen.py b/tests
-by: Cleber Rosa
---
tests/acceptance/boot_linux_console.py | 6 +++---
tests/acceptance/replay_kernel.py | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/acceptance/boot_linux_console.py
b/tests/acceptance/boot_linux_console.py
index 80e3a2f7a6..bc0caa1099 100644
While it's a good practice to have reusable base classes, in this
specific case there's no other user of the BootXenBase class.
By unifying the class used in this test, we can improve readability
and have the opportunity to add some future improvements in a clearer
fashion.
Signed-off-by: Cleber
It is helpful to know the tests that would be executed with a "make
check-acceptance" without executing them. Let's introduce a "make
list-acceptance" rule for that purpose.
Signed-off-by: Cleber Rosa
---
tests/Makefile.include | 8
1 file changed, 8 insertions(+)
Signed-off-by: Cleber Rosa
---
tests/acceptance/boot_linux_console.py | 84 ++
tests/acceptance/boot_xen.py | 6 +-
tests/acceptance/replay_kernel.py | 12 ++--
tests/acceptance/tcg_plugins.py| 2 +-
4 files changed, 41 insertions(+), 63 deletions
ementation to be used by
those that choose to do so.
More information can be found at the release notes at:
https://avocado-framework.readthedocs.io/en/latest/releases/91_0.html
Signed-off-by: Cleber Rosa
---
docs/devel/testing.rst | 12
tests/Makefile.include | 2 ++
tests/requirements.tx
This matches the command line on 82a17d1d67, where the "on" or "off"
should be explicitly given.
Signed-off-by: Cleber Rosa
---
tests/acceptance/vnc.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/acceptance/vnc.py b/tests/acceptance/vnc
proposal is a convention to tag tests that either
have known issues, or that touch on QEMU features that have known
issues. By tagging those tests accordingly, they will be
automatically excluded from the regular execution with "make
check-acceptance".
Finally, some updates to as
ll
Signed-off-by: Cleber Rosa
---
tests/acceptance/avocado_qemu/__init__.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/acceptance/avocado_qemu/__init__.py
b/tests/acceptance/avocado_qemu/__init__.py
index 2c4fef3e14..1e807e2e55 100644
--- a/tests/acceptance/avocado_qemu/__init__.py
+
unintentionally by a third party.
Signed-off-by: Cleber Rosa
---
MAINTAINERS | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 37b1a8e442..d35b948e8d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3418,11 +3418,11 @@ F: tests/tcg/Makefile
F: tests
This is a reply to an issue[1] reported by Peter, and while at it, an
update of the MAINTAINERS entry so other people now the status and
where to go for help with regards to the acceptance tests'
infrastructure.
[1] https://lists.gnu.org/archive/html/qemu-devel/2021-08/msg00349.html
Cleber Rosa
ll
Signed-off-by: Cleber Rosa
---
tests/acceptance/avocado_qemu/__init__.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/acceptance/avocado_qemu/__init__.py
b/tests/acceptance/avocado_qemu/__init__.py
index 2c4fef3e14..1e807e2e55 100644
--- a/tests/acceptance/avocado_qemu/__init__.py
+
On Tue, Aug 3, 2021 at 9:47 AM Peter Maydell wrote:
>
> On Tue, 3 Aug 2021 at 13:58, Cleber Rosa wrote:
> >
> > On Tue, Aug 3, 2021 at 8:43 AM Peter Maydell
> > wrote:
> > >
> > > It looks like 'make check-acceptance' creates directories in
>
On Tue, Aug 3, 2021 at 8:43 AM Peter Maydell wrote:
>
> It looks like 'make check-acceptance' creates directories in
> build/clang/tests/results which are huge and which it never
> cleans up. For example one of my build directories (configured
> just for arm targets) has over 350
On Sat, Jul 31, 2021 at 4:33 PM Peter Maydell wrote:
>
> On Sat, 31 Jul 2021 at 19:43, Alex Bennée wrote:
> >
> >
> > Peter Maydell writes:
> >
> > > "make check-acceptance" takes way way too long. I just did a run
> > > on an arm-and-aarch64-targets-only debug build and it took over
> > > half
On Sat, Jul 31, 2021 at 2:40 AM Thomas Huth wrote:
>
> On 31/07/2021 00.04, Cleber Rosa wrote:
> > On Fri, Jul 30, 2021 at 11:43 AM Peter Maydell
> > wrote:
> >>
> >> On Fri, 30 Jul 2021 at 16:12, Peter Maydell
> >> wrote:
> >>>
> &g
On Fri, Jul 30, 2021 at 11:43 AM Peter Maydell wrote:
>
> On Fri, 30 Jul 2021 at 16:12, Peter Maydell wrote:
> >
> > "make check-acceptance" takes way way too long. I just did a run
> > on an arm-and-aarch64-targets-only debug build and it took over
> > half an hour, and this despite it skipping
On Tue, Jul 27, 2021 at 9:48 AM Peter Maydell wrote:
>
> On Tue, 27 Jul 2021 at 14:24, Cleber Rosa wrote:
> > Yes, I've spent quite some time with some flaky behavior while running
> > the replay tests as well. But in the end, the test remained unchanged
> > bec
On Tue, Jul 27, 2021 at 5:17 AM Peter Maydell wrote:
>
> On Tue, 27 Jul 2021 at 01:39, Cleber Rosa wrote:
> > tests/acceptance/replay_kernel.py:ReplayKernelNormal.test_aarch64_virt
> > is currently failing consistently (first found that in [1]).
>
> FWIW I find that on my
On Tue, Jul 27, 2021 at 3:37 AM Peter Maydell wrote:
>
> On Tue, 27 Jul 2021 at 01:39, Cleber Rosa wrote:
> >
> >
> > Hi everyone,
> >
> > tests/acceptance/replay_kernel.py:ReplayKernelNormal.test_aarch64_virt
> > is currently failing consistently (first
ab.com/qemu-project/qemu/-/jobs/1445513133#L268
--
Cleber Rosa
[ Sr Software Engineer - Virtualization Team - Red Hat ]
[ Avocado Test Framework - avocado-framework.github.io ]
[ 7ABB 96EB 8B46 B94D 5E0F E9BB 657E 8D33 A5F2 09F3 ]
wangyanan (Y) writes:
> Hi Cleber,
>
> On 2021/7/23 6:25, Cleber Rosa wrote:
>> Yanan Wang writes:
>>
>>> In the SMP configuration, we should either specify a topology
>>> parameter with a reasonable value (equal to or greater than 1)
>>> or j
Yanan Wang writes:
> In the SMP configuration, we should either specify a topology
> parameter with a reasonable value (equal to or greater than 1)
> or just leave it omitted and QEMU will calculate its value.
> Configurations which explicitly specify the topology parameters
> as zero like
; ---
> v2: when 'always' -> 'on_success' & reworded (danpb)
>
> Supersedes: <20210720164829.3949558-1-phi...@redhat.com>
> ---
> .gitlab-ci.d/opensbi.yml | 30 ++
> 1 file changed, 18 insertions(+), 12 deletions(-)
>
Reviewed-by: Cleber Rosa
m/generic-loader.rst | 4 ++--
> 3 files changed, 13 insertions(+), 10 deletions(-)
Reviewed-by: Cleber Rosa
ell
> ---
> docs/interop/qemu-ga-ref.rst | 9 -
> docs/interop/qemu-qmp-ref.rst| 9 -
> docs/interop/qemu-storage-daemon-qmp-ref.rst | 9 -
> 3 files changed, 27 deletions(-)
>
Reviewed-by: Cleber Rosa
Both tests use the same kernel command line arguments, so there's no
need to have a common and then an additional set of arguments.
Signed-off-by: Cleber Rosa
Message-Id: <20210714174051.28164-5-cr...@redhat.com>
Reviewed-by: Willian Rampazzo
Signed-off-by: Cleber Rosa
---
tests/acce
Like previously done with the arch tags, all tests use the same CPU
value so it's possible to combine them at the class level.
Signed-off-by: Cleber Rosa
Message-Id: <20210714174051.28164-4-cr...@redhat.com>
Reviewed-by: Willian Rampazzo
Signed-off-by: Cleber Rosa
---
tests/acceptance/
Since efe30d501 there's a shorthand for requiring specific
accelerators, and canceling the test if it's not available.
Signed-off-by: Cleber Rosa
Message-Id: <20210714174051.28164-2-cr...@redhat.com>
Reviewed-by: Willian Rampazzo
Signed-off-by: Cleber Rosa
---
tests/acceptance/virtio-
The test class in question is x86_64 specific, so it's possible to set
the tags at the class level.
Signed-off-by: Cleber Rosa
Message-Id: <20210714174051.28164-3-cr...@redhat.com>
Reviewed-by: Willian Rampazzo
Signed-off-by: Cleber Rosa
---
tests/acceptance/virtio-gpu.py | 3 +--
Since 49afbca3b, the use of an optional virgl renderer is not
available anymore, and since b36eb8860f, the way to choose a GL based
rendered is to use the "virtio-vga-gl" device.
Signed-off-by: Cleber Rosa
Message-Id: <20210714174051.28164-6-cr...@redhat.com>
Reviewed-by:
588-1-yang.zh...@intel.com>
Signed-off-by: Cleber Rosa
---
hw/remote/memory.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/remote/memory.c b/hw/remote/memory.c
index 472ed2a272..6e21ab1a45 100644
--- a/hw/remote/memory.c
+++ b/hw/remote/memory.c
@@ -46,7 +46,7 @@ void remote
By providing kernel and initrd hashes, the test guarantees the
integrity of the images used and avoids the warnings set by
fetch_asset() when hashes are lacking.
Signed-off-by: Cleber Rosa
Message-Id: <20210714174051.28164-7-cr...@redhat.com>
Reviewed-by: Willian Rampazzo
Signed-off-by:
.
Cleber Rosa (6):
tests/acceptance/virtio-gpu.py: use require_accelerator()
tests/acceptance/virtio-gpu.py: combine x86_64 arch tags
tests/acceptance/virtio-gpu.py: combine CPU tags
tests/acceptance/virtio-gpu.py: combine kernel command line
tests/acceptance/virtio-gpu.py: use
Jag Raman writes:
>
> Hi Cleber,
>
> We presently don’t have permissions to send a PR to
> upstream (Peter Maydell).
>
> Presently, we are requesting someone else who has
> permissions to do PRs on our behalf. We will work
> on getting permissions to send PRs going forward.
>
> Thank you!
Hi
orry for missing that patch on the ml.
Maintainers (Elena, Jagannathan, John),
Are you planning a PR with this patch?
Thanks,
--
Cleber Rosa
[ Sr Software Engineer - Virtualization Team - Red Hat ]
[ Avocado Test Framework - avocado-framework.github.io ]
[ 7ABB 96EB 8B46 B94D 5E0F E9BB 657E 8D33 A5F2 09F3 ]
Hi everyone,
The tests/acceptance/multiprocess.py:Multiprocess.test_multiprocess_x86_64
is currently failing (as of a9649a719a44894b81f38dc1c5c1888ee684acef).
Unfortunately CI was unable to catch this issue earlier, because tests
that require KVM are not yet running (but this should change soon).
s: # 'artifacts.zip' will contains the following files:
> - pc-bios/edk2*bz2
Hi Phil,
This looks good, but have you triggered a pipeline with these changes?
It'd be helpful for an extra level of assurance for the intended results.
Thanks!
--
Cleber Rosa
[ Sr Software Engineer - Virtualization Team - Red Hat ]
[ Avocado Test Framework - avocado-framework.github.io ]
[ 7ABB 96EB 8B46 B94D 5E0F E9BB 657E 8D33 A5F2 09F3 ]
Both tests use the same kernel command line arguments, so there's no
need to have a common and then an additional set of arguments.
Signed-off-by: Cleber Rosa
---
tests/acceptance/virtio-gpu.py | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/tests/acceptance
By providing kernel and initrd hashes, the test guarantees the
integrity of the images used and avoids the warnings set by
fetch_asset() when hashes are lacking.
Signed-off-by: Cleber Rosa
---
tests/acceptance/virtio-gpu.py | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff
Since 49afbca3b, the use of an optional virgl renderer is not
available anymore, and since b36eb8860f, the way to choose a GL based
rendered is to use the "virtio-vga-gl" device.
Signed-off-by: Cleber Rosa
---
tests/acceptance/virtio-gpu.py | 4 ++--
1 file changed, 2 insertions(+), 2
1 - 100 of 1612 matches
Mail list logo