[Buildbot-commits] [Buildbot] #1895: reconfig that removes a builder may fail first time

[Buildbot]

#1895: reconfig that removes a builder may fail first time
--------------------+----------------------
Reporter:  dustin   |      Owner:
    Type:  defect   |     Status:  new
Priority:  major    |  Milestone:  0.8.+
 Version:  0.8.3p1  |   Keywords:  reconfig
--------------------+----------------------
 If you remove a builder, the checks seem to occur in the wrong order, and
 schedulers will complain about pointing to a newly non-existent builder:

 {{{
 2011-03-20 02:45:31+0100 [-] Unhandled Error
         Traceback (most recent call last):
           File
 "/home/buildbot/metabuildbot/src/master/buildbot/master.py", line 700, in
 loadTheConfigFile
             d = self.loadConfig(f)
           File
 "/home/buildbot/metabuildbot/src/master/buildbot/master.py", line 1039, in
 loadConfig
             d.addCallback(do_load)
           File "/home/buildbot/sandbox/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 297, in addCallback
             callbackKeywords=kw)
           File "/home/buildbot/sandbox/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 286, in addCallbacks
             self._runCallbacks()
         --- <exception caught here> ---
           File "/home/buildbot/sandbox/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 542, in _runCallbacks
             current.result = callback(current.result, *args, **kw)
           File
 "/home/buildbot/metabuildbot/src/master/buildbot/master.py", line 914, in
 do_load
             "%s uses unknown builder %s" % (s, b)
         exceptions.AssertionError: <buildbot.schedulers.basic.Scheduler
 instance at 0x2a7fa28> uses unknown builder slave-minimata
 }}}

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