Quick update on flaky dashboard: Flaky dashboard wasn't working earlier because our trunk build was broken. After trunk was fixed, the format of log lines in consoleText was not the same, so findHangingTests.py was not able to parse it correctly for broken/hanging/timeout tests. That's been fixed now HBASE-18341 <https://issues.apache.org/jira/browse/HBASE-18341>. Drob brought up in other thread that 'treads' isn't working. It's probably because i changed tests names (which are used as keys in python dicts) from just class name to package name+classname (without common org.apache.hadoop.hbase prefix). I had to do it because we have some tests with same class name but in different packages.
I'll take a look at it sometime this week (unless someone wants to take it up and work on this beautiful piece of infra ;) ) On Thu, Jul 6, 2017 at 11:25 PM, Stack <st...@duboce.net> wrote: > On Thu, Jul 6, 2017 at 3:45 PM, Sean Busbey <bus...@apache.org> wrote: > > > that sounds like our project structure is broken. Please make sure > there's > > a jira that tracks it and I'll take a look later. > > > > > > Filed HBASE-18331 for now. > > I can take a look too later. > > St.Ack > > > > > On Thu, Jul 6, 2017 at 6:15 PM, Stack <st...@duboce.net> wrote: > > > > > I tried publishing hbase-3.0.0-SNAPSHOT... so hbase-checkstyle was up > in > > > repo (presuming it relied on an aged-out snapshot). Seems to have > 'fixed' > > > it for now.... > > > > > > St.Ack > > > > > > On Thu, Jul 6, 2017 at 12:50 PM, Stack <st...@duboce.net> wrote: > > > > > > > The 3.0.0-SNAPSHOT looks suspicious ... the hbase version.... > > > > St.Ack > > > > > > > > On Thu, Jul 6, 2017 at 12:49 PM, Stack <st...@duboce.net> wrote: > > > > > > > >> On Thu, Jul 6, 2017 at 12:48 PM, Stack <st...@duboce.net> wrote: > > > >> > > > >>> Checkstyle is currently broke on our builds... looking. > > > >>> St.Ack > > > >>> > > > >>> > > > >> Works if I run it locally (of course) > > > >> St.Ack > > > >> > > > >> > > > >> > > > >> > > > >>> > > > >>> > > > >>> [ERROR] Failed to execute goal org.apache.maven.plugins: > > > maven-checkstyle-plugin:2.17:checkstyle (default-cli) on project > hbase: > > > Execution default-cli of goal org.apache.maven.plugins: > > > maven-checkstyle-plugin:2.17:checkstyle failed: Plugin > > > org.apache.maven.plugins:maven-checkstyle-plugin:2.17 or one of its > > > dependencies could not be resolved: Could not find artifact > > > org.apache.hbase:hbase-checkstyle:jar:3.0.0-SNAPSHOT in Nexus ( > > > http://repository.apache.org/snapshots) -> [Help 1][ERROR] [ERROR] To > > see > > > the full stack trace of the errors, re-run Maven with the -e > > switch.[ERROR] > > > Re-run Maven using the -X switch to enable full debug logging.[ERROR] > > > [ERROR] For more information about the errors and possible solutions, > > > please read the following articles:[ERROR] [Help 1] > > > http://cwiki.apache.org/confluence/display/MAVEN/ > > > PluginResolutionExceptionBuild step 'Invoke top-level Maven targets' > > > marked build as failure > > > >>> Performing Post build task... > > > >>> Match found for :.* : True > > > >>> Logical operation result is TRUE > > > >>> Running script : # Run zombie detector script > > > >>> ./dev-support/zombie-detector.sh --jenkins ${BUILD_ID} > > > >>> [a3159d73] $ /bin/bash -xe /tmp/hudson1697041977582083402.sh > > > >>> + ./dev-support/zombie-detector.sh --jenkins 3320 > > > >>> Thu Jul 6 01:37:09 UTC 2017 We're ok: there is no zombie test > > > >>> > > > >>> > > > >>> > > > >>> > > > >>> On Fri, Jun 30, 2017 at 2:43 PM, Sean Busbey <bus...@apache.org> > > > wrote: > > > >>> > > > >>>> jacoco was added ages ago. I'd guess that something changed on the > > > >>>> machines > > > >>>> we use to cause it to stop working. > > > >>>> > > > >>>> On Thu, Jun 29, 2017 at 12:02 PM, Stack <st...@duboce.net> wrote: > > > >>>> > > > >>>> > On Wed, Jun 28, 2017 at 8:43 AM, Josh Elser <els...@apache.org> > > > >>>> wrote: > > > >>>> > > > > >>>> > > > > > >>>> > > > > > >>>> > > On 6/27/17 7:20 PM, Stack wrote: > > > >>>> > > > > > >>>> > >> * test-patch's whitespace plugin can configured to ignore > some > > > >>>> files > > > >>>> > (but > > > >>>> > >>> I > > > >>>> > >>> can't think of any we'd care to so whitelist) > > > >>>> > >>> > > > >>>> > >>> Generated files. > > > >>>> > >> > > > >>>> > > > > > >>>> > > Oh my goodness, yes, please. This has been such a pain in the > > rear > > > >>>> for me > > > >>>> > > as I've been rebasing space quota patches. Sometimes, the > spaces > > > in > > > >>>> > > pb-gen'ed code are removed by folks before commit, other times > > > they > > > >>>> > aren't. > > > >>>> > > > > > >>>> > > > > >>>> > Agree sir. Its a distraction at least. > > > >>>> > > > > >>>> > I see Jacoco report here now: > > > >>>> > https://builds.apache.org/job/HBase-Trunk_matrix/jdk=JDK% > > > >>>> > 201.8%20(latest),label=Hadoop/3277/ > > > >>>> > > > > >>>> > Maybe it has been there always and I just haven't noticed. > > > >>>> > > > > >>>> > Its all 0%. We need to turn on stuff? > > > >>>> > > > > >>>> > St.Ack > > > >>>> > > > > >>>> > > > >>> > > > >>> > > > >> > > > > > > > > > > -- -- Appy
