You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bookkeeper.apache.org by Sijie Guo <gu...@gmail.com> on 2017/07/19 22:30:09 UTC
CI
Two problems:
- it seems that maven-checkstyle doesn't run well for bookkeeper-stats-api
module. so the CI build fails. we might need to take a look into it.
- it seems that -Dmaven.test.failure.ignore=true is used for this job. this
hides problems in master. so it would be good to separate 'deploy' from
other build targets. for example, remove 'deloy' and
'Dmaven.test.failure.ignore=true' from bookkeeper-master, create a separate
job 'bookkeeper-publish-snapshot-artifacts' only for publishing snapshot
artifacts.
Any thoughts?
- Sijie
Re: CI
Posted by Sijie Guo <gu...@gmail.com>.
Cool. Thank you Enrico.
On Wed, Jul 19, 2017 at 9:53 PM, Enrico Olivelli <eo...@gmail.com>
wrote:
> Il gio 20 lug 2017, 00:30 Sijie Guo <gu...@gmail.com> ha scritto:
>
> > Two problems:
> >
> > - it seems that maven-checkstyle doesn't run well for
> bookkeeper-stats-api
> > module. so the CI build fails. we might need to take a look into it.
> >
>
> I will work on it
>
> >
> > - it seems that -Dmaven.test.failure.ignore=true is used for this job.
> this
> > hides problems in master. so it would be good to separate 'deploy' from
> > other build targets. for example, remove 'deloy' and
> > 'Dmaven.test.failure.ignore=true' from bookkeeper-master, create a
> separate
> > job 'bookkeeper-publish-snapshot-artifacts' only for publishing snapshot
> > artifacts.
> >
>
> Yesterday I had set it to true to make coveralls work, because if the
> builds fails it seems not to push data.
> I am working on it
>
> >
> >
> >
> >
> > Any thoughts?
> >
>
>
> If it is ok for you I will take care of setting up correctly all of these
> things
>
> Enrico
>
> >
> > - Sijie
> >
> --
>
>
> -- Enrico Olivelli
>
Re: CI
Posted by Enrico Olivelli <eo...@gmail.com>.
Il gio 20 lug 2017, 00:30 Sijie Guo <gu...@gmail.com> ha scritto:
> Two problems:
>
> - it seems that maven-checkstyle doesn't run well for bookkeeper-stats-api
> module. so the CI build fails. we might need to take a look into it.
>
I will work on it
>
> - it seems that -Dmaven.test.failure.ignore=true is used for this job. this
> hides problems in master. so it would be good to separate 'deploy' from
> other build targets. for example, remove 'deloy' and
> 'Dmaven.test.failure.ignore=true' from bookkeeper-master, create a separate
> job 'bookkeeper-publish-snapshot-artifacts' only for publishing snapshot
> artifacts.
>
Yesterday I had set it to true to make coveralls work, because if the
builds fails it seems not to push data.
I am working on it
>
>
>
>
> Any thoughts?
>
If it is ok for you I will take care of setting up correctly all of these
things
Enrico
>
> - Sijie
>
--
-- Enrico Olivelli