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

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

Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/486#issuecomment-95938335
  
    @HeartSaVioR great to hear that the setup is complete.  I'm not sure if we 
want to merge this in as is, knowing that the majority of test cases will fail, 
or if we want to wait for the tests to pass more regularly.


> 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)

Reply via email to