@monsieuricon Nice.
It's not working for linuxppc because our checks are posted on the final patch of the series.
I can make it work if I drop the skip case when the first patch has no status.
@monsieuricon Because these checks are "apply the series and do a bunch of build & boot tests", which is too expensive to do for every patch.
@monsieuricon Yeah we could end up doing that one day. Even so we might skip checks for eg. documentation only patches. So I guess my point is a single patch with no checks doesn't necessarily mean there are no checks on later patches.
@monsieuricon @mpe that's what we do for the #MPTCP subsystem using GitHub Actions:
- Each commit is built separately and checkpatch is also executed on each patch.
https://github.com/multipath-tcp/mptcp-upstream-validate-export-action
- Tests are executed once on the last patch of the series.
https://github.com/multipath-tcp/mptcp-upstream-virtme-docker
- Everything is sent to patchwork later. The tests result is duplicated on each patch.
https://github.com/multipath-tcp/mptcp_net-next/tree/export/.github/workflows
(It's just a shame this has to be done differently for each subsystem)
Thx for your work on this.
Really looking forward to the day where we can use bugbot on bugs without reassigning them to a specific product/component. I recently had a situation again where that topic came up.
@monsieuricon FYI, this won't work for the intel-gfx mailing list, as patchwork-fdo (forked long ago) operates on series and series revisions rather than per-patch.
Running our CI on all patches would pretty much kill the entire point of CI as it would increase testing latency way too much: Imagine if we only boot-tested a 100-patches series... It would easily take 3 hours for very little benefit. Instead, in this time, we can run thousands of tests... but once per revision of the series.