All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cleber Rosa <crosa@redhat.com>
To: "Philippe Mathieu-Daudé" <philmd@redhat.com>
Cc: peter.maydell@linaro.org, "Thomas Huth" <thuth@redhat.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	Pavel.Dovgaluk@ispras.ru, "Markus Armbruster" <armbru@redhat.com>,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
	qemu-devel@nongnu.org, b.galvani@gmail.com,
	"Niek Linnenbank" <nieklinnenbank@gmail.com>,
	qemu-arm@nongnu.org, "Stefan Hajnoczi" <stefanha@redhat.com>,
	"Willian Rampazzo" <wrampazz@redhat.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	f4bug@amsat.org
Subject: Re: [PATCH 0/2] Allwinner H3 fixes for EMAC and acceptance tests
Date: Fri, 19 Feb 2021 13:58:55 -0500	[thread overview]
Message-ID: <20210219185855.GA619233@localhost.localdomain> (raw)
In-Reply-To: <cc5f65a8-204b-17ad-3bba-61a3919ef798@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1380 bytes --]

On Fri, Feb 19, 2021 at 07:24:01PM +0100, Philippe Mathieu-Daudé wrote:
> 
> I hope you understand the concern I have is not with you in particular,
> and I used your case to start a discussion with the QEMU community.
> 
> FWIW I missed the URL change because I still have the image cached in
> Avocado so my testing ran fine. Which makes me wonder...
> 
> Cleber, Willian, should Avocado display information about cached
> artifacts? Such "Using artifact downloaded 7 months ago".
>

As of Avocado 85.0 (currently used in QEMU), it's possible to set the
"expire" parameter to "fetch_asset", see:

  https://avocado-framework.readthedocs.io/en/85.0/api/test/avocado.html#avocado.Test.fetch_asset

In this case, if we want assets to not be used if they're are 30 days
or older, that could be set to 86400.  The expired asset not being used,
and then not being able to be fetched again, would cause a test to be
canceled.

Cache browsing/listing/manipulation using the "avocado assets" command
is planned for Avocado 86.0, see:

  https://github.com/avocado-framework/avocado/issues/4311

> > So what I can do
> > instead is:
> > 
> >   - update the patch to use github to store the artifacts, and their
> > licenses (other tests also use github)
> 
> Until there is better solutions, this is the option I prefer.
>

+1.

Regards,
- Cleber.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2021-02-19 19:00 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-11 22:00 [PATCH 0/2] Allwinner H3 fixes for EMAC and acceptance tests Niek Linnenbank
2021-02-11 22:00 ` [PATCH 1/2] tests/acceptance: replace unstable apt.armbian.com URLs for orangepi-pc, cubieboard Niek Linnenbank
2021-02-12  1:15   ` Cleber Rosa
2021-02-12 20:19   ` Willian Rampazzo
2021-02-11 22:00 ` [PATCH 2/2] hw/net/allwinner-sun8i-emac: traverse transmit queue using TX_CUR_DESC register value Niek Linnenbank
2021-02-11 23:29   ` Philippe Mathieu-Daudé
2021-02-12 14:10 ` [PATCH 0/2] Allwinner H3 fixes for EMAC and acceptance tests Philippe Mathieu-Daudé
2021-02-15 20:30   ` Niek Linnenbank
2021-02-16  9:48   ` Daniel P. Berrangé
2021-02-17 20:57     ` Niek Linnenbank
2021-02-19 18:24       ` Philippe Mathieu-Daudé
2021-02-19 18:58         ` Cleber Rosa [this message]
2021-02-21 20:40         ` Niek Linnenbank

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=20210219185855.GA619233@localhost.localdomain \
    --to=crosa@redhat.com \
    --cc=Pavel.Dovgaluk@ispras.ru \
    --cc=alex.bennee@linaro.org \
    --cc=armbru@redhat.com \
    --cc=b.galvani@gmail.com \
    --cc=berrange@redhat.com \
    --cc=f4bug@amsat.org \
    --cc=nieklinnenbank@gmail.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@redhat.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=thuth@redhat.com \
    --cc=wainersm@redhat.com \
    --cc=wrampazz@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.