Infra has enabled Travis-CI for us. mvn clean install -DskipTests is executed on all PR-s and commits from now on.
However, builds occasionally fail with heap space error. To fix that issue, I have opened a pull request: https://github.com/apache/flume/pull/75 . Any reviews and comments are welcome. Thank you, Donat On Fri, Oct 21, 2016 at 6:14 PM, Mike Percy <mpe...@apache.org> wrote: > Hi Lior! > > No, my message was not directed at you, or any person in particular. I > intended this message for those paying attention to this topic to try and > set expectations for how decision making for things like this usually (and > hopefully) works in Apache: If you are willing to do the work to get > something done, then it will probably get done the way you want! > > (Assuming the end result is something that others in the community want -- > in this case, it's very basic pre-commit checks. I think we have consensus > that pre-commit checks are something that would be a net benefit to > everyone.) > > Sorry for any confusion! And don't let me stop people from voicing their > views and concerns. > > Best, > Mike > > On Fri, Oct 21, 2016 at 4:44 PM, Lior Zeno <liorz...@gmail.com> wrote: > >> Mike, I was not holding Donat back. I was just suggesting ways to configure >> Jenins, per Donat's request. I'm sorry if my former post delivered the >> wrong message. >> >> On Fri, Oct 21, 2016 at 6:29 PM, Mike Percy <mpe...@apache.org> wrote: >> >> > Personally I prefer Jenkins over TravisCI for various reasons however if >> > Donat is willing to do the work of adding pre-commit checks on PRs via >> > Travis then I say let him do it, in the Apache spirit of "let they that >> do >> > the work make the decisions". >> > >> > If someone actually spends the time to set up Jenkins and configure it to >> > do the same thing, then great, let's switch when it's ready. >> > >> > Note that only ASF committers have access to Jenkins so non-committers >> will >> > need to work with a committer to get it done if they want to help. >> > >> > Mike >> > >> > On Fri, Oct 21, 2016 at 3:46 PM, Lior Zeno <liorz...@gmail.com> wrote: >> > >> > > There are many ways to do it, for example: >> > > https://www.theguild.nl/building-github-pull-requests-using- >> > > jenkins-pipelines/ >> > > or https://www.theguild.nl/building-github-pull-requests-with-jenkins/ >> > for >> > > earlier versions of Jenkins. >> > > I do not really care if it would be Jenkins or Travis, but I do think >> > that >> > > we can get Jenkins configured faster since we already have it. I can >> help >> > > with the configuration. >> > > >> > > On Fri, Oct 21, 2016 at 5:17 PM, Balazs Donat Bessenyei < >> > > bes...@cloudera.com >> > > > wrote: >> > > >> > > > As I haven't received any objections to enabling Travis, I'm going to >> > > > ask INFRA to enable it for Flume soon. >> > > > >> > > > This change would help submitting and reviewing pull requests. >> > > > >> > > > If someone figures out how we could use Jenkins for this purpose, we >> > > > can always disable Travis. >> > > > >> > > > PS. there are more projects using Travis: >> > > > https://issues.apache.org/jira/browse/INFRA-12757?jql= >> > > > project%20%3D%20INFRA%20AND%20text%20~%20travis%20ORDER% >> > > > 20BY%20updated%20DESC >> > > > >> > > > On Fri, Oct 14, 2016 at 5:41 PM, Attila Simon <s...@cloudera.com> >> > wrote: >> > > > > Denes I'm happy to help you in this endeavor of setting up jenkins >> > job >> > > > for >> > > > > verifying pull requests. >> > > > > >> > > > > >> > > > > *Attila Simon* >> > > > > Software Engineer >> > > > > Email: s...@cloudera.com >> > > > > >> > > > > [image: Cloudera Inc.] >> > > > > >> > > > > On Fri, Oct 14, 2016 at 2:47 PM, Denes Arvay <de...@cloudera.com> >> > > wrote: >> > > > > >> > > > >> I'd also vote for Jenkins with github PRs. >> > > > >> I just checked Mesos and the PRs are checked by Travis, or at >> least >> > > they >> > > > >> experienced with it, there's a short discussion regarding to >> Travis >> > at >> > > > >> https://github.com/apache/mesos/pull/165 >> > > > >> >> > > > >> As for the jenkins pull request job I'd be happy to set it up or >> > help >> > > > >> setting it up. >> > > > >> >> > > > >> Denes >> > > > >> >> > > > >> On Fri, Oct 14, 2016 at 2:15 PM Lior Zeno <liorz...@gmail.com> >> > wrote: >> > > > >> >> > > > >> Are we switching to PRs from patches + RB? In Apache Mesos, they >> > have >> > > a >> > > > >> >> > > > >> review bot that can leave a comment on the patch, we could try and >> > > port >> > > > it >> > > > >> >> > > > >> to Flume. I think they use Jenkins too. >> > > > >> >> > > > >> >> > > > >> >> > > > >> On Fri, Oct 14, 2016 at 3:11 PM, Balazs Donat Bessenyei < >> > > > >> bes...@cloudera.com >> > > > >> >> > > > >> > wrote: >> > > > >> >> > > > >> >> > > > >> >> > > > >> > If the same function can be achieved with Jenkins and it's easy >> > > > >> >> > > > >> > (+quick) to set up, I'm totally happy with that. >> > > > >> >> > > > >> > >> > > > >> >> > > > >> > What do we have to do to enable Jenkins builds on PR-s? >> > > > >> >> > > > >> > >> > > > >> >> > > > >> > On Fri, Oct 14, 2016 at 2:05 PM, Lior Zeno <liorz...@gmail.com> >> > > > wrote: >> > > > >> >> > > > >> > > There are ways to do the same with Jenkins, for instance, see >> > this >> > > > SO >> > > > >> >> > > > >> > > thread >> > > > >> >> > > > >> > > http://stackoverflow.com/questions/37661602/how-to-set- >> > > > >> >> > > > >> > up-a-github-pull-request-build-in-a-jenkinsfile >> > > > >> >> > > > >> > > >> > > > >> >> > > > >> > > On Fri, Oct 14, 2016 at 11:09 AM, Balazs Donat Bessenyei < >> > > > >> >> > > > >> > > bes...@cloudera.com> wrote: >> > > > >> >> > > > >> > > >> > > > >> >> > > > >> > >> My primary reason for Travis (vs. Jenkins) was that I have >> > > > experience >> > > > >> >> > > > >> > with >> > > > >> >> > > > >> > >> it. >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> And it leaves these happy little checkmarks: >> > > > >> >> > > > >> > >> https://github.com/sebastianbergmann/phpunit/ >> pull/1051/commits >> > > on >> > > > the >> > > > >> >> > > > >> > >> commits and messages as seen on >> > > > >> >> > > > >> > >> https://github.com/apache/hive/pull/107 . >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> Jenkins is probably configurable to achieve similar function. >> > > > However, >> > > > >> >> > > > >> > >> I have no idea how to do such. (And could not find an example >> > > when >> > > > I >> > > > >> >> > > > >> > >> did a quick search.) >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> Are there any disadvantages of enabling Travis on Flume? >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> Thank you, >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> Donat >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> On Thu, Oct 13, 2016 at 6:06 PM, Lior Zeno < >> liorz...@gmail.com >> > > >> > > > >> wrote: >> > > > >> >> > > > >> > >> > Jenkins can do PRs as well. If we can upgrade Jenkins to >> 2.0, >> > > we >> > > > >> will >> > > > >> >> > > > >> > be >> > > > >> >> > > > >> > >> > able to define the build step via Jenkinsfile which becomes >> > > very >> > > > >> >> > > > >> > similar >> > > > >> >> > > > >> > >> to >> > > > >> >> > > > >> > >> > Travis. >> > > > >> >> > > > >> > >> > Is there any reason to prefer Travis over Jenkins in our >> > case? >> > > > >> >> > > > >> > >> > >> > > > >> >> > > > >> > >> > On Thu, Oct 13, 2016 at 7:01 PM, Balazs Donat Bessenyei < >> > > > >> >> > > > >> > >> bes...@cloudera.com >> > > > >> >> > > > >> > >> >> wrote: >> > > > >> >> > > > >> > >> > >> > > > >> >> > > > >> > >> >> Hi All, >> > > > >> >> > > > >> > >> >> >> > > > >> >> > > > >> > >> >> Having something that checks proposed patches (PR-s >> > > especially) >> > > > >> >> > > > >> > >> >> automatically would help a lot with the development on >> > Flume. >> > > > >> >> > > > >> > >> >> >> > > > >> >> > > > >> > >> >> I think, Travis-CI could be an easy solution and (afaik) >> > we'd >> > > > only >> > > > >> >> > > > >> > have >> > > > >> >> > > > >> > >> to >> > > > >> >> > > > >> > >> >> ask infra to enable it for us. >> > > > >> >> > > > >> > >> >> >> > > > >> >> > > > >> > >> >> Please, let me know your thoughts. >> > > > >> >> > > > >> > >> >> >> > > > >> >> > > > >> > >> >> Thank you, >> > > > >> >> > > > >> > >> >> >> > > > >> >> > > > >> > >> >> Donat >> > > > >> >> > > > >> > >> >> >> > > > >> >> > > > >> > >> >> > > > >> >> > > > >> > >> > > > >> >> > > > >> > > >> > >>