All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Huth <thuth@redhat.com>
To: "Ani Sinha" <ani@anisinha.ca>, "Alex Bennée" <alex.bennee@linaro.org>
Cc: "Cleber Rosa" <crosa@redhat.com>,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
	"Beraldo Leal" <bleal@redhat.com>,
	"Qemu Devel" <qemu-devel@nongnu.org>,
	"Daniel P . Berrangé" <berrange@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Maydell Peter" <peter.maydell@linaro.org>,
	"John Snow" <jsnow@redhat.com>,
	"Igor Mammedov" <imammedo@redhat.com>,
	"Michael Tsirkin" <mst@redhat.com>
Subject: Re: [PATCH v4 00/10] Introduce new acpi/smbios avocado tests using biosbits
Date: Wed, 19 Oct 2022 08:22:07 +0200	[thread overview]
Message-ID: <b8ceab73-bca8-e2c0-daac-e72d0e4e48d2@redhat.com> (raw)
In-Reply-To: <CAARzgww2sDAYonM61N1OYvGeWCQRrr6NKY1YezpeJXiej_gNHQ@mail.gmail.com>

On 18/10/2022 17.34, Ani Sinha wrote:
> On Tue, Oct 18, 2022 at 9:01 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>>
>>
>> Ani Sinha <ani@anisinha.ca> writes:
>>
>>> Please see the README file in patch 9 for more details.
>>> Sample runs are as follows:
>>>
>>> Passed tests:
>>>
>>> $ ./tests/venv/bin/avocado run -t acpi tests/avocado
>>> Fetching asset from tests/avocado/acpi-bits.py:AcpiBitsTest.test_acpi_smbios_bits
>>> JOB ID     : e95c7413e996bfb59389839e5ca5105464ef098f
>>> JOB LOG    : /home/anisinha/avocado/job-results/job-2022-10-14T19.15-e95c741/job.log
>>>   (1/1) tests/avocado/acpi-bits.py:AcpiBitsTest.test_acpi_smbios_bits: PASS (33.08 s)
>>> RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
>>> JOB TIME   : 39.18 s
>>
>> Hmm something went wrong:
>>
>>    ➜  ./tests/venv/bin/avocado run -t acpi tests/avocado
>>    Fetching asset from tests/avocado/acpi-bits.py:AcpiBitsTest.test_acpi_smbios_bits
>>    JOB ID     : 08b6d26bf52a22ccbe1908b14d5b7b27f76ad7fa
>>    JOB LOG    : /home/alex/avocado/job-results/job-2022-10-18T16.29-08b6d26/job.log
>>     (1/1) tests/avocado/acpi-bits.py:AcpiBitsTest.test_acpi_smbios_bits: ERROR: Error while generating the bits iso. Pass V=1 in the environment to get more details. Command '[
>>    '/var/tmp/acpi-bits-yy4mfxrz.tmp/grub-inst-x86_64-efi/bin/grub-mkrescue', '-o', '/var/tmp/acpi-bits-yy4mfxrz.tmp/bits-2020.iso', '/var/tmp/acpi-bits-yy4mfxrz.t... (7.21 s)
>>    RESULTS    : PASS 0 | ERROR 1 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
>>    JOB TIME   : 7.79 s
>>
>> have we added another binary dependency?
> 
> Try with V=1 , Maybe you do not have xorriso? If that is the case, you
> need to install it in your test environment.

If a tool is missing, the test should be SKIP and not ERROR, I think?

  Thomas




  reply	other threads:[~2022-10-19  6:27 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-14 17:34 [PATCH v4 00/10] Introduce new acpi/smbios avocado tests using biosbits Ani Sinha
2022-10-14 17:34 ` [PATCH v4 01/10] acpi/tests/avocado/bits: initial commit of test scripts that are run by biosbits Ani Sinha
2022-10-17 21:13   ` Philippe Mathieu-Daudé
2022-10-17 22:06     ` Michael S. Tsirkin
2022-10-18  3:18     ` Ani Sinha
2022-10-14 17:35 ` [PATCH v4 02/10] acpi/tests/avocado/bits: add SPDX license identifiers for bios bits tests Ani Sinha
2022-10-14 17:35 ` [PATCH v4 03/10] acpi/tests/avocado/bits: disable acpi PSS tests that are failing in biosbits Ani Sinha
2022-10-14 17:35 ` [PATCH v4 04/10] acpi/tests/avocado/bits: add smilatency test suite from bits in order to disable it Ani Sinha
2022-10-14 17:35 ` [PATCH v4 05/10] acpi/tests/avocado/bits: add SPDX license identifiers for bios bits smilatency tests Ani Sinha
2022-10-14 17:35 ` [PATCH v4 06/10] acpi/tests/avocado/bits: disable smilatency test since it does not pass everytime Ani Sinha
2022-10-14 17:35 ` [PATCH v4 07/10] acpi/tests/avocado/bits: add biosbits config file for running bios tests Ani Sinha
2022-10-14 17:35 ` [PATCH v4 08/10] acpi/tests/avocado/bits: add acpi and smbios avocado tests that uses biosbits Ani Sinha
2022-10-18 15:37   ` Alex Bennée
2022-10-18 15:43     ` Ani Sinha
2022-10-14 17:35 ` [PATCH v4 09/10] acpi/tests/avocado/bits: add a README file to describe the test Ani Sinha
2022-10-17  9:39   ` Ani Sinha
2022-10-14 17:35 ` [PATCH v4 10/10] MAINTAINERS: add myself as the maintainer for acpi biosbits avocado tests Ani Sinha
2022-10-18 15:36   ` Alex Bennée
2022-10-18 15:30 ` [PATCH v4 00/10] Introduce new acpi/smbios avocado tests using biosbits Alex Bennée
2022-10-18 15:34   ` Ani Sinha
2022-10-19  6:22     ` Thomas Huth [this message]
2022-10-19  6:26       ` Ani Sinha

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b8ceab73-bca8-e2c0-daac-e72d0e4e48d2@redhat.com \
    --to=thuth@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=ani@anisinha.ca \
    --cc=berrange@redhat.com \
    --cc=bleal@redhat.com \
    --cc=crosa@redhat.com \
    --cc=f4bug@amsat.org \
    --cc=imammedo@redhat.com \
    --cc=jsnow@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=wainersm@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.