On Tue, Feb 9, 2016 at 1:59 PM, John Sirois <jsir...@apache.org> wrote:
> > > On Tue, Feb 9, 2016 at 1:47 PM, Roger Meier <ro...@bufferoverflow.ch> > wrote: > >> John, thanks for fixing this! > > > You're welcome, but things went immediately red [1] - so more work to do. > > [1] https://builds.apache.org/job/Thrift-all/557/ > libboost-thread fubar'd, on advice of Jake switching the > Thrift-precommit which had same issue to Docker CI job. > Once that job is sorted I'll return here. > The Thrift-precommit job appears to now be stable using Docker. I've filed https://issues.apache.org/jira/browse/THRIFT-3620 to both kill obsolete jobs to reduce confusion and convert the proper master-protecting job (maybe Thrift-all) over to Docker. I could use background info from any committers with knowledge on that ticket. > > >> >> >> -roger >> >> >> Quoting John Sirois <jsir...@apache.org>: >> >> On Mon, Feb 8, 2016 at 12:49 PM, John Sirois <jsir...@apache.org> wrote: >>> >>> >>>> >>>> On Wed, Feb 3, 2016 at 6:20 PM, John Sirois <jsir...@apache.org> wrote: >>>> >>>> I did some more digging on the `make check` hang I encountered which led >>>>> to this proposal. The originating github PR actually failed TravisCI >>>>> in >>>>> the go shard with exactly this same hang: >>>>> https://travis-ci.org/apache/thrift/jobs/101493938 >>>>> >>>>> I'm an Apache Thrift dev newcomer (though long-time user back to >>>>> ~2010), >>>>> so I have almost no context, but is it typical to merge PRs that fail >>>>> either TravisCI or Jenkins checks? >>>>> I'd be willing to invest a bit of time into process docs, CI setups, >>>>> etc >>>>> to make sure the checks already in-place are actually used to keep >>>>> master >>>>> green. >>>>> >>>>> I may have dipped my toes into the dev waters with simple bad timing, >>>>> but >>>>> pretending that is not the case and I'm observing status quo, things >>>>> like a >>>>> red master are a big detractor to contribution enthusiasm. They give >>>>> me >>>>> less faith in the product as a long-time user, and more work to do to >>>>> get >>>>> to the the work I want to do to develop my own features. >>>>> >>>>> On Wed, Feb 3, 2016 at 5:37 PM, John Sirois <jsir...@apache.org> >>>>> wrote: >>>>> >>>>> I just noticed there is a Thrift-precommit job that _does_ have `make >>>>>> check`: >>>>>> >>>>>> $ curl --netrc -sS >>>>>> https://builds.apache.org/job/Thrift-precommit/config.xml | xmllint >>>>>> --xpath "//command/text()" - >>>>>> sh bootstrap.sh >>>>>> sloccount --duplicates --wide --details lib tutorial test > >>>>>> sloccount.sc >>>>>> cppcheck --error-exitcode=0 --force --xml lib/cpp/src/ lib/c_glib/src/ >>>>>> tutorial/cpp/ test/cpp/ 2> cppcheck-result.xml >>>>>> sh bootstrap.sh >>>>>> sh configure --without-erlang --without-haskell >>>>>> make >>>>>> make check >>>>>> make dist >>>>>> cd lib/java >>>>>> ant javadocsh test/test.sh | tee test.log >>>>>> >>>>>> Perhaps `Thrift-all` is obsolete? >>>>>> >>>>>> Any background on the jenkins jobs for Thrift is appreaciated. There >>>>>> are more than a few jobs and maybe some could be deleted? >>>>>> >>>>>> On Wed, Feb 3, 2016 at 5:34 PM, John Sirois <jsir...@apache.org> >>>>>> wrote: >>>>>> >>>>>> >>>>>>> >>>>>>> On Wed, Feb 3, 2016 at 5:33 PM, John Sirois <jsir...@apache.org> >>>>>>> wrote: >>>>>>> >>>>>>> In the course of working on a new feature for thrift java, I rebased >>>>>>>> against master and hit a test hang runing `make check`. I tracked >>>>>>>> this >>>>>>>> down and have a fix out [1], but I was curious how this hang >>>>>>>> slipped by. I >>>>>>>> happen to have jenkins rights through Apache Aurora to view the job >>>>>>>> configuration for Thrift-all and I found it runs: >>>>>>>> >>>>>>>> >>>>>>> where [1] is: >>>>>>> >>>>>>> [1] https://issues.apache.org/jira/browse/THRIFT-3597 >>>>>>> >>>>>>> >>>>>>> >>>>>>>> $ curl --netrc -sS >>>>>>>> https://builds.apache.org/job/Thrift-all/config.xml >>>>>>>> | xmllint --xpath "//command/text()" - >>>>>>>> #sh bootstrap.sh >>>>>>>> #sloccount --duplicates --wide --details lib tutorial test > >>>>>>>> sloccount.sc >>>>>>>> #cppcheck --error-exitcode=0 --force --xml lib/cpp/src/ >>>>>>>> lib/c_glib/src/ tutorial/cpp/ test/cpp/ 2> cppcheck-result.xml >>>>>>>> sh bootstrap.sh >>>>>>>> sh configure >>>>>>>> make >>>>>>>> make dist >>>>>>>> cd lib/java >>>>>>>> ant javadoc >>>>>>>> >>>>>>>> So `make` and `make dist`, but no `make check`. >>>>>>>> >>>>>>>> I'd like to propose adding make check to this set of commands to >>>>>>>> ensure make check is green. >>>>>>>> >>>>>>>> If folks agree this is a good idea, I can go ahead and make the >>>>>>>> edit. >>>>>>>> >>>>>>>> Thanks for your feedback. >>>>>>>> >>>>>>>> >>>>>>> There has been no feedback on this proposal and so I'd like to >>>> proceed on >>>> lazy consensus (Although I'd rather proceed on active consensus!) and >>>> modify the jenkins Thrift-all job to include a `make check` like the >>>> `Thrift-precommit` job. I'll do this this afternoon unless I hear >>>> objections. >>>> >>>> >>> This change is now in-effect. >>> >> >> >> >