On Fri, Nov 22, 2024 at 9:17 AM Daniel Sahlberg <daniel.l.sahlb...@gmail.com> wrote: > > Den fre 22 nov. 2024 kl 09:39 skrev Daniel Sahlberg > <daniel.l.sahlb...@gmail.com>: >> >> Den ons 20 nov. 2024 kl 18:15 skrev Daniel Sahlberg >> <daniel.l.sahlb...@gmail.com>: >>> >>> So I'd say we have consensus on moving to the Python implementation - >>> several +1 and no-one objecting. And since this is far from core code and >>> it can easily be reverted I don't see a problem at all. >> >> >> I've been working on this today: > > ... > > I've also update the buildbot "backport-conflicts"-job to the Python version, > it seems to work - at least it completes successfully: > https://ci2.apache.org/#/builders/109/builds/20609. > > It was this job that made me catch the backport conflict lately, but I saw it > just because I went looking for it. > > We currently have jobs running on both GitHub Actions and on Buildbot. > Neither of them is reporting to any mailing list (we have notifications@ that > should be used for this). > > Should we try to get e-mail notifications working again or is there a better > method (in particular in GHA)? Should we try to consolidate all jobs to GHA?
We should get email notifications working. Is that something Infra needs to do? Regarding consolidation, that probably will ease maintenance, especially since the GHA stuff is in our repo. I don't think consolidation needs to happen immediately as long as we can get email notifications of any failures at both GHA and Buildbot. > The jobs we have on Buildbot is: > - backport-conflicts, running tools/dist/detect-conflicting-backports.py on > /branches/1.14.x > - warnings, commit-triggered standard build on Linux, basically the same we > have on GHA (but GHA covers both Linux and Windows and multiple > configurations). We need testing for the currently supported release branch(es) as well as trunk. At least that's what we had before the Buildbot upgrade of a few years ago. I tried to tackle that some time back and wasn't successful. Currently Buildbot is only testing trunk. Let me know if I need to request Infra to set up the email notifications... Thanks, Nathan
