qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 00/11] Introduce new acpi/smbios python tests using biosbi


From: Daniel P . Berrangé
Subject: Re: [PATCH v2 00/11] Introduce new acpi/smbios python tests using biosbits
Date: Tue, 6 Sep 2022 14:30:31 +0100
User-agent: Mutt/2.2.6 (2022-06-05)

On Tue, Sep 06, 2022 at 06:58:02PM +0530, Ani Sinha wrote:
> On Tue, Sep 6, 2022 at 18:45 Daniel P. Berrangé <berrange@redhat.com> wrote:
> 
> > On Thu, Jul 14, 2022 at 02:24:18PM +0100, Peter Maydell wrote:
> > > On Mon, 11 Jul 2022 at 10:34, Michael S. Tsirkin <mst@redhat.com> wrote:
> > > >
> > > > On Sun, Jul 10, 2022 at 10:30:03PM +0530, Ani Sinha wrote:
> > > > > Changelog:
> > > > > v2:
> > > > >  - a new class of python based tests introduced that is separate
> > from avocado
> > > > >    tests or qtests. Can be run by using "make check-pytest".
> > > > >  - acpi biosbits tests are the first tests to use pytest environment.
> > > > >  - bios bits tests now download the bits binary archives from a
> > remote
> > > > >    repository if they are not found locally. The test skips if
> > download
> > > > >    fails.
> > > > >  - A new environment variable is introduced that can be passed by
> > the tester
> > > > >    to specify the location of the bits archives locally. test skips
> > if the
> > > > >    bits binaries are not found in that location.
> > > > >  - if pip install of python module fails for whatever reaoson, the
> > test skips.
> > > > >  - misc code fixes including spell check of the README doc. README
> > has been
> > > > >    updated as well.
> > > > >  - addition of SPDX license headers to bits test files.
> > > > >  - update MAINTAINERS to reflect the new pytest test class.
> > > > >
> > > > > For biosbits repo:
> > > > >  - added Dockerfile and build script. Made bios bits build on gcc 11.
> > > > >
> > https://github.com/ani-sinha/bits/blob/bits-qemu-logging/Dockerfile
> > > > >
> > https://github.com/ani-sinha/bits/blob/bits-qemu-logging/build-artifacts.sh
> > > > >    The build script generates the zip archive and tarball used by
> > the test.
> > > >
> > > > So far so good, I think it's ok for a start. It's probably a good idea
> > > > to host the source on qemu.org. Peter - any objection to this?
> > >
> > > Dan was looking at v1 from the point of view of how we handle the
> > > guest binary blobs for these tests -- I'd rather defer to him rather
> > > than taking the time to get up to speed on the issue myself.
> >
> > Storing the *source* git repo for biosbits on gitlab.com/qemu-project
> > is sensible, as that's what we've done for other 3rd party bits that
> > we bundle/depend on git repo access for.
> 
> 
> Great. Can you or Peter please create a git repo cloned from the official
> bios bits repo please? You don’t have to clone mine. Please provide me with
> push access so that I can push fixes that I have made in order to build it.

I don't have any admin privileges for qemu infra, so can't do this
myself.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




reply via email to

[Prev in Thread] Current Thread [Next in Thread]