[Buildbot] #3313: Improve nine migration notes

Buildbot trac trac at buildbot.net
Sun Jul 19 23:51:02 UTC 2015


#3313: Improve nine migration notes
------------------------+--------------------
Reporter:  dustin       |       Owner:
    Type:  enhancement  |      Status:  new
Priority:  major        |   Milestone:  0.9.0
 Version:  0.8.10       |  Resolution:
Keywords:               |
------------------------+--------------------

Comment (by dustin):

 More from email:

 ----

 Migration pain-points that can be mined from release notes

  *  MailNotifier moved to: buildbot.reporters.mail
  *  messageFormatter argument to MailNotifier is gone.  Replacement
 buildbot.status.message.MessageFormatter doesn’t appear to have any
 documentation.
  *  Getting the status from the build/buildstep doesn’t actually return a
 populated object? Something like that.
      *  Thankfully the non-status objects contained enough information for
 this not to be an issue (e.g. executedSteps in the process.build.Build)

 Migration surprises

  *  ‘status’ key is not allowed in configuration - any appearance of it
 will cause startup to fail.  No notes about where things are supposed to
 be.  Had to go spelunking through the manual.
      *  MailNotifier needs to be added to ‘services’ not ‘status’
      *  PBChangeSource goes into ‘change_source’ not ‘status’
      *  I’m sure there are others
  *  GitLab web hook moved to buildbot.www.hooks.gitlab was
 buildbot.status.web.hooks.gitlab (normally probably wouldn’t care since
 these are normally accessed by name but my gitlab change-hook extends the
 built-in one).
  *  mergeRequests option no longer understood by Builder
  *  Had a doIf that performed an action if the previous step failed.  It
 was doing build.getStatus() & looking for the matching step that way.
 Build now has executedSteps so was able to migrate.
  *  BuildStep#describe is not allowed for new-style steps in 0.9 but is in
 0.8.12
  *  authz is gone with no apparent replacement.  In other words, no way to
 allow certain actions or disallow others.  Unclear if this is actually
 important.

--
Ticket URL: <http://trac.buildbot.net/ticket/3313#comment:1>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the bugs mailing list