From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH v2] Nokia RX-51: Add automated test for running RX-51 build in qemu
Date: Fri, 15 May 2020 14:34:55 -0400 [thread overview]
Message-ID: <20200515183455.GN4794@bill-the-cat> (raw)
In-Reply-To: <20200515174001.nth2p3rw7eovuvor@pali>
On Fri, May 15, 2020 at 07:40:25PM +0200, Pali Roh?r wrote:
> On Friday 15 May 2020 10:16:16 Tom Rini wrote:
> > On Fri, May 15, 2020 at 03:58:20PM +0200, Pali Roh?r wrote:
> > > On Friday 15 May 2020 09:53:23 Tom Rini wrote:
> > > > On Fri, May 15, 2020 at 03:51:22PM +0200, Pali Roh?r wrote:
> > > > > On Friday 15 May 2020 09:48:48 Tom Rini wrote:
> > > > > > On Fri, May 15, 2020 at 03:46:02PM +0200, Pali Roh?r wrote:
> > > > > > > On Friday 15 May 2020 09:20:20 Tom Rini wrote:
> > > > > > > > On Fri, May 15, 2020 at 09:33:47AM +0200, Pali Roh?r wrote:
> > > > > > > > > On Thursday 14 May 2020 20:01:19 Tom Rini wrote:
> > > > > > > > > > On Fri, May 15, 2020 at 12:41:52AM +0200, Pali Roh?r wrote:
> > > > > > > > > > > On Saturday 09 May 2020 16:56:10 Tom Rini wrote:
> > > > > > > > > > > > On Sat, May 09, 2020 at 06:35:40PM +0200, Pali Roh?r wrote:
> > > > > > > > > > > > > On Saturday 09 May 2020 21:58:19 Lokesh Vutla wrote:
> > > > > > > > > > > > > > On 08/05/20 6:40 PM, Tom Rini wrote:
> > > > > > > > > > > > > > > On Fri, May 08, 2020 at 02:52:55PM +0200, Pali Roh?r wrote:
> > > > > > > > > > > > > > >> On Tuesday 28 April 2020 09:37:21 Pali Roh?r wrote:
> > > > > > > > > > > > > > >>> On Monday 27 April 2020 14:00:47 Tom Rini wrote:
> > > > > > > > > > > > > > >>>> I'll take care of that shortly. Otherwise:
> > > > > > > > > > > > > > >>>>
> > > > > > > > > > > > > > >>>> Reviewed-by: Tom Rini <trini@konsulko.com>
> > > > > > > > > > > > > > >>>
> > > > > > > > > > > > > > >>> Ok, thank you!
> > > > > > > > > > > > > > >>
> > > > > > > > > > > > > > >> Hello Tom! Will you take whole patch series? Or is there anything else
> > > > > > > > > > > > > > >> needed to be done from my side for this N900 patch series?
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > Lokesh is the custodian for the TI tree these days, so whenever it's in
> > > > > > > > > > > > > > > his next PR. Thanks!
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > I see gitlab is failing to build this patch[0]. Does cross compiler needs to be
> > > > > > > > > > > > > > changed?
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > [0] https://gitlab.denx.de/u-boot/custodians/u-boot-ti/-/jobs/91228
> > > > > > > > > > > > >
> > > > > > > > > > > > > Cross compiler arm-linux-gnueabi-gcc needs to be in $PATH.
> > > > > > > > > > > > >
> > > > > > > > > > > > > I figured out that on Travis it is available in ~/.buildman-toolchains
> > > > > > > > > > > > > but not exported to $PATH. So for Travis build I added...
> > > > > > > > > > > > >
> > > > > > > > > > > > > export PATH=~/.buildman-toolchains/gcc-9.2.0-nolibc/arm-linux-gnueabi/bin/:$PATH
> > > > > > > > > > > > >
> > > > > > > > > > > > > ... as can be seen in the last patch.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Do you know where is installed arm-linux-gnueabi toolchain on Gitlab?
> > > > > > > > > > > > > Maybe Tom knows it as he already wrote that would take care of updating
> > > > > > > > > > > > > Gitlab image.
> > > > > > > > > > > >
> > > > > > > > > > > > All of the buildman-fetched toolchains are always in the same place, so
> > > > > > > > > > > > a similar change to gitlab/azure will fix those. Thanks!
> > > > > > > > > > >
> > > > > > > > > > > I see that all patches except this one were merged, thanks.
> > > > > > > > > > >
> > > > > > > > > > > Tom, are you going to take look at this last patch?
> > > > > > > > > > >
> > > > > > > > > > > It already passed on travis [1] [2] but I do not have those gitlab and
> > > > > > > > > > > azure accounts to trigger their jobs. But I think that only correct
> > > > > > > > > > > $PATH is needed for azure and gitlab.
> > > > > > > > > > >
> > > > > > > > > > > [1] - https://github.com/u-boot/u-boot/pull/30
> > > > > > > > > > > [2] - https://travis-ci.org/github/u-boot/u-boot/jobs/679162986
> > > > > > > > > >
> > > > > > > > > > No, I've been waiting for you to make an attempt at fixing the jobs.
> > > > > > > > > > Anyone can get Azure running and there's enough examples to make a
> > > > > > > > > > reasonable attempt at making it work without testing.
> > > > > > > > >
> > > > > > > > > So can you give me pointers how to run it?
> > > > > > > >
> > > > > > > > It's a well documented public service. The only slight trick is you
> > > > > > > > need to point it at .azure-pipeline.yml and not whatever the default
> > > > > > > > non-dotfile name is.
> > > > > > >
> > > > > > > Tom, sorry, but I grepped whole u-boot source code repository and I did
> > > > > > > not find any documentation nor README nor any other information how to
> > > > > > > run / extend or modify this service. That is why I asked for some
> > > > > > > information... e.g. how I can I run it and check if it is working or
> > > > > > > not.
> > > > > > >
> > > > > > > > > And is there something more needed for travis job?
> > > > > > > >
> > > > > > > > All 3 CIs need to pass, but no, if Travis is passing, that part is fine.
> > > > > > > > Since Azure/GitLab share the same docker image (which sadly I don't see
> > > > > > > > how to make Travis also do), that's why fixing Azure should let you see
> > > > > > > > what to drop in for GitLab.
> > > > > > >
> > > > > > > I prepared this N900 Travis setup for your request [1] and I do not like
> > > > > > > to see it thrown away, just because there is unrelated issue on Azure.
> > > > > > >
> > > > > > > I have used Travis before, so I know that opening pull request on github
> > > > > > > triggers Travis build and Github directly shows me links to result.
> > > > > > >
> > > > > > > But whatever I did, I was not able to trigger that azure from github
> > > > > > > pull request.
> > > > > > >
> > > > > > > [1] - https://lists.denx.de/pipermail/u-boot/2018-December/353019.html
> > > > > >
> > > > > > Sorry, I mean Azure itself is a well documented public SaaS CI tool. It
> > > > > > plugs in to GitHub just as easy as Travis does, but runs quicker.
> > > > >
> > > > > So seems it is buggy, it was not triggered, see that only Travis was
> > > > > triggered in pull request: https://github.com/u-boot/u-boot/pull/30
> > > >
> > > > Did you configure your Azure account? If you've used Travis elsewhere
> > > > that's why U-Boot just runs there.
> > >
> > > I do not have any Azure account.
> > >
> > > For running Travis I do not need any account. It is enough if repository
> > > owner (probably you?) on github enable Travis for particular github
> > > repository. And then Travis is automatically triggered for every open
> > > pull request (even for those who do not have Travis account).
> >
> > Oh, interesting. Since PRs to U-Boot github are ignored I didn't notice
> > you could get Travis triggered on that. I've finally hooked "u-boot" in
> > to Azure as well, rather than just my repository, so a new PR may
> > trigger a build there.
>
> Now I updated pull request on github and Azure build was triggered, thanks!
>
> But job is failing because in your (probably?) runner image is missing
> mtools package, see:
>
> https://dev.azure.com/u-boot/u-boot/_build/results?buildId=587&view=logs&j=9a06d2a9-1498-5de0-2a01-be581d48ba67&t=f9a6b761-daa3-500f-4840-65a939c5040d
>
> Could you add mtools, fakeroot and mtd-utils packages into that image?
More PATH problems I guess, the Dockerfile is at
https://gitlab.denx.de/u-boot/gitlab-ci-runner/blob/master/Dockerfile
and I put mtools, etc, in last time I updated things.
--
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/20200515/865d5b45/attachment.sig>
next prev parent reply other threads:[~2020-05-15 18:34 UTC|newest]
Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-31 22:35 [PATCH 00/11] Fixes for Nokia RX-51 Pali Rohár
2020-03-31 22:35 ` [PATCH 01/11] Nokia RX-51: Update my email address Pali Rohár
2020-03-31 22:35 ` [PATCH 02/11] Nokia RX-51: Add README.nokia_rx51 file to MAINTAINERS Pali Rohár
2020-03-31 22:35 ` [PATCH 03/11] Nokia RX-51: Move comment about CONFIG_SYS_TEXT_BASE to correct place Pali Rohár
2020-03-31 22:35 ` [PATCH 04/11] Nokia RX-51: Move code from defconfig back to C header file Pali Rohár
2020-03-31 22:35 ` [PATCH 05/11] Nokia RX-51: Revert back onenand defitions Pali Rohár
2020-03-31 22:35 ` [PATCH 06/11] Nokia RX-51: Remove PART* macros Pali Rohár
2020-03-31 22:35 ` [PATCH 07/11] Nokia RX-51: Remember setup_console_atag option Pali Rohár
2020-03-31 22:35 ` [PATCH 08/11] Nokia RX-51: Enable CONFIG_CONSOLE_MUX Pali Rohár
2020-04-14 10:19 ` Lokesh Vutla
2020-03-31 22:35 ` [PATCH 09/11] Nokia RX-51: Disable some unused features to decrease size of u-boot binary Pali Rohár
2020-03-31 22:35 ` [PATCH 10/11] Nokia RX-51: Update README.nokia_rx51 Pali Rohár
2020-03-31 22:35 ` [PATCH 11/11] Nokia RX-51: Add automated test for running RX-51 build in qemu Pali Rohár
2020-04-14 10:40 ` Pali Rohár
2020-04-21 14:55 ` Lokesh Vutla
2020-04-21 17:36 ` Simon Glass
2020-04-21 20:12 ` Tom Rini
2020-04-21 20:37 ` Simon Glass
2020-04-21 20:46 ` Tom Rini
2020-04-21 20:49 ` Simon Glass
2020-04-21 20:51 ` Tom Rini
2020-04-21 21:34 ` Pali Rohár
2020-04-21 23:24 ` Tom Rini
2020-04-23 7:34 ` Pali Rohár
2020-04-23 12:24 ` Tom Rini
2020-04-23 17:48 ` Pali Rohár
2020-04-25 9:00 ` [PATCH v2] " Pali Rohár
2020-04-27 8:40 ` Pali Rohár
2020-04-27 18:00 ` Tom Rini
2020-04-28 7:37 ` Pali Rohár
2020-05-08 12:52 ` Pali Rohár
2020-05-08 13:10 ` Tom Rini
2020-05-09 16:28 ` Lokesh Vutla
2020-05-09 16:35 ` Pali Rohár
2020-05-09 20:56 ` Tom Rini
2020-05-14 22:41 ` Pali Rohár
2020-05-15 0:01 ` Tom Rini
2020-05-15 7:33 ` Pali Rohár
2020-05-15 13:20 ` Tom Rini
2020-05-15 13:46 ` Pali Rohár
2020-05-15 13:48 ` Tom Rini
2020-05-15 13:51 ` Pali Rohár
2020-05-15 13:53 ` Tom Rini
2020-05-15 13:58 ` Pali Rohár
2020-05-15 14:16 ` Tom Rini
2020-05-15 17:40 ` Pali Rohár
2020-05-15 18:34 ` Tom Rini [this message]
2020-05-17 12:31 ` Pali Rohár
2020-05-17 12:38 ` [PATCH v3] " Pali Rohár
2020-05-26 9:18 ` Pali Rohár
2020-05-26 9:22 ` Lokesh Vutla
2020-05-26 9:32 ` Pali Rohár
2020-05-26 9:33 ` Lokesh Vutla
2020-04-21 21:03 ` [PATCH 11/11] " Pali Rohár
2020-04-21 20:53 ` Pali Rohár
2020-03-31 22:42 ` U-Boot is broken on real N900 HW (Was: Re: [PATCH 00/11] Fixes for Nokia RX-51) Pali Rohár
[not found] ` <3c7dda52-10b3-e8c3-a382-785c80f124e7@wizzup.org>
2020-04-02 18:42 ` U-Boot is broken on real N900 HW Pali Rohár
2020-04-25 10:42 ` Pali Rohár
2020-04-25 11:36 ` Adam Ford
2020-04-25 11:50 ` Pali Rohár
2020-04-25 12:00 ` Adam Ford
2020-04-25 12:11 ` Pali Rohár
2020-04-25 23:54 ` U-Boot i2c bus num 1 is broken on Nokia N900 (Was: Re: U-Boot is broken on real N900 HW) Pali Rohár
2020-04-27 7:03 ` Heiko Schocher
2020-10-26 21:48 ` U-Boot i2c bus num 1 is broken on Nokia N900 Pali Rohár
2020-10-28 5:42 ` Heiko Schocher
2020-10-28 10:46 ` Pali Rohár
2020-10-29 7:51 ` Ivaylo Dimitrov
2020-10-29 9:32 ` Heiko Schocher
2020-10-29 9:36 ` Pali Rohár
2020-10-30 7:00 ` Ivaylo Dimitrov
2020-10-30 7:24 ` Heiko Schocher
2020-10-31 11:47 ` Ivaylo Dimitrov
2020-11-02 7:13 ` Heiko Schocher
2020-04-25 12:07 ` U-Boot is broken on real N900 HW Pali Rohár
2020-04-25 13:19 ` Pali Rohár
2020-04-25 13:48 ` Pali Rohár
2020-04-25 21:26 ` Bisected: mmc cause reboot loops on N900 (Was: Re: U-Boot is broken on real N900 HW) Pali Rohár
2020-04-25 22:20 ` Pali Rohár
2020-04-25 22:29 ` Bisected: omap_hsmmc 3.3V IO voltage incompatible with N900 (Was: Re: Bisected: mmc cause reboot loops on N900) Pali Rohár
2020-05-07 13:40 ` Faiz Abbas
2020-05-07 15:19 ` Pali Rohár
2020-05-26 17:49 ` Pali Rohár
2020-06-12 13:03 ` Pali Rohár
2020-07-01 8:32 ` Pali Rohár
2020-07-01 8:51 ` Faiz Abbas
2020-05-03 21:31 ` Bisected: mmc cause reboot loops on N900 Pali Rohár
2020-04-26 17:13 ` Bisected: mmc cause reboot loops on N900 (Was: Re: U-Boot is broken on real N900 HW) Pavel Machek
2020-04-06 20:12 ` U-Boot is broken on real N900 HW (Was: Re: [PATCH 00/11] Fixes for Nokia RX-51) Pavel Machek
2020-04-06 22:27 ` Pali Rohár
2020-04-13 10:41 ` [PATCH 00/11] Fixes for Nokia RX-51 Pali Rohár
2020-04-14 10:23 ` Lokesh Vutla
2020-04-14 10:31 ` Pali Rohár
2020-04-14 10:44 ` Lokesh Vutla
2020-04-14 11:17 ` Pali Rohár
2020-04-14 11:51 ` Lokesh Vutla
2020-04-14 12:01 ` Pali Rohár
2020-04-16 21:57 ` Pali Rohár
2020-04-20 8:12 ` Lokesh Vutla
2020-04-20 23:21 ` Pali Rohár
2020-05-11 12:47 ` Lokesh Vutla
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=20200515183455.GN4794@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox