The Gearpump runner has been on master for a long time and is part of our releases. I don't know when this issue showed up of if the Jenkins job never was switched back then, but we should definitely switch it to be analogous to all the other ValidatesRunner suites running on master. It is possible this will reveal some issues if the tests have really not been running for a long time. But it is a valid signal that we should have.
Kenn On Thu, Jun 21, 2018 at 10:04 AM Mikhail Gryzykhin <[email protected]> wrote: > Hello everyone, > > I see that in Jenkins we have > a beam_PostCommit_Java_ValidatesRunner_Gearpump_Gradle job configured. > > Yesterday, I unified creation of post-commit jobs and this made all > post-commit tests run on schedule and enabled Gearpump tests job. > Unfortunately, that job is configured to run against unexisting branch > "gearpump-runner" and began failing. > > I wonder what is the history of Gearpump runner. If we are not working on > this runner, it would be good to disable and remove/rename this job to > reduce clutter in Jenkins. > > Otherwise any person looking at post-commit tests will be confused with > not running or disabled job. > > Currently, I'm going to revert my change to make that job only run on > comment on PullRequest as it was before. > > However I want your input is it ok removing this job completely or > renaming it? > > Regards, > --Mikhail > >
