The problem is only on backward compatibiliy tests I am going to disable Jacoco in master build I wlil be back when I have news
-- Enrico ------------------------------------------------------- T E S T S ------------------------------------------------------- Running org.apache.bookkeeper.tests.backward.TestBookieRecovery Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 2.621 sec <<< FAILURE! - in org.apache.bookkeeper.tests.backward.TestBookieRecovery ensurePasswordUsedForOldLedgers(org.apache.bookkeeper.tests.backward.TestBookieRecovery) Time elapsed: 2.284 sec <<< ERROR! java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/bookkeeper/client/BookKeeper$DigestType at org.apache.bookkeeper.tests.backward.TestBookieRecovery.ensurePasswordUsedForOldLedgers(TestBookieRecovery.java:266) Caused by: java.lang.ClassNotFoundException: org.apache.bk_v4_1_0.bookkeeper.client.BookKeeper$DigestType at org.apache.bookkeeper.tests.backward.TestBookieRecovery.ensurePasswordUsedForOldLedgers(TestBookieRecovery.java:266) Running org.apache.bookkeeper.tests.backward.TestBackwardCompat Tests run: 5, Failures: 0, Errors: 5, Skipped: 0, Time elapsed: 1.087 sec <<< FAILURE! - in org.apache.bookkeeper.tests.backward.TestBackwardCompat testCompatHierarchicalLedgerManager(org.apache.bookkeeper.tests.backward.TestBackwardCompat) Time elapsed: 0.646 sec <<< ERROR! java.lang.NoClassDefFoundError: org/apache/bk_v4_2_0/bookkeeper/conf/ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testCompatHierarchicalLedgerManager(TestBackwardCompat.java:677) Caused by: java.lang.ClassNotFoundException: org.apache.bk_v4_2_0.bookkeeper.conf.ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testCompatHierarchicalLedgerManager(TestBackwardCompat.java:677) testCompatWrites(org.apache.bookkeeper.tests.backward.TestBackwardCompat) Time elapsed: 0.033 sec <<< ERROR! java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/bookkeeper/conf/ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testCompatWrites(TestBackwardCompat.java:623) Caused by: java.lang.ClassNotFoundException: org.apache.bk_v4_1_0.bookkeeper.conf.ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testCompatWrites(TestBackwardCompat.java:623) testCompatReads(org.apache.bookkeeper.tests.backward.TestBackwardCompat) Time elapsed: 0.041 sec <<< ERROR! java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/bookkeeper/conf/ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testCompatReads(TestBackwardCompat.java:573) testCompat410(org.apache.bookkeeper.tests.backward.TestBackwardCompat) Time elapsed: 0.017 sec <<< ERROR! java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/bookkeeper/conf/ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testCompat410(TestBackwardCompat.java:455) testOldCookieAccessingNewCluster(org.apache.bookkeeper.tests.backward.TestBackwardCompat) Time elapsed: 0.018 sec <<< ERROR! java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/bookkeeper/conf/ServerConfiguration at org.apache.bookkeeper.tests.backward.TestBackwardCompat.testOldCookieAccessingNewCluster(TestBackwardCompat.java:403) 2017-09-20 20:37 GMT+02:00 Enrico Olivelli <eolive...@gmail.com>: > Sorry, > I did not try with the latest master but it should not be a problem. > I will investigate and maybe revert > It could be a problem related to the env, > Eventually using docker will be nicer and more reproducible > I will work on it asap tomorrow > Enrico > > On mer 20 set 2017, 20:28 Sijie Guo <guosi...@gmail.com> wrote: > >> did you test this with current master before enabling in >> bookkeeper-master job? It somehow broke the jenkins. >> >> because I saw ClassNotFoundException in latest jekins build. >> >> https://builds.apache.org/blue/organizations/jenkins/ >> bookkeeper-master/detail/bookkeeper-master/1889/pipeline >> >> >> >> On Wed, Sep 20, 2017 at 2:34 AM, Enrico Olivelli <eolive...@gmail.com> >> wrote: >> >>> I have updated master job >>> https://builds.apache.org/job/bookkeeper-master/ >>> >>> let's see the results >>> -- Enrico >>> >>> 2017-08-24 10:27 GMT+02:00 Enrico Olivelli <eolive...@gmail.com>: >>> >>>> It seems that JaCoCo works like a charm >>>> https://builds.apache.org/job/bookkeeper-codecoverage/3/console >>>> https://coveralls.io/github/apache/bookkeeper >>>> >>>> I will integrate it into nightly builds, the only concern is that >>>> CodeCoverage instrumentation may affect tests >>>> thoughts ? >>>> >>>> -- Enrico >>>> >>>> >>>> >>>> 2017-08-24 2:09 GMT+02:00 Jia Zhai <zhaiji...@gmail.com>: >>>> >>>>> 👍 >>>>> >>>>> On Wed, Aug 23, 2017 at 11:38 PM, Enrico Olivelli <eolive...@gmail.com >>>>> > >>>>> wrote: >>>>> >>>>> > 2017-08-23 17:34 GMT+02:00 Sijie Guo <guosi...@gmail.com>: >>>>> > >>>>> > > On Aug 23, 2017 7:31 AM, "Enrico Olivelli" <eolive...@gmail.com> >>>>> wrote: >>>>> > > >>>>> > > 2017-08-23 8:29 GMT+02:00 Enrico Olivelli <eolive...@gmail.com>: >>>>> > > >>>>> > > > Latest news >>>>> > > > >>>>> > > > The build works and coveralls reports a 63% code coverage. >>>>> > > > Maybe the link to the badge on readme.md is wrong, I will check >>>>> > > > >>>>> > > >>>>> > > Our core in README.md is good, but github caches images, this is >>>>> the good >>>>> > > link but GitHub cached an old version of the image >>>>> > > https://coveralls.io/repos/github/apache/bookkeeper/ >>>>> > > badge.svg?branch=master >>>>> > > >>>>> > > >>>>> > > >>>>> > > > Cobertura reports a lot of parsing errors, I will continue to try >>>>> > JaCoCo >>>>> > > > >>>>> > > >>>>> > > I have tried with JaCoCo the result is similar, but there are no >>>>> "scary" >>>>> > > message on console output like this: >>>>> > > >>>>> > > net.sourceforge.cobertura.javancss.parser.ParseException: >>>>> Encountered >>>>> > > " "default" "default "" at line 332, column 5. >>>>> > > Was expecting one of: >>>>> > > "abstract" ... >>>>> > > "boolean" ... >>>>> > > "byte" ... >>>>> > > "char" ... >>>>> > > "class" ... >>>>> > > "double" ... >>>>> > > "enum" ... >>>>> > > "final" ... >>>>> > > "float" ... >>>>> > > "int" ... >>>>> > > "interface" ... >>>>> > > "long" ... >>>>> > > "native" ... >>>>> > > "private" ... >>>>> > > "protected" ... >>>>> > > "public" ... >>>>> > > "short" ... >>>>> > > "static" ... >>>>> > > "strictfp" ... >>>>> > > "synchronized" ... >>>>> > > "transient" ... >>>>> > > "void" ... >>>>> > > "volatile" ... >>>>> > > <IDENTIFIER> ... >>>>> > > "}" ... >>>>> > > ";" ... >>>>> > > "@" ... >>>>> > > "<" ... >>>>> > > >>>>> > > at net.sourceforge.cobertura.javancss.parser.JavaParser. >>>>> > > generateParseException(JavaParser.java:10490) >>>>> > > at net.sourceforge.cobertura. >>>>> javancss.parser.JavaParser.jj_ >>>>> > > consume_token(JavaParser.java:10366) >>>>> > > at net.sourceforge.cobertura.javancss.parser.JavaParser. >>>>> > > UnmodifiedInterfaceDeclaration(JavaParser.java:1475) >>>>> > > at net.sourceforge.cobertura.javancss.parser.JavaParser. >>>>> > > InterfaceDeclaration(JavaParser.java:1273) >>>>> > > at net.sourceforge.cobertura.javancss.parser.JavaParser. >>>>> > > TypeDeclaration(JavaParser.java:580) >>>>> > > at net.sourceforge.cobertura.javancss.parser.JavaParser. >>>>> > > CompilationUnit(JavaParser.java:318) >>>>> > > at net.sourceforge.cobertura.javancss.parser.JavaParser. >>>>> > > parse(JavaParser.java:100) >>>>> > > at net.sourceforge.cobertura.javancss.Javancss._ >>>>> > > measureSource(Javancss.java:192) >>>>> > > at net.sourceforge.cobertura.javancss.Javancss._ >>>>> > > measureRoot(Javancss.java:268) >>>>> > > at net.sourceforge.cobertura.javancss.Javancss.<init>( >>>>> > > Javancss.java:347) >>>>> > > at net.sourceforge.cobertura.javancss.JavancssFactory. >>>>> > > newInstance(JavancssFactory.java:40) >>>>> > > at net.sourceforge.cobertura.reporting. >>>>> ComplexityCalculator. >>>>> > > getFunctionMetricsForSingleFile(ComplexityCalculator.java:243) >>>>> > > >>>>> > > >>>>> > > >>>>> > > Maybe it is better to switch to JaCoCo, Cobertura wants to read >>>>> java code >>>>> > > but cannot handle Java8 default methods interface syntax >>>>> > > >>>>> > > Issue: >>>>> > > https://github.com/apache/bookkeeper/issues/464 >>>>> > > >>>>> > > Patch to introduce the configuration: >>>>> > > https://github.com/apache/bookkeeper/pull/465 >>>>> > > >>>>> > > >>>>> > > Good job! I made one comment in the pull request. Otherwise, >>>>> overall >>>>> > looks >>>>> > > good. >>>>> > > >>>>> > > >>>>> > > >>>>> > > >>>>> > > Once we have a stable code coverage tool I would like to setup an >>>>> > automatic >>>>> > > check upon Pull Requests >>>>> > > >>>>> > > >>>>> > > Sure, are you planning to enforce pull request should increase >>>>> coverage? >>>>> > > >>>>> > >>>>> > >>>>> > It would be great >>>>> > but I think it is not really possible, sometimes we have patches on >>>>> the >>>>> > website or on poms >>>>> > I have to understand better how code coverage tools work >>>>> > for instance I see that interfaces code seems to be "uncovered" so >>>>> > introducing a new interface will decrease the coverage >>>>> > I have to play with it >>>>> > >>>>> > >>>>> > > >>>>> > > >>>>> > > >>>>> > > Thoughts ? >>>>> > > >>>>> > > Enrico >>>>> > > >>>>> > > >>>>> > > >>>>> > > I will try on DL to in case we have to switch >>>>> > > > >>>>> > > > Enrico >>>>> > > > >>>>> > > > On mar 22 ago 2017, 22:00 Enrico Olivelli <eolive...@gmail.com> >>>>> wrote: >>>>> > > > >>>>> > > >> 2017-08-22 21:52 GMT+02:00 Sijie Guo <guosi...@gmail.com>: >>>>> > > >> >>>>> > > >>> Hi Enrico, >>>>> > > >>> >>>>> > > >>> You should use 'ubuntu' rather than 'Ubuntu' (it is case >>>>> sensitive). >>>>> > > It >>>>> > > >>> is >>>>> > > >>> running now - >>>>> > > >>> https://builds.apache.org/job/bookkeeper-codecoverage/1/ >>>>> console >>>>> > > >>> >>>>> > > >>> >>>>> > > >> Thank you. I thought I had tried but without success. >>>>> > > >> >>>>> > > >> >>>>> > > >>> Also It would be good to put code-coverage into >>>>> > > >>> https://builds.apache.org/job/bookkeeper-release-nightly- >>>>> snapshot/ >>>>> > > >>> eventually. >>>>> > > >>> >>>>> > > >> >>>>> > > >> I will play and the integrate the two jobs >>>>> > > >> >>>>> > > >> Thank you very much >>>>> > > >> Enrico >>>>> > > >> >>>>> > > >> >>>>> > > >>> >>>>> > > >>> - Sijie >>>>> > > >>> >>>>> > > >>> >>>>> > > >>> >>>>> > > >>> On Mon, Aug 21, 2017 at 9:48 PM, Sijie Guo <guosi...@gmail.com >>>>> > >>>>> > wrote: >>>>> > > >>> >>>>> > > >>> > I will check it tomorrow. >>>>> > > >>> > >>>>> > > >>> > On Aug 20, 2017 1:23 AM, "Enrico Olivelli" < >>>>> eolive...@gmail.com> >>>>> > > >>> wrote: >>>>> > > >>> > >>>>> > > >>> >> Sijie, >>>>> > > >>> >> I am not able to make the new job run >>>>> > > >>> >> https://builds.apache.org/job/bookkeeper-codecoverage/ >>>>> > > >>> >> >>>>> > > >>> >> I cannot find any available node, I tried with ubumtu and >>>>> haddop, >>>>> > > with >>>>> > > >>> >> different cases for the first char >>>>> > > >>> >> >>>>> > > >>> >> Ideas? >>>>> > > >>> >> Enrico >>>>> > > >>> >> >>>>> > > >>> >> On sab 19 ago 2017, 15:35 Enrico Olivelli < >>>>> eolive...@gmail.com> >>>>> > > >>> wrote: >>>>> > > >>> >> >>>>> > > >>> >> > I have created a new jenkins job >>>>> > > >>> >> > https://builds.apache.org/job/bookkeeper-codecoverage/ >>>>> > > >>> >> > >>>>> > > >>> >> > emails will be sent to my address eolive...@apache.org >>>>> > > >>> >> > >>>>> > > >>> >> > I am playing with cobertura, if I won't succeed I will >>>>> create a >>>>> > PR >>>>> > > >>> in >>>>> > > >>> >> > order to introduce the configuration of JaCoCo on master >>>>> branch >>>>> > > >>> >> > >>>>> > > >>> >> > Enrico >>>>> > > >>> >> > >>>>> > > >>> >> > >>>>> > > >>> >> > 2017-08-04 21:55 GMT+02:00 Sijie Guo <guosi...@gmail.com >>>>> >: >>>>> > > >>> >> > >>>>> > > >>> >> >> Sure go for it. >>>>> > > >>> >> >> >>>>> > > >>> >> >> Sijie >>>>> > > >>> >> >> >>>>> > > >>> >> >> On Aug 4, 2017 12:46 PM, "Enrico Olivelli" < >>>>> > eolive...@gmail.com> >>>>> > > >>> >> wrote: >>>>> > > >>> >> >> >>>>> > > >>> >> >> > Il gio 20 lug 2017, 17:06 Enrico Olivelli < >>>>> > eolive...@gmail.com >>>>> > > > >>>>> > > >>> ha >>>>> > > >>> >> >> > scritto: >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > > Jacoco is running as expected >>>>> > > >>> >> >> > > see: >>>>> > > >>> >> >> > > https://coveralls.io/jobs/27785961 >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > > I think it is better to switch to Jacoco >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > > I wonder why on DL there is no such problem with >>>>> Cobertura >>>>> > > >>> >> >> > > @Sijie do you have any idea ? >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > Other opinions? I am going to test codecov.io too in >>>>> other >>>>> > > >>> projects >>>>> > > >>> >> and >>>>> > > >>> >> >> > compare the results >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > Enrico >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > > -- Enrico >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > > 2017-07-20 16:06 GMT+02:00 Enrico Olivelli < >>>>> > > >>> eolive...@gmail.com>: >>>>> > > >>> >> >> > > >>>>> > > >>> >> >> > >> I am sorry but it seems that cobertura is not >>>>> working with >>>>> > > >>> java8, >>>>> > > >>> >> as >>>>> > > >>> >> >> > >> reported in Coveralls.io homepage >>>>> > > >>> >> >> > >> https://github.com/trautonen/coveralls-maven-plugin >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> see our error: >>>>> > > >>> >> >> > >> https://builds.apache.org/job/ >>>>> bookkeeper-tests/6/console >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> I will be back to Jacoco which seems to be the only >>>>> one >>>>> > > which >>>>> > > >>> >> >> supports >>>>> > > >>> >> >> > >> java8 for Coveralls >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> -- Enrico >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> 2017-07-19 18:02 GMT+02:00 Sijie Guo < >>>>> guosi...@gmail.com >>>>> > >: >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >>> Hmm he should mail you the token rather then >>>>> posting on >>>>> > the >>>>> > > >>> jira >>>>> > > >>> >> :) >>>>> > > >>> >> >> > >>> >>>>> > > >>> >> >> > >>> On Jul 19, 2017 8:56 AM, "Enrico Olivelli" < >>>>> > > >>> eolive...@gmail.com> >>>>> > > >>> >> >> > wrote: >>>>> > > >>> >> >> > >>> >>>>> > > >>> >> >> > >>> > We have the token, it is in the ticket. >>>>> > > >>> >> >> > >>> > I will do some tests soon >>>>> > > >>> >> >> > >>> > Enrico >>>>> > > >>> >> >> > >>> > >>>>> > > >>> >> >> > >>> > Il mer 19 lug 2017, 17:52 Enrico Olivelli < >>>>> > > >>> eolive...@gmail.com >>>>> > > >>> >> > >>>>> > > >>> >> >> ha >>>>> > > >>> >> >> > >>> > scritto: >>>>> > > >>> >> >> > >>> > >>>>> > > >>> >> >> > >>> > > this is the INFRA ticket for having a >>>>> coveralls.io >>>>> > > >>> token in >>>>> > > >>> >> our >>>>> > > >>> >> >> > >>> Jenkins >>>>> > > >>> >> >> > >>> > > builds >>>>> > > >>> >> >> > >>> > > https://issues.apache.org/ >>>>> jira/browse/INFRA-14649 >>>>> > > >>> >> >> > >>> > > >>>>> > > >>> >> >> > >>> > > -- Enrico >>>>> > > >>> >> >> > >>> > > >>>>> > > >>> >> >> > >>> > > 2017-07-17 23:49 GMT+02:00 Enrico Olivelli < >>>>> > > >>> >> eolive...@gmail.com >>>>> > > >>> >> >> >: >>>>> > > >>> >> >> > >>> > > >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >> Il lun 17 lug 2017, 22:55 Sijie Guo < >>>>> > > guosi...@gmail.com >>>>> > > > >>>>> > > >>> ha >>>>> > > >>> >> >> > >>> scritto: >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >>> On Tue, Jul 18, 2017 at 4:46 AM, Enrico >>>>> Olivelli < >>>>> > > >>> >> >> > >>> eolive...@gmail.com> >>>>> > > >>> >> >> > >>> > >>> wrote: >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >>> > Il lun 17 lug 2017, 21:34 Sijie Guo < >>>>> > > >>> guosi...@gmail.com> >>>>> > > >>> >> ha >>>>> > > >>> >> >> > >>> scritto: >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > > On Sun, Jul 16, 2017 at 8:06 PM, Enrico >>>>> > Olivelli >>>>> > > < >>>>> > > >>> >> >> > >>> > >>> eolive...@gmail.com> >>>>> > > >>> >> >> > >>> > >>> > > wrote: >>>>> > > >>> >> >> > >>> > >>> > > >>>>> > > >>> >> >> > >>> > >>> > > > I have just committed coveralls.io and >>>>> > > cobertura >>>>> > > >>> >> >> plugins >>>>> > > >>> >> >> > >>> into >>>>> > > >>> >> >> > >>> > main >>>>> > > >>> >> >> > >>> > >>> > > pom.xml >>>>> > > >>> >> >> > >>> > >>> > > > (as DistributedLog does) >>>>> > > >>> >> >> > >>> > >>> > > > The next step is to enable cobertura >>>>> and >>>>> > tests >>>>> > > in >>>>> > > >>> >> >> Travis. >>>>> > > >>> >> >> > >>> > >>> > > > Currently we do not run tests on Travis >>>>> > because >>>>> > > >>> it is >>>>> > > >>> >> >> very >>>>> > > >>> >> >> > >>> > >>> slow..... >>>>> > > >>> >> >> > >>> > >>> > > > any ideas ? >>>>> > > >>> >> >> > >>> > >>> > > > >>>>> > > >>> >> >> > >>> > >>> > > >>>>> > > >>> >> >> > >>> > >>> > > I believe you need a coveralls token for >>>>> > > reporting >>>>> > > >>> the >>>>> > > >>> >> >> > coverage >>>>> > > >>> >> >> > >>> > >>> results. >>>>> > > >>> >> >> > >>> > >>> > > Jenkins is probably better for this >>>>> purpose >>>>> > > >>> because the >>>>> > > >>> >> >> token >>>>> > > >>> >> >> > >>> has >>>>> > > >>> >> >> > >>> > to >>>>> > > >>> >> >> > >>> > >>> be >>>>> > > >>> >> >> > >>> > >>> > > store secretly. >>>>> > > >>> >> >> > >>> > >>> > > >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > In this doc it seems that using the >>>>> coveralls >>>>> > > plugin >>>>> > > >>> >> inside >>>>> > > >>> >> >> a >>>>> > > >>> >> >> > >>> travis >>>>> > > >>> >> >> > >>> > >>> build >>>>> > > >>> >> >> > >>> > >>> > does not require tokens >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> https://github.com/trautonen/ >>>>> > > >>> coveralls-maven-plugin/blob/ >>>>> > > >>> >> >> > >>> > master/README.md >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >>> I barely remembered that it doesn't actually >>>>> work, >>>>> > > but >>>>> > > >>> I >>>>> > > >>> >> >> might be >>>>> > > >>> >> >> > >>> > wrong. >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >> I just pinged infra, my email was not >>>>> responded >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >> Enrico >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > Enrico >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>>>> > > >>> >> >> > >>> > >>> > > > Meanwhile I am going to submit a PR >>>>> with a >>>>> > > >>> change in >>>>> > > >>> >> >> Travis >>>>> > > >>> >> >> > >>> > config >>>>> > > >>> >> >> > >>> > >>> in >>>>> > > >>> >> >> > >>> > >>> > > order >>>>> > > >>> >> >> > >>> > >>> > > > to test Cobertura speed on Travis >>>>> > > >>> >> >> > >>> > >>> > > > >>>>> > > >>> >> >> > >>> > >>> > > > -- Enrico >>>>> > > >>> >> >> > >>> > >>> > > > >>>>> > > >>> >> >> > >>> > >>> > > > 2017-07-06 17:30 GMT+02:00 Enrico >>>>> Olivelli < >>>>> > > >>> >> >> > >>> eolive...@gmail.com >>>>> > > >>> >> >> > >>> > >: >>>>> > > >>> >> >> > >>> > >>> > > > >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > 2017-07-05 16:13 GMT+02:00 Enrico >>>>> Olivelli >>>>> > < >>>>> > > >>> >> >> > >>> > eolive...@gmail.com >>>>> > > >>> >> >> > >>> > >>> >: >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >> 2017-07-05 16:05 GMT+02:00 Sijie >>>>> Guo < >>>>> > > >>> >> >> > guosi...@gmail.com >>>>> > > >>> >> >> > >>> >: >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> It would be worth checking with >>>>> the INFRA >>>>> > > to >>>>> > > >>> see >>>>> > > >>> >> >> what >>>>> > > >>> >> >> > >>> kind of >>>>> > > >>> >> >> > >>> > >>> code >>>>> > > >>> >> >> > >>> > >>> > > > >>> coverage >>>>> > > >>> >> >> > >>> > >>> > > > >>> that it provides. I remembered >>>>> that a few >>>>> > > ASF >>>>> > > >>> >> >> projects >>>>> > > >>> >> >> > >>> using >>>>> > > >>> >> >> > >>> > >>> > > > >>> coveralls.io, >>>>> > > >>> >> >> > >>> > >>> > > > >>> INFRA can enable this for you and >>>>> give >>>>> > you >>>>> > > an >>>>> > > >>> >> token >>>>> > > >>> >> >> > >>> which you >>>>> > > >>> >> >> > >>> > >>> can >>>>> > > >>> >> >> > >>> > >>> > > > >>> configure >>>>> > > >>> >> >> > >>> > >>> > > > >>> in the CI job. I think coveralls >>>>> supports >>>>> > > >>> JaCoco. >>>>> > > >>> >> >> > >>> > >>> > > > >>> >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >> I will sent and email to INFRA and >>>>> come >>>>> > back >>>>> > > >>> with >>>>> > > >>> >> the >>>>> > > >>> >> >> > >>> response >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > no answer from INFRA >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > meanwhile I was playing with >>>>> coveralls.io >>>>> > > and >>>>> > > >>> I >>>>> > > >>> >> found >>>>> > > >>> >> >> > the >>>>> > > >>> >> >> > >>> our >>>>> > > >>> >> >> > >>> > >>> repo >>>>> > > >>> >> >> > >>> > >>> > is >>>>> > > >>> >> >> > >>> > >>> > > > > somehow already configured, see >>>>> > > >>> >> >> > >>> > >>> > > > > https://coveralls.io/github/ >>>>> > > apache/bookkeeper >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > I don't know whoever did it >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > It seams that in order to have such >>>>> service >>>>> > > use >>>>> > > >>> >> have >>>>> > > >>> >> >> to >>>>> > > >>> >> >> > >>> include >>>>> > > >>> >> >> > >>> > >>> > JaCoCo >>>>> > > >>> >> >> > >>> > >>> > > > > maven plugin, then enable some >>>>> automatic >>>>> > > build >>>>> > > >>> >> using >>>>> > > >>> >> >> > >>> Jenkins or >>>>> > > >>> >> >> > >>> > >>> > Travis >>>>> > > >>> >> >> > >>> > >>> > > > and >>>>> > > >>> >> >> > >>> > >>> > > > > then add another coveralls plugin >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > see this interesting document >>>>> > > >>> >> >> > >>> > >>> > > > > https://github.com/ >>>>> > > >>> asciidoctor/asciidoctor/wiki/ >>>>> > > >>> >> >> > >>> > >>> > > > > Coveralls.io-configuration- >>>>> > > for-maven-projects >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > I think that the starting point is to >>>>> > > integrate >>>>> > > >>> >> JaCoco >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > -- Enrico >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > >> -- Enrico >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> >>>>> > > >>> >> >> > >>> > >>> > > > >>> Sijie >>>>> > > >>> >> >> > >>> > >>> > > > >>> >>>>> > > >>> >> >> > >>> > >>> > > > >>> On Jul 5, 2017 1:57 AM, "Enrico >>>>> > Olivelli" < >>>>> > > >>> >> >> > >>> > eolive...@gmail.com >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > > wrote: >>>>> > > >>> >> >> > >>> > >>> > > > >>> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > I am attaching a report (I am >>>>> using >>>>> > > Google >>>>> > > >>> >> Drive >>>>> > > >>> >> >> as >>>>> > > >>> >> >> > >>> most >>>>> > > >>> >> >> > >>> > >>> email >>>>> > > >>> >> >> > >>> > >>> > > > clients >>>>> > > >>> >> >> > >>> > >>> > > > >>> will >>>>> > > >>> >> >> > >>> > >>> > > > >>> > block attachments with html >>>>> inside) . >>>>> > > >>> >> >> > >>> > >>> > > > >>> > Just untar and play with your >>>>> browser >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > 2017-07-04 18:18 GMT+02:00 Dávid >>>>> > > Szigecsán >>>>> > > >>> < >>>>> > > >>> >> >> > >>> > >>> sige...@gmail.com>: >>>>> > > >>> >> >> > >>> > >>> > > > >>> > > SonarCloud is free for open >>>>> source. >>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > > https://about.sonarcloud.io/ >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > Yes I known >>>>> > > >>> >> >> > >>> > >>> > > > >>> > recently we have activated >>>>> Travis CI, I >>>>> > > >>> think >>>>> > > >>> >> that >>>>> > > >>> >> >> > the >>>>> > > >>> >> >> > >>> > trend >>>>> > > >>> >> >> > >>> > >>> is >>>>> > > >>> >> >> > >>> > >>> > to >>>>> > > >>> >> >> > >>> > >>> > > > >>> > integrate with such free tools >>>>> in order >>>>> > > to >>>>> > > >>> >> achieve >>>>> > > >>> >> >> > the >>>>> > > >>> >> >> > >>> best >>>>> > > >>> >> >> > >>> > >>> > quality >>>>> > > >>> >> >> > >>> > >>> > > > >>> with >>>>> > > >>> >> >> > >>> > >>> > > > >>> > the minimum effort >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > -- Enrico >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > > 2017-07-04 18:15 GMT+02:00 >>>>> Enrico >>>>> > > >>> Olivelli < >>>>> > > >>> >> >> > >>> > >>> > eolive...@gmail.com >>>>> > > >>> >> >> > >>> > >>> > > >: >>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> Il mar 4 lug 2017, 18:08 Dávid >>>>> > > >>> Szigecsán < >>>>> > > >>> >> >> > >>> > >>> sige...@gmail.com> >>>>> > > >>> >> >> > >>> > >>> > ha >>>>> > > >>> >> >> > >>> > >>> > > > >>> > scritto: >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > +1 >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > What do you think about >>>>> sonar? :) >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> I am using sonar in projects >>>>> in my >>>>> > > >>> company, >>>>> > > >>> >> it >>>>> > > >>> >> >> is >>>>> > > >>> >> >> > >>> great. >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> But it needs a server, so it >>>>> is not >>>>> > > >>> easy to >>>>> > > >>> >> use >>>>> > > >>> >> >> > for >>>>> > > >>> >> >> > >>> > >>> everyone >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> Enrico >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > 2017-07-04 17:24 GMT+02:00 >>>>> Enrico >>>>> > > >>> >> Olivelli < >>>>> > > >>> >> >> > >>> > >>> > > eolive...@gmail.com >>>>> > > >>> >> >> > >>> > >>> > > > >: >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > Hi all, >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > as we are in the mood of >>>>> > enhancing >>>>> > > >>> >> >> automatic >>>>> > > >>> >> >> > QA >>>>> > > >>> >> >> > >>> and >>>>> > > >>> >> >> > >>> > >>> > quality >>>>> > > >>> >> >> > >>> > >>> > > of >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > BookKeeper I have created >>>>> this >>>>> > > >>> issue in >>>>> > > >>> >> >> order >>>>> > > >>> >> >> > to >>>>> > > >>> >> >> > >>> > >>> introduce >>>>> > > >>> >> >> > >>> > >>> > > > >>> JaCoCo in >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > the build/test pipeline >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>> https://github.com/apache/ >>>>> > > >>> >> >> > bookkeeper/issues/232 >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > I will do some tests and >>>>> report >>>>> > to >>>>> > > >>> the >>>>> > > >>> >> >> > comunity >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > -- Enrico >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> -- >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> -- Enrico Olivelli >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > ​ >>>>> > > >>> >> >> > >>> > >>> > > > >>> > bookkeeper-server-jacoco.tar.gz >>>>> > > >>> >> >> > >>> > >>> > > > >>> > <https://drive.google.com/ >>>>> > > >>> file/d/0Bz7lsWhNTGF- >>>>> > > >>> >> >> > >>> > OVZFblRpZE00Nz >>>>> > > >>> >> >> > >>> > >>> > > > >>> A/view?usp= >>>>> > > >>> >> >> > >>> > >>> > > > >>> > drive_web> >>>>> > > >>> >> >> > >>> > >>> > > > >>> > ​ >>>>> > > >>> >> >> > >>> > >>> > > > >>> > >>>>> > > >>> >> >> > >>> > >>> > > > >>> >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > >> >>>>> > > >>> >> >> > >>> > >>> > > > > >>>>> > > >>> >> >> > >>> > >>> > > > >>>>> > > >>> >> >> > >>> > >>> > > >>>>> > > >>> >> >> > >>> > >>> > -- >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> > -- Enrico Olivelli >>>>> > > >>> >> >> > >>> > >>> > >>>>> > > >>> >> >> > >>> > >>> >>>>> > > >>> >> >> > >>> > >> -- >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > >> -- Enrico Olivelli >>>>> > > >>> >> >> > >>> > >> >>>>> > > >>> >> >> > >>> > > >>>>> > > >>> >> >> > >>> > > -- >>>>> > > >>> >> >> > >>> > >>>>> > > >>> >> >> > >>> > >>>>> > > >>> >> >> > >>> > -- Enrico Olivelli >>>>> > > >>> >> >> > >>> > >>>>> > > >>> >> >> > >>> >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > >> >>>>> > > >>> >> >> > > -- >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > >>>>> > > >>> >> >> > -- Enrico Olivelli >>>>> > > >>> >> >> > >>>>> > > >>> >> >> >>>>> > > >>> >> > >>>>> > > >>> >> > -- >>>>> > > >>> >> >>>>> > > >>> >> >>>>> > > >>> >> -- Enrico Olivelli >>>>> > > >>> >> >>>>> > > >>> > >>>>> > > >>> >>>>> > > >> -- >>>>> > > > >>>>> > > > >>>>> > > > -- Enrico Olivelli >>>>> > > > >>>>> > > >>>>> > >>>>> >>>> >>>> >>> >> -- > > > -- Enrico Olivelli >