You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2015/04/07 00:56:12 UTC

[jira] [Commented] (STORM-704) Apply Travis CI

    [ https://issues.apache.org/jira/browse/STORM-704?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14482135#comment-14482135 ] 

ASF GitHub Bot commented on STORM-704:
--------------------------------------

Github user HeartSaVioR commented on the pull request:

    https://github.com/apache/storm/pull/486#issuecomment-90273015
  
    @revans2 
    Could you take a look? 
    We were talking about adopting Travis CI and I feel you would like this, and there're no hurdle left (except Apache Infra)


> Apply Travis CI
> ---------------
>
>                 Key: STORM-704
>                 URL: https://issues.apache.org/jira/browse/STORM-704
>             Project: Apache Storm
>          Issue Type: Improvement
>         Environment: Travis CI
>            Reporter: Jungtaek Lim
>            Assignee: Jungtaek Lim
>            Priority: Minor
>
> Now Apache Storm takes advantage of Github, we can apply Travis CI to some more advantages.
> - Build matrix
> -- Travis CI supports various JDK versions (openjdk6, openjdk7, oraclejdk7, oraclejdk8), and it can be tested separately.
> - Build automatically
> -- pushed new commits, any new PRs
> - Integrated with Github
> -- Contributors can see his/her PR breaks compilation / test in some minutes.
> -- If he/she adds commits to PR, Travis builds it automatically and update build result.
> Please see [https://travis-ci.org/xetorthio/jedis] for example.
> There're some hurdles applying Travis CI to Apache Storm project, but we can overcome these and finally get great CI.
> Current hurdles
> - asfgit should manage Travis CI setup for the first time
> -- other Apache projects already did it by requesting it to INFRA
> --- ex. [https://issues.apache.org/jira/browse/INFRA-6161]
> - Travis CI restricts stdout with 4M which is too small for Storm maven output.
> -- Change log level in tests to WARN
> - In storm-core, we can't see tests failure information on stdout cause it just prints 'clojure failed'.
> -- We need to find a way to upload surefire / clojure tests report files to somewhere, and uploaded files should be visible easily.
> --- Travis supports uploading artifacts to S3 by [https://github.com/travis-ci/artifacts], but S3 is not free.
> -- I'm not familiar with Clojure, but can we print tests summary with clojure tests as same as Java junit tests?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)