Hi Michal,
Am 14.07.2020 um 08:22 schrieb Heiko Schocher:
Hi Michal,
Am 14.07.2020 um 08:09 schrieb Michal Simek:
On 13. 07. 20 22:36, Tom Rini wrote:
On Mon, Jul 13, 2020 at 12:25:42PM -0600, Simon Glass wrote:
Hi,
At present U-Boot uses the mailing list for patch review. What do
people think about trying out geritt or github for this? I'd be
willing to do a trial with the -dm mailing list.
My idea is that patman would email out the patches and also upload
them to one of these systems. With geritt, emails are sent every time
there is a review, but for github I'm not sure.
As you said, you also intended to include gitlab in the list.
So, things that I think could be better, or more widely known, would be
how to trigger Azure CI runs (as there's good resources available for
free) and GitLab for non-custodians. By which I mean, anyone can get a
CI run on Azure today via a GitHub PR. Making that known more widely
would be good.
For patch review, are there any of the Linux kernel bots that are well
enough documented for someone else to pick up and use? I feel like our
biggest problems are:
- Lack of reviewers in general for various areas.
- Lack of feedback to users (developers) before being picked up for
general changes.
For the second problem, I feel like some of the Linux bots would be a
little helpful, but probably require some tweaking (figure out when /
how to fire off more limited CI, or just do a daily CI run vs every
patch, given the size of our CI build).
Don't have details but patchwork should support (somehow) to run some
checks.
Here you can see it working.
http://patchwork.ozlabs.org/project/devicetree-bindings/patch/1594676120-5862-7-git-send-email-prabhakar.mahadev-lad...@bp.renesas.com/
or
http://patchwork.ozlabs.org/project/devicetree-bindings/patch/20200713132927.24925-2-abai...@baylibre.com/
It means when the patch reach patchwork checks can run. Would be
wonderful to get PASS/FAIL with link via email from CI loop.
That looks interesting, indeed!
A fast google search:
https://patchwork-freedesktop.readthedocs.io/en/latest/testing.html
Ok, I think in tbot (sorry) ... make a small script which periodically
polls the patchwork api for a new event:
https://patchwork-freedesktop.readthedocs.io/en/latest/testing.html#polling-for-events
and than start tbot which do all the stuff (download patches, check them
apply, build... call test.py) and write a tbot generator which post
testresult back to patchwork... sounds easy and interesting...
Damn, I have no time currently to prepare such a demo ...
(I think, it can run in 1-2 work days... so if people think this is
interesting may I find some time on weekend...)
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: h...@denx.de
