[tahoe-lafs-trac-stream] [tahoe-lafs] #2052: Automate testing of merge requests to streamline review.

tahoe-lafs trac at tahoe-lafs.org
Sat Aug 31 15:05:57 UTC 2013


#2052: Automate testing of merge requests to streamline review.
--------------------------+-----------------------------------------
     Reporter:  nejucomo  |      Owner:  daira
         Type:  defect    |     Status:  new
     Priority:  normal    |  Milestone:  undecided
    Component:  unknown   |    Version:  1.10.0
   Resolution:            |   Keywords:  dev-infrastructure buildbot
Launchpad Bug:            |
--------------------------+-----------------------------------------

Comment (by zooko):

 I had trouble understanding comment:15 until I went ahead and read
 http://about.travis-ci.org/blog/announcing-pull-request-support/ . Now it
 is all clear! Here is the desired functionality (desired at least by
 amontero and by me):

 1. A new/casual contributor clicks the "fork this" button on github.
 2. The contributor write a patch and pushes it to their repo on github.
 3. The contributor pushes the "pull request" button on github.
 4. Very soon, and with no manual intervention from any Tahoe-LAFS
 developer, some automated bot somewhere runs the unit tests on the version
 in the pull request, and posts a comment on the pull request indicating
 whether the tests pass or not, and including a link to the test results.

 This is exactly what I want, and apparently other people also want this
 and have already implemented it, using travis-ci. It looks like we might
 be able to get the travis-ci people to do this for us and avoid all the
 problems of implementing it ourselves in our buildbot scripts, and the
 problems of security of our buildslaves.

 I intend to investigate signing up for this. It looks like, from the end
 of http://about.travis-ci.org/blog/announcing-pull-request-support/, that
 you have to pay ("donate") to the travis-ci project in order to get in on
 this.

-- 
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2052#comment:16>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage


More information about the tahoe-lafs-trac-stream mailing list