[Buildbot] #2968: Include enough data to calculate blamelists

Buildbot trac trac at buildbot.net
Fri Oct 24 14:48:11 UTC 2014


#2968: Include enough data to calculate blamelists
---------------------+---------------------
Reporter:  dustin    |      Owner:
    Type:  defect    |     Status:  new
Priority:  critical  |  Milestone:  0.9.0
 Version:  master    |   Keywords:  changes
---------------------+---------------------
 {{{
 10:41 < djmitche> so "what changes are included in a build" is actually a
 function of *two* builds -- this one and the previous
 10:41 < djmitche> think of "what changes are included in a build" as "git
 log"
 10:41 < djmitche> if you just give it one revision, it gives you all
 changes from the first commit to that one
 10:42 < djmitche> to get a limited set of commits, you need to give it two
 revisions
 10:43 < djmitche> what's missing right now in Buildbot is a 'parent'
 pointer for changes
 10:44 < djmitche> with that in place, the MailNotifier, etc. could
 calculate the equivalent of 'git log last_build..this_build'
 }}}

 So this involves:
  * adding a 'parents' field to changes (one-to-many)
  * defining a utility to return the changes between two sourcestamps
  * using that utility in status displays to define blamelists

 Marked as a defect since this is re-implementing behavior that was
 available in eight (only with more sense and less insanity).

--
Ticket URL: <http://trac.buildbot.net/ticket/2968>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the bugs mailing list