[U-Boot] [PATCH] test/run: Exit on error from tests

Tom Rini trini at konsulko.com
Tue Nov 28 01:11:45 UTC 2017


On Mon, Nov 27, 2017 at 02:45:14PM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On 27 November 2017 at 14:21, Tom Rini <trini at konsulko.com> wrote:
> > Currently, result isn't saved outside of the scope of run_test, so we
> > always exit on error here.  An easier way to deal with having test/run
> > show failures is to exit on failure.
> >
> > Cc: Simon Glass <sjg at chromium.org>
> > Fixes: 2f52018c3cbb ("test/run: Report and return failure")
> > Signed-off-by: Tom Rini <trini at konsulko.com>
> > ---
> > Automation is only as good as the humans behind it.  If you forget, like
> > I did, that you finally fixed Jenkins to email you every time, that lack
> > of emails about my non-travis jobs failing wasn't "no problems", it was
> > "You forgot to migrate all the Docker stuff to the new build server".
> > So, I'm going to apply this ASAP as I would not have let in the
> > u-boot-dm PR, had my setup been functioning as expected.
> > ---
> >  test/run | 18 ++++--------------
> >  1 file changed, 4 insertions(+), 14 deletions(-)
> 
> I'm OK with this if it's what you want. This means that we will not
> run all the tests if one of the earlier one fails.
> 
> Did you see this one?
> 
> http://patchwork.ozlabs.org/patch/841453/

I'm playing a little catch-up and that's fine too.  I've unearthed
another bit of breakage to fix now (because the time PRs break my setup
is the time I break runtime of my setup and assume it's working).

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171127/dd53b42d/attachment.sig>


More information about the U-Boot mailing list