Ok, I'll check it out. Il mar 3 ott 2017, 19:52 Sijie Guo <guosi...@gmail.com> ha scritto:
> It seems that the jacoco was not disabled at master build. so all the > master builds are still failing. I just disabled it. > > - Sijie > > On Wed, Sep 20, 2017 at 11:51 PM, Enrico Olivelli <eolive...@gmail.com> > wrote: > >> 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 >>> >> >> > -- -- Enrico Olivelli