I believe license errors are detected by Apache RAT, which are part of the release profile. I believe you need to run "mvn clean verify -Prelease" to enable anything associated with that profile.
On Mon, Oct 23, 2017 at 11:46 AM Valentyn Tymofieiev <[email protected]> wrote: > Hi Beam-Dev, > > It's been >5 days since the last successful run of a > beam_PreCommit_Java_MavenInstall build[1] and >4 days since last > successful run of beam_PreCommit_Java_MavenInstall[2]. > > Looking at build logs I see following problems. > > 1. After October 17, postcommit builds started to fail with > > Failed to execute goal org.apache.rat:apache-rat-plugin:0.12:check > (default) on project beam-parent: Too many files with unapproved license: 1 > See RAT report in: /home/jenkins/jenkins-slave/workspace/beam_PostCommit_ > Java_MavenInstall/target/beam-parent-2.3.0-SNAPSHOT.rat > > The earliest build that I see this error is Postcommit #5052 [3]. > > This makes me suspect [4] or [5] as a breaking change, since they change > pom files. > > Questions: > - Is there a way we can reproduce this failure locally? mvn clean verify > passes locally for me. > - Is there a way we can see the See RAT report mentioned in the error log? > > > 2. Prior to onset of #1 Java Precommit builds no longer complete within > allotted 150 min time. Looking at [6-8] it seems the build makes consistent > progress, but just does not finish on time. We can also see several recent > successful builds with execution time very close to time out [9-11]. > > I'd like to propose to increase time limit for Java precommit test suite > from 2.5 to 4 hours. 4 hours is long time. I agree that we should > definitely try to reduce the test execution time, and reduce flakiness. > However we need the tests at least pass for now. If we write off failed > test suites as 'flakes' and merge PRs without having a green test signal, > we will have to spend more time tracing breakages such as #1. > > Thoughts? > > Thanks, > Valentyn > > [1] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/ > [2] https://builds.apache.org/job/beam_PostCommit_Java_MavenInstall/ > [3] https://builds.apache.org/job/beam_PostCommit_Java_ > MavenInstall/5052/changes > > [4] https://github.com/apache/beam/commit/d745cc9d8cc1735d3bc3c67ba3e261 > 7cb7f11a8c > <https://github.com/apache/beam/commit/d745cc9d8cc1735d3bc3c67ba3e2617cb7f11a8c> > [5] https://github.com/apache/beam/commit/0d8ab6cbbc762dd9f9be1b3e9a26b6 > c9d0bb6dc3 > <https://github.com/apache/beam/commit/0d8ab6cbbc762dd9f9be1b3e9a26b6c9d0bb6dc3> > > [6] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15222/ > [7] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15195/ > [8] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15220/ > > [9] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15009/ > [10] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15068/ > [11] https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/15016/ >
