[u-boot-test-hooks PATCH v2] travis-ci: Add tests for booting from coreboot

[Tom Rini]

On Fri, Dec 03, 2021 at 09:17:20AM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Fri, 3 Dec 2021 at 09:02, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Fri, Dec 03, 2021 at 08:52:38AM -0700, Simon Glass wrote:
> >
> > > Add a means of testing a coreboot + U-Boot build using qemu.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > ---
> > > As to what to do with people's labs, I think applying these patches does
> > > encourage them and provide people with examples. Having 'no mainline' for
> > > these is going to be an impediment I think.
> > >
> > > Changes in v2:
> > > - Drop the ellesmere symlink
> > >
> > >  bin/travis-ci/conf.coreboot_qemu | 28 ++++++++++++++++++++++++++++
> > >  1 file changed, 28 insertions(+)
> > >  create mode 100644 bin/travis-ci/conf.coreboot_qemu
> >
> > So, now we have the conf, for something 100% virtual, in bin/travis-ci
> > (which yes, ugh, bad name).  If we had a top-level README.rst how would
> > it be unclear that this is a functional example?  In fact, maybe we
> > should clear something up.  How does the network test work for you?  I
> > have to modify them to pass -tftp=/tftpboot so that I can then give it
> > things like helloworld.efi and appropriate grub.efi.  Are you using some
> > wrapper to make things look more like what CI does?
> 
> I haven't tried the network tests, actually. Mostly I run individual
> tests or just boot to a prompt and try things out. One day I would
> like to get labman closer to making this automatic.

Ah.  I have a wrapper to activate a virtualenv for the pytest stuff,
then set PYTHONPATH / UBOOT_TRAVIS_BUILD_DIR and it all just works.
Except since I want to test the network a bit more heavily, I pass in my
own tftpboot dir, with a larger test file.  I could possibly change
things to wrap up and look more like CI does, and require less tweaking.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20211203/8731f476/attachment.sig>