beam-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kenneth Knowles <>
Subject Re: [REQUEST] Python Tests (pre/post-commits) Status
Date Tue, 16 Jul 2019 02:33:59 GMT
Huge +1

Once tests are a little bit red most of the time, they get ignored. I'd be
in favor of automation to make it extra hard to merge when things are red.
For example, making it so you cannot use the GitHub UI to merge when master
is red, but of course you can do a git push not using the UI.


On Mon, Jul 15, 2019 at 5:13 PM Udi Meiri <> wrote:

> Hi,
> I've been trying to merge several Python PRs in the past weeks, but
> Jenkins pre- and post-commit jobs have been red all this time due to
> various reasons. I have a proposal in mind to help deal with this, but it
> can't happen without cooperation from a majority of committers.
> The request from Beam committers is to:
> - Avoid merging when tests are red (the images in the PR template);
> - Use common sense for exceptions to the above (such as if the PR has
> nothing to do with a failing test);
> - Act to make tests green (open a JIRA issue, find the PR with the
> regression, rollback as necessary).
> This stuff has already been discussed on this list and documented here:
> The idea is that if everyone pitches in it will spread the load.
> Thanks for reading.

View raw message